Attention is currently required from: fixeria.

pespin has posted comments on this change by fixeria. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/42602?usp=email )

Change subject: sgsn_libgtp: fix NULL dereference in cb_delete_context()
......................................................................


Patch Set 1:

(2 comments)

Patchset:

PS1:
Is there some ticket related to this scenario?


File src/sgsn/sgsn_libgtp.c:

https://gerrit.osmocom.org/c/osmo-sgsn/+/42602/comment/e7107333_043cfb46?usp=email
 :
PS1, Line 708:  if (!pctx)
So iiuc this patch fixes a crash happening after a condition which shouldn't 
have happened in first place? Maybe Try to understand why this condition is 
happening and add in that place an ASSERT or panic so it can be noticed and 
debugged further?



--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/42602?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I7d800766b08d87b6f2b23a0cd45435925f9998ff
Gerrit-Change-Number: 42602
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Tue, 14 Apr 2026 16:17:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to