• src/sbbs3/baja.c dupefind.c execmisc.cpp fixsmb.c jsexec.c qwk.cpp qwk

    From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Saturday, December 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.

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