Actions
Bug #5329
closedosmo_wqueue_enqueue*() doesn't msgb_free in case enqueing fails
Start date:
11/25/2021
Due date:
% Done:
90%
Spec Reference:
Description
This is a somewhat questionable API decision and we need to audit our code base accordingly.
functions that do it correctly¶
- ctrl_cmd_send
- osmo-bsc:bsc_rf_ctrl.c:send_resp
- osmo-bsc:meas_feed.c:process_meas_rep
- osmo-mgw:mgcp_client_tx
- osmo-msc:smpp_smsc.c:pack_and_send
- osmo-msc:smpp_mirror.c:pack_and_send
- osmo-pcap:osmo_client_network.c:write_data
- osmo-pcu:osmo-bts-*/*_l1_if.c
- osmcoom-bb:layer23:osmo_send_l1
- osmocom-bb:layer23:sap_fsm:sap_send_msgb
functions that get it wrong¶
- osmo-bts:osmo-bts-{lc15,oc2g,octphy}/*_l1_if.c:ph_tch_req
- osmo-iuh/src/tests/hnb-test.c (just a test cse)
functions that pass return value of osmo_wqueue_enqueue* up to caller¶
We need to trace the usage of those...
- gsmtap_sendmsg
- tcap_tp_udp_unitdata_req
- tcap_scXp_n_unitdata_req
- osmo-el2tpd:l2tp_socket_enqueue
Related issues
Actions