Automated Code Analysis

cppcheck analysis of timidity_2.13.2-39.dsc

• ./interface/gtk_i.c:693 [error] - Memory leak: t_back
• ./interface/tk_c.c:1067 [error] - Buffer access out-of-bounds: Panel.v_flags
• ./interface/tk_c.c:1066 [error] - Buffer access out-of-bounds: Panel.cvel
• ./interface/tk_c.c:1065 [error] - Buffer access out-of-bounds: Panel.ctotal
• ./interface/tk_c.c:1068 [error] - Buffer access out-of-bounds: Panel.c_flags
• ./interface/w32g_syn.c:401 [error] - Buffer access out-of-bounds: w32g_syn_id_port
• ./interface/w32g_syn.c:1124 [error] - Buffer access out-of-bounds: w32g_syn_id_port
• ./interface/wrdt_mac.c:1317 [error] - Dangerous usage of 'filename' (strncpy doesn't always 0-terminate it)
• ./libunimod/load_imf.c:649 [error] - Common realloc mistake: "nextwav" nulled but not freed upon failure
• ./libunimod/load_xm.c:591 [error] - Common realloc mistake: "nextwav" nulled but not freed upon failure
• ./libunimod/mloader.c:708 [error] - Undefined behaviour: list is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./timidity/m2m.c:450 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./timidity/mfi.c:520 [error] - Uninitialized variable: part
• ./timidity/mfi.c:528 [error] - Uninitialized variable: part
• ./timidity/mfi.c:533 [error] - Uninitialized variable: part
• ./timidity/mfi.c:539 [error] - Uninitialized variable: part
• ./timidity/mfi.c:561 [error] - Uninitialized variable: part
• ./timidity/readmidi.c:610 [style] - Redundant assignment of "type" in switch

---