• src/sbbs3/un_qwk.cpp un_rep.cpp

    From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Wednesday, September 11, 2024 20:26:18
    https://gitlab.synchro.net/main/sbbs/-/commit/29a35642160a7f4148134ce3
    Modified Files:
    src/sbbs3/un_qwk.cpp un_rep.cpp
    Log Message:
    Blocked IP addresses in ip-silent.can weren't filtered from QWK and REP pkts

    The "from_ip" header field of QWK messages is checked against blocked IP addresses, but was only checking against IP addresses from ip.can, not any addresses listed in ip-silent.can. These 2 list files are now merged together for the purposes of filtering during QWK/REP packet import.

    ---
    þ 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 Thursday, September 12, 2024 21:37:53
    https://gitlab.synchro.net/main/sbbs/-/commit/aff554a44aa48a1e1565f450
    Modified Files:
    src/sbbs3/un_qwk.cpp un_rep.cpp
    Log Message:
    Fix crash due to use after free or double-free

    ... bug introduced in commit 29a35642160a7f4.

    strListMerge() doesn't realloc the strings in the list, so we don't want to free the strings in this list here.

    This is likely the cause of the crash Keyop eluded to in #synchronet.

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