Subj : src/sbbs3/baja.c dupefind.c execmisc.cpp fixsmb.c jsexec.c qwk.cpp qwk
To   : Git commit to main/sbbs/master
From : Rob Swindell (on Windows 11)
Date : Sat Dec 23 2023 13:02:29

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.
--- SBBSecho 3.20-Linux
 * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

.