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

  • src/sbbs3/writemsg.cpp

    From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Mon Dec 9 19:15:31 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/fbe51f3eec27287c29f058a1
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    Fix off-by-one usage of snprintf() when copying message subject

    - when converting from CP437 to UTF-8
    - when reading from RESULT.ED drop file

    This effectively limited message subjects in some instances to 69 chars
    instead of 70. This bug was caught while debugging a replied-message subject conversion from UTF-8 to CP437 issue reported by Accession.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Tue Dec 24 13:33:38 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/51cb7273a3a6d6c14e64e07f
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    Close file descriptor in error condition

    Fix CID 516461

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Mar 30 10:15:36 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/9c3a122c0a6fa522d91807c4
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    Fix another Coverity defect

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Mon Mar 31 15:28:25 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/7a7caa110f6693d15ee2e611
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    Fix MSVC warning about signed/unsigned comparison

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Tue Apr 22 15:25:50 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/d513362bbc8619e98695e333
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    If editor doesn't create a file, just return false from sbbs_t::editfile()

    ... don't log any error message.

    This fixes issue #914

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Mon Mar 2 23:37:52 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/9abbcf31c3e92cc111f9189c
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    Display changed attributes in change msg attribute function

    This appears to have been broken since commit 556225fa7 (4 years ago, addressing a Coverity complaint <sigh>).

    When changing (toggling) the attributes of a message, the new/changed attributes were supposed to be shown, but were not. Exiting did save those attributes correctly, but the UI was very non-intuitive because of this bug.

    ---
    þ 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 Wed May 6 19:41:53 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/a7106cca0cf813beb39710d0
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    writemsg: bail out on ftell error before reusing offset (CID 640333)

    writemsg() captures the message-start offset with ftell() and reuses
    it via three subsequent fseek(stream, l, SEEK_SET) calls. ftell may
    return -1 on error, in which case the fseek calls would seek to a
    negative offset (UB) and corrupt the quoted-text buffer. Bail out
    cleanly instead, mirroring the existing error-cleanup paths.

    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 Wed May 6 22:36:57 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/fcffe36822eb49d81f25d7f6
    Modified Files:
    src/sbbs3/writemsg.cpp
    Log Message:
    writemsg: handle fseek/fexistcase return values (CIDs 486496, 548248)

    - writemsg(): two fexistcase() calls are case-fix-only (same pattern
    as 76b5c7f43); cast to (void).
    - movemsg(): combine fseek+fread error handling so a seek failure
    takes the same error-recovery path as a read failure.

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

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