Automated Code Analysis

cppcheck analysis of lcdproc_0.5.4-4.dsc

• ./clients/lcdexec/menu.c:325 [error] - Common realloc mistake: "tmp" nulled but not freed upon failure
• ./clients/lcdproc/machine_FreeBSD.c:128 [error] - Resource leak: apmd
• ./clients/lcdproc/machine_NetBSD.c:115 [error] - Resource leak: apmd
• ./clients/lcdproc/machine_OpenBSD.c:107 [error] - Resource leak: apmd
• ./clients/lcdvc/vc_link.c:77 [error] - Common realloc mistake: "vc_buf" nulled but not freed upon failure
• ./contrib/interface-demo2/if_demo.c:172 [error] - Undefined behaviour: buffer 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.
• ./server/client.c:153 [error] - Possible null pointer dereference: c - otherwise it is redundant to check if c is null at line 155
• ./server/drivers/pylcd.c:368 [error] - Memory leak: p
• ./server/drivers/text.c:195 [error] - fflush() called on input stream "stdin" may result in undefined behaviour
• ./server/menuitem.c:1231 [error] - Common realloc mistake: "chars" nulled but not freed upon failure
• ./server/render.c:475 [error] - Dangerous usage of 'str' (strncpy doesn't always 0-terminate it)
• ./server/widget.c:152 [error] - Possible null pointer dereference: w - otherwise it is redundant to check if w is null at line 154

---