fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/35334?usp=email )
Change subject: vgcs_fsm: fix NULL pointer dereference in vgcs_call_fsm_busy() ...................................................................... vgcs_fsm: fix NULL pointer dereference in vgcs_call_fsm_busy() Change-Id: I14ad41bfa1ab3455a6a175cfb05341ca12c8d73e Fixes: CID#322299 --- M src/osmo-bsc/vgcs_fsm.c 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved diff --git a/src/osmo-bsc/vgcs_fsm.c b/src/osmo-bsc/vgcs_fsm.c index 37473c5..1f2bbef 100644 --- a/src/osmo-bsc/vgcs_fsm.c +++ b/src/osmo-bsc/vgcs_fsm.c @@ -317,8 +317,10 @@ break; case VGCS_EV_MSC_DTAP: LOG_CALL(conn, LOGL_DEBUG, "MSC sends DTAP message to talker.\n"); - if (!conn->vgcs_call.talker) + if (!conn->vgcs_call.talker) { msgb_free(data); + break; + } rc = osmo_fsm_inst_dispatch(conn->vgcs_call.talker->vgcs_chan.fi, VGCS_EV_MSC_DTAP, data); if (rc < 0) msgb_free(data); -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/35334?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I14ad41bfa1ab3455a6a175cfb05341ca12c8d73e Gerrit-Change-Number: 35334 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanits...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de> Gerrit-Reviewer: pespin <pes...@sysmocom.de> Gerrit-MessageType: merged