Change in ...libosmocore[master]: fix: vty crash by logging to killed telnet session
gerrit-no-reply at lists.osmocom.org
Tue Aug 27 16:24:52 UTC 2019
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/15265 )
Change subject: fix: vty crash by logging to killed telnet session
Patch Set 2:
I'm sorry but I don't agree here. I don't think adding lots of checks against null pointers all around the code to make sure we don't crash on situation that shouldn't happen is good practice. It makes code harder to follow and make people infer bad assumptions, and masks bugs.
I'm not making a huge thing of it, that's why I think these checks shouldn't be there.
If someone else agrees to merge this one go on, but I don't like it. I expect the correct fix for this issue (that is, as you said, removing the conn from the target list before printing in telnet_close_client()).
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/15265
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-CC: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Tue, 27 Aug 2019 16:24:52 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log