https://gitlab.synchro.net/main/sbbs/-/commit/6080737aa131d0b919e46d3a
Modified Files:
src/sbbs3/baja.c dupefind.c execmisc.cpp fixsmb.c jsexec.c qwk.cpp qwknodes.c rechocfg.c sbbsecho.c src/sbbs3/scfg/scfg.c scfgchat.c scfgnet.c scfgxfr1.c scfgxfr2.c scfgxtrn.c src/sbbs3/smbutil.c targets.mk src/xpdev/genwrap.c genwrap.h
Log Message:
Add/use realloc_or_free() instead of realloc() in some places
Where ever we assign the realloc() result to the same pointer we pass, use this new function instead to eliminate the cppcheck error reported by Nelgin:
Common realloc mistake: 'p' nulled but not freed upon failure [memleakOnRealloc]

This isn't going to actually solve any memory leaks, it's just good practice for critical error (e.g. no memory error) handling.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net