Fix P5 messages memory leaks ( CONFIG.request / CONFIG.response )
This commit is contained in:
@@ -351,12 +351,10 @@ int aerial_nr_send_config_request(nfapi_vnf_config_t *config, int p5_idx)
|
|||||||
|
|
||||||
nfapi_p4_p5_message_header_t *msg = &req->header;
|
nfapi_p4_p5_message_header_t *msg = &req->header;
|
||||||
uint16_t msg_len = sizeof(nfapi_nr_config_request_scf_t);
|
uint16_t msg_len = sizeof(nfapi_nr_config_request_scf_t);
|
||||||
nfapi_p4_p5_message_header_t *msgFAPI = calloc(1, msg_len);
|
|
||||||
memcpy(msgFAPI, &req->header, msg_len);
|
|
||||||
uint8_t tx_messagebufferFAPI[sizeof(_this->tx_message_buffer)];
|
uint8_t tx_messagebufferFAPI[sizeof(_this->tx_message_buffer)];
|
||||||
int packedMessageLengthFAPI = -1;
|
int packedMessageLengthFAPI = -1;
|
||||||
packedMessageLengthFAPI =
|
packedMessageLengthFAPI =
|
||||||
fapi_nr_p5_message_pack(msgFAPI, msg_len, tx_messagebufferFAPI, sizeof(tx_messagebufferFAPI), &_this->_public.codec_config);
|
fapi_nr_p5_message_pack(msg, msg_len, tx_messagebufferFAPI, sizeof(tx_messagebufferFAPI), &_this->_public.codec_config);
|
||||||
|
|
||||||
aerial_send_P5_msg(tx_messagebufferFAPI, packedMessageLengthFAPI, msg);
|
aerial_send_P5_msg(tx_messagebufferFAPI, packedMessageLengthFAPI, msg);
|
||||||
|
|
||||||
|
|||||||
@@ -1529,13 +1529,11 @@ int vnf_nr_pack_and_send_p5_message(vnf_t* vnf, uint16_t p5_idx, nfapi_p4_p5_mes
|
|||||||
#ifdef ENABLE_AERIAL
|
#ifdef ENABLE_AERIAL
|
||||||
// In case it is a FAPI message, create 2 messages, one with nFAPI header for OAI PNF and one with no nFAPI header for Aerial
|
// In case it is a FAPI message, create 2 messages, one with nFAPI header for OAI PNF and one with no nFAPI header for Aerial
|
||||||
// L1
|
// L1
|
||||||
nfapi_p4_p5_message_header_t *msgFAPI = calloc(1, msg_len);
|
|
||||||
memcpy(msgFAPI, msg, msg_len);
|
|
||||||
// create FAPI tx_buffer
|
// create FAPI tx_buffer
|
||||||
uint8_t tx_messagebufferFAPI[sizeof(vnf->tx_message_buffer)];
|
uint8_t tx_messagebufferFAPI[sizeof(vnf->tx_message_buffer)];
|
||||||
int packedMessageLengthFAPI = -1;
|
int packedMessageLengthFAPI = -1;
|
||||||
packedMessageLengthFAPI =
|
packedMessageLengthFAPI =
|
||||||
fapi_nr_p5_message_pack(msgFAPI, msg_len, tx_messagebufferFAPI, sizeof(tx_messagebufferFAPI), &vnf->_public.codec_config);
|
fapi_nr_p5_message_pack(msg, msg_len, tx_messagebufferFAPI, sizeof(tx_messagebufferFAPI), &vnf->_public.codec_config);
|
||||||
return aerial_send_P5_msg(tx_messagebufferFAPI, packedMessageLengthFAPI, msg);
|
return aerial_send_P5_msg(tx_messagebufferFAPI, packedMessageLengthFAPI, msg);
|
||||||
#else
|
#else
|
||||||
return 0;
|
return 0;
|
||||||
|
|||||||
Reference in New Issue
Block a user