• 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 Sat Dec 23 13:02:29 2023
    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