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 only one place where we need to check for the 
tlv parser result.
File src/libmsc/a_iface_bssap.c:

Line 228:               LOGP(DBSSAP, LOGL_ERROR, "Failed parsing TLV -- 
discarding message!\n");
makes sense to hexdump the message.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I6aef9a94fa5b2e0b62a9c1744b8e18e5985f788f
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <>
Gerrit-Reviewer: Harald Welte <>
Gerrit-Reviewer: Jenkins Builder
Gerrit-HasComments: Yes

Reply via email to