Activity log for bug #1838001

Date Who What changed Old value New value Message
2019-07-26 09:09:03 xiaodonghu bug added bug
2019-07-26 09:21:14 xiaodonghu description In the 200th line of the fault\snmp-audittrail\sources\snmpAuditPlugin.c file, the pointer parameter value of the free function may be NULL(the pointer value has been checked in the previous snmp_log function), so this pointer needs to be checked before the free function is called. The function sprint_realloc_objid may return NULL in abnormal case. So, the function fmtoid and fmtmsg_var may return NULL. However, in these function log_results log_pdu_header and log_invalid_oid_trees, the tempory value var_str point to the return value of fmtmsg_var may be NULL. In these function, snmp_log have checked whether var_str is NULL or not, but free doesn't check it. When the var_str is NULL, free(NULL) will make the program crashed. So we need to add the checking before free function.
2019-07-26 18:41:26 Ghada Khalil tags stx.fault
2019-07-26 18:41:34 Ghada Khalil starlingx: importance Undecided Low
2019-07-26 18:44:37 Ghada Khalil starlingx: assignee xiaodonghu (xiaodonghu)
2019-07-26 18:44:41 Ghada Khalil starlingx: status New Triaged
2020-05-06 07:02:04 Ambarish Das starlingx: assignee xiaodonghu (xiaodonghu) Ambarish Das (hiambar)
2020-05-06 07:09:06 Ambarish Das starlingx: status Triaged Invalid