1. Forum
  2. DOVE-Net
  3. Synchronet Programming

  • src/sbbs3/un_qwk.cpp un_rep.cpp

    From Rob Swindell (on Debian Linux)@VERT to Git commit to main/sbbs/master on Wed May 6 19:41:53 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/de5f05837b15d144b2c14b17
    Modified Files:
    src/sbbs3/un_qwk.cpp un_rep.cpp
    Log Message:
    un_qwk/un_rep: suppress listFree FORWARD_NULL false-positive (CIDs 631130, 631142)

    user_list is zero-initialized as link_list_t{0}. listFree's only
    list->sem dereference is gated on (list->flags & LINK_LIST_SEMAPHORE),
    which is never set here, so the call is safe. Coverity loses track of
    the flag check; document the invariant and suppress the FORWARD_NULL
    warning at both call sites.

    Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Debian Linux)@VERT to Git commit to main/sbbs/master on Mon Oct 26 17:38:19 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/b058694b9062f75ca0071d62
    Modified Files:
    src/sbbs3/un_qwk.cpp un_rep.cpp
    Log Message:
    unpack_qwk, unpack_rep: create qnet/<ID>.in with mkpath()

    mkpath() returns success when the directory is already there and also
    creates a missing data/qnet/, where MKDIR() would fail and the file move
    after it would fail too. unpack_rep() discarded MKDIR()'s result (because it failed for every file after the first); it now logs a failure like
    unpack_qwk() does.

    Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

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