[MERGED] osmo-msc[master]: libmsc: bssap: Catch TLV parse failures

2018-02-15 Thread Pau Espin Pedrol
Pau Espin Pedrol has submitted this change and it was merged. Change subject: libmsc: bssap: Catch TLV parse failures .. libmsc: bssap: Catch TLV parse failures Change-Id: I1d1951f4a5daf200e85c76fea14a35e952491d27 --- M src/lib

osmo-msc[master]: libmsc: bssap: Catch TLV parse failures

2018-02-14 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/6451 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1d1951f4a5daf200e85c76fea14a35e952491d27 Gerrit-PatchSet: 1 Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit

[PATCH] osmo-msc[master]: libmsc: bssap: Catch TLV parse failures

2018-02-14 Thread Pau Espin Pedrol
Review at https://gerrit.osmocom.org/6451 libmsc: bssap: Catch TLV parse failures Change-Id: I1d1951f4a5daf200e85c76fea14a35e952491d27 --- M src/libmsc/a_iface_bssap.c 1 file changed, 7 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/51/6451/1 dif

osmo-msc[master]: libmsc: bssap: Catch TLV parse failures

2018-02-14 Thread Harald Welte
Patch Set 1: Code-Review-1 (1 comment) Sorry, I think it's ugly to have the same code copy+pasted everywhere. Can't we simply do the TLV parsing at a common/shared point and hand the already-parsed tlv_parsed into the indiividual functions? This way there's reduction of code duplication and

[PATCH] osmo-msc[master]: libmsc: bssap: Catch TLV parse failures

2018-02-14 Thread Pau Espin Pedrol
Review at https://gerrit.osmocom.org/6448 libmsc: bssap: Catch TLV parse failures Change-Id: I6aef9a94fa5b2e0b62a9c1744b8e18e5985f788f --- M src/libmsc/a_iface_bssap.c 1 file changed, 49 insertions(+), 9 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/48/6448/1 d