Change in osmo-sgsn[master]: Store GSN address in libosmocore struct

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Sat Jan 5 03:59:12 UTC 2019


Neels Hofmeyr has posted comments on this change. ( https://gerrit.osmocom.org/12352 )

Change subject: Store GSN address in libosmocore struct
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/#/c/12352/2/src/gprs/gtphub.c
File src/gprs/gtphub.c:

https://gerrit.osmocom.org/#/c/12352/2/src/gprs/gtphub.c@196
PS2, Line 196: 	gsna->length = 4;
if using the new struct, should also populate the type member throughout the code. Alternatively spread API doc everywhere that the field is ignored, but that's ugly IMHO. That kind of makes it not worthwhile bothering, I guess.



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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6ed32a91483dc608c47df77869033a6e891e9e6a
Gerrit-Change-Number: 12352
Gerrit-PatchSet: 2
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Stefan Sperling <stsp at stsp.name>
Gerrit-Comment-Date: Sat, 05 Jan 2019 03:59:12 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190105/6c370aec/attachment.htm>


More information about the gerrit-log mailing list