pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/16208 )
Change subject: enlarge the GSUP message headroom ...................................................................... Patch Set 10: (1 comment) https://gerrit.osmocom.org/c/osmo-hlr/+/16208/10/src/gsup_server.c File src/gsup_server.c: https://gerrit.osmocom.org/c/osmo-hlr/+/16208/10/src/gsup_server.c@40 PS10, Line 40: struct msgb *msg = msgb_alloc_headroom(1024+512, 512, label); That looks like a big increase imho, 128 or 256 looks more realistic for longer names. -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/16208 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: I7d86f2dadcae29fe1550ea2c9773394ab31a837b Gerrit-Change-Number: 16208 Gerrit-PatchSet: 10 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Comment-Date: Tue, 26 Nov 2019 09:53:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
