Commit fa6aa1a7 authored by Tien Thinh NGUYEN's avatar Tien Thinh NGUYEN

Fix CI warnings

parent 42f17859
......@@ -157,7 +157,10 @@ std::string conv::toString(const struct in6_addr& in6addr) {
void conv::convert_string_2_hex(
std::string& input_str, std::string& output_str) {
unsigned char* data = (unsigned char*) malloc(input_str.length() + 1);
if (!data) return;
if (!data) {
free_wrapper((void**) &data);
return;
}
memset(data, 0, input_str.length() + 1);
memcpy((void*) data, (void*) input_str.c_str(), input_str.length());
......@@ -167,7 +170,11 @@ void conv::convert_string_2_hex(
printf("\n");
char* datahex = (char*) malloc(input_str.length() * 2 + 1);
if (!datahex) return;
if (!datahex) {
free_wrapper((void**) &datahex);
free_wrapper((void**) &data);
return;
}
memset(datahex, 0, input_str.length() * 2 + 1);
for (int i = 0; i < input_str.length(); i++)
......@@ -185,7 +192,10 @@ unsigned char* conv::format_string_as_hex(std::string str) {
if (!datavalue) return nullptr;
unsigned char* data = (unsigned char*) malloc(str_len + 1);
if (!data) return nullptr;
if (!data) {
free_wrapper((void**) &data);
return nullptr;
}
memset(data, 0, str_len + 1);
memcpy((void*) data, (void*) str.c_str(), str_len);
......@@ -238,13 +248,19 @@ void conv::msg_str_2_msg_hex(std::string msg, bstring& b) {
printf("tmp string: %s\n", msg_hex_str.c_str());
unsigned int msg_len = msg_hex_str.length();
char* data = (char*) malloc(msg_len + 1);
if (!data) return;
if (!data) {
free_wrapper((void**) &data);
return;
}
memset(data, 0, msg_len + 1);
memcpy((void*) data, (void*) msg_hex_str.c_str(), msg_len);
printf("data: %s\n", data);
uint8_t* msg_hex = (uint8_t*) malloc(msg_len / 2 + 1);
if (!msg_hex) return;
if (!msg_hex) {
free_wrapper((void**) &msg_hex);
return;
}
conv::ascii_to_hex(msg_hex, (const char*) data);
b = blk2bstr(msg_hex, (msg_len / 2));
......
......@@ -49,7 +49,8 @@ class PDUSessionResourceFailedToSetupItemCxtFail {
private:
PDUSessionID pdu_session_id_; // Mandatory
OCTET_STRING_t pdu_session_resource_setup_unsuccessful_transfer_; // Mandatory
OCTET_STRING_t
pdu_session_resource_setup_unsuccessful_transfer_; // Mandatory
};
} // namespace ngap
......
......@@ -128,8 +128,10 @@ void HandoverCommandMsg::setNASSecurityParametersFromNGRAN(
Ngap_HandoverCommandIEs__value_PR_NASSecurityParametersFromNGRAN;
if (!conv::octet_string_copy(
ie->value.choice.NASSecurityParametersFromNGRAN,
nASSecurityParametersFromNGRAN.value()))
nASSecurityParametersFromNGRAN.value())) {
free_wrapper((void**) &ie);
return;
}
int ret = ASN_SEQUENCE_ADD(&handoverCommandIEs->protocolIEs.list, ie);
if (ret != 0)
......
......@@ -34,6 +34,7 @@ namespace ngap {
//------------------------------------------------------------------------------
HandoverPreparationFailure::HandoverPreparationFailure() : NgapUEMessage() {
hoPreparationFailureIEs = nullptr;
criticalityDiagnostics = nullptr;
setMessageType(NgapMessageType::HANDOVER_PREPARATION_FAILURE);
initialize();
......
......@@ -23,6 +23,7 @@ AmfEventReport::AmfEventReport() {
m_SubscriptionId = "";
m_SubscriptionIdIsSet = false;
m_AnyUe = false;
m_AmfUeNgapId = 0;
m_AnyUeIsSet = false;
m_Supi = "";
m_SupiIsSet = false;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment