Automated Code Analysis

cppcheck analysis of nap_1.5.4-5.dsc

• ./src/alias.c:886 [error] - Common realloc mistake: "ret" nulled but not freed upon failure
• ./src/alias.c:482 [error] - Undefined behaviour: st 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.
• ./src/cmds.c:1368 [error] - Common realloc mistake: "servers" nulled but not freed upon failure
• ./src/cmds.c:1488 [error] - Memory leak: urlhost
• ./src/event.c:1391 [error] - Resource leak: f
• ./src/handlers.c:216 [error] - Common realloc mistake: "ret" nulled but not freed upon failure
• ./src/library.c:653 [error] - Common realloc mistake: "t" nulled but not freed upon failure
• ./src/memwatch.c:147 [error] - Memory leak: m
• ./src/nap.c:172 [error] - Common realloc mistake: "buf" nulled but not freed upon failure
• ./src/nap.c:260 [error] - Common realloc mistake: "buf" nulled but not freed upon failure
• ./src/nap.c:1419 [error] - Memory leak: aliasfile
• ./src/nap.c:1419 [error] - Memory leak: handlerfile
• ./src/winio.c:957 [error] - Possible null pointer dereference: cmdlend - otherwise it is redundant to check if cmdlend is null at line 962
• ./src/winio.c:447 [error] - Resource leak: f

---