Attention is currently required from: pespin.
fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/42213?usp=email )
Change subject: stp: Enable tracking of NULL memory context ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File stp/stp_main.c:
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42213/comment/75d6bcf3_5f4a60... : PS1, Line 220: talloc_enable_null_tracking I know we already do this in many projects (and not voting against doing this), but there must be a reason why it's disabled by default in talloc. One thing that comes to my mind is potential performance impact. If this is a concern, we may want to enable NULL-tracking conditionally by introducing an environment variable.