Automated Code Analysis

cppcheck analysis of lcdproc_0.5.2-3.dsc

• ./clients/lcdproc/machine_FreeBSD.c:105 [error] - Resource leak: apmd
• ./clients/lcdproc/machine_NetBSD.c:112 [error] - Resource leak: apmd
• ./clients/lcdproc/machine_OpenBSD.c:104 [error] - Resource leak: apmd
• ./clients/lcdvc/vc_link.c:63 [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:165 [error] - Possible null pointer dereference: c - otherwise it is redundant to check if c is null at line 167
• ./server/drivers/bayrad.c:739 [error] - Uninitialized variable: readchar
• ./server/drivers/pylcd.c:347 [error] - Memory leak: p
• ./server/drivers/serialVFD_displays.c:249 [error] - Buffer access out-of-bounds: p.charmap
• ./server/drivers/text.c:180 [error] - fflush() called on input stream "stdin" may result in undefined behaviour
• ./server/menuitem.c:1185 [error] - Common realloc mistake: "chars" nulled but not freed upon failure
• ./server/screen.c:59 [error] - Memory leak: s
• ./server/screen.c:89 [error] - Memory leak: s.id
• ./server/widget.c:123 [error] - Possible null pointer dereference: w - otherwise it is redundant to check if w is null at line 125

---