Max has posted comments on this change. ( https://gerrit.osmocom.org/10216 )

Change subject: ipa_ccm_idtag_parse*: Fix reported length value
......................................................................


Patch Set 3: Code-Review+1

> @Max: there's an unanswered question from Neels.

Sure, but if I understood it correctly it's not related to commit itself so it 
shouldn't stop us from merging this.

FYI, I've stumbled across related commit 
da5f41cdada843ede0a360dc72601fc2125d0677 in libosmo-netif.


--
To view, visit https://gerrit.osmocom.org/10216
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I435aaa33605bd48635715a2c81aa2d231c1abf51
Gerrit-Change-Number: 10216
Gerrit-PatchSet: 3
Gerrit-Owner: Harald Welte <[email protected]>
Gerrit-Assignee: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Holger Freyther <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Comment-Date: Thu, 21 Feb 2019 09:39:05 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes

Reply via email to