Attention is currently required from: osmith, fixeria.
View Change
1 comment:
File src/sbcap_server.c:
Patch Set #1, Line 62: msgb_alloc_c
I would simply use msgb_alloc() here, which uses dedicated 'msgb' context.
I already changed this in a previous review, which made this crash. TBH I don't care, any of the 2 is good for me, so let's simply merge this.
To view, visit change 28691. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I201f44efa24a514e0087b6dcd01115b9b2b8e9db
Gerrit-Change-Number: 28691
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Attention: osmith <osmith@sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Comment-Date: Wed, 20 Jul 2022 17:21:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: comment