Change in osmo-mgw[master]: add DLCX command statistics to osmo-mgw
gerrit-no-reply at lists.osmocom.org
Fri Nov 9 10:16:12 UTC 2018
Stefan Sperling has posted comments on this change. ( https://gerrit.osmocom.org/11521 )
Change subject: add DLCX command statistics to osmo-mgw
Patch Set 4:
PS4, Line 1510:
> rate_ctr_group_alloc() may return NULL also for invalid naming, so I would rather keep these.
Seriously, you want me to change this _again_? Have you read the previous discussion?
It seems this entire patch is getting hung up just because we won't agree on something as basic as how to check for allocation failures in osmocom.
I'd rather keep it as it is now instead of bringing this question back up with Pau and Harald all over again.
The code it would likely crash in talloc_set_destructor() on the next line anyway.
By the way, shouldn't rate_ctr_group_alloc() allow its caller to differentiate between memory allocation failures and an invalid counter group description?
To view, visit https://gerrit.osmocom.org/11521
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Stefan Sperling <ssperling at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Stefan Sperling <ssperling at sysmocom.de>
Gerrit-Comment-Date: Fri, 09 Nov 2018 10:16:12 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log