gerrit-log August 2022

gerrit-log@lists.osmocom.org

3 participants
2402 discussions

Change in osmo-cbc[master]: cbsp/sbcap: Fix memleak in error condition (tx without link)

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-cbc/+/28878 ) Change subject: cbsp/sbcap: Fix memleak in error condition (tx without link) ...................................................................... cbsp/sbcap: Fix memleak in error condition (tx without link) Change-Id: I95f6ab4e642529afc583b0747450b83af1e8f32e --- M src/cbsp_link.c M src/sbcap_link.c 2 files changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/78/28878/1 diff --git a/src/cbsp_link.c b/src/cbsp_link.c index a6462ee..5587f58 100644 --- a/src/cbsp_link.c +++ b/src/cbsp_link.c @@ -325,6 +325,7 @@ if (!link) { LOGP(DCBSP, LOGL_NOTICE, "Cannot transmit %s: no connection\n", get_value_string(cbsp_msg_type_names, cbsp->msg_type)); + talloc_free(cbsp); return -ENOLINK; } diff --git a/src/sbcap_link.c b/src/sbcap_link.c index b2a5aa2..07ada0c 100644 --- a/src/sbcap_link.c +++ b/src/sbcap_link.c @@ -392,7 +392,8 @@ if (!link) { LOGP(DSBcAP, LOGL_NOTICE, "Cannot transmit msg %s: no connection\n", sbcap_pdu_get_name(pdu)); - return -ENOLINK; + rc = -ENOLINK; + goto ret_free; } LOGPSBCAPC(link, LOGL_INFO, "Tx msg %s\n", -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28878 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: I95f6ab4e642529afc583b0747450b83af1e8f32e Gerrit-Change-Number: 28878 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 10 months

Change in osmo-cbc[master]: sbcap: Fix NULL pointer dereference

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-cbc/+/28877 ) Change subject: sbcap: Fix NULL pointer dereference ...................................................................... sbcap: Fix NULL pointer dereference Related: Coverity CID#275347 Change-Id: I9ae1cad4ce4e3fe6cfd67ac6058fb7345a3dd49e --- M src/sbcap_link_fsm.c 1 file changed, 6 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/77/28877/1 diff --git a/src/sbcap_link_fsm.c b/src/sbcap_link_fsm.c index 685ce7d..3b929ed 100644 --- a/src/sbcap_link_fsm.c +++ b/src/sbcap_link_fsm.c @@ -302,12 +302,13 @@ LOGPSBCAPC(link, LOGL_ERROR, "SBcAP initiatingMessage procedure=%ld not implemented?\n", pdu->choice.initiatingMessage.procedureCode); err_ind_pdu = sbcap_gen_error_ind(link, SBcAP_Cause_valid_message_not_identified, pdu); - if (err_ind_pdu) - cbc_sbcap_link_tx(link, err_ind_pdu); - else + if (!err_ind_pdu) { LOGPSBCAPC(link, LOGL_ERROR, - "Tx SBc-AP %s: msg gen failed\n", - sbcap_pdu_get_name(err_ind_pdu)); + "Tx SBc-AP Error-Indication: msg gen failed\n"); + } else if (cbc_sbcap_link_tx(link, err_ind_pdu) < 0) { + LOGPSBCAPC(link, LOGL_ERROR, + "Tx SBc-AP Error-Indication failed\n"); + } return 0; } break; -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28877 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: I9ae1cad4ce4e3fe6cfd67ac6058fb7345a3dd49e Gerrit-Change-Number: 28877 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 10 months

Jump to page:

2025

2024

2023

2022

gerrit-log August 2022