1. Forum
  2. FidoNet
  3. SYNC PROGRAMMING

  • src/sbbs3/postmsg.cpp

    From Rob Swindell (on Windows)@1:103/705 to Git commit to main/sbbs/master on Sun May 7 23:22:44 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/4a86ab8768e486a58d22d55e
    Modified Files:
    src/sbbs3/postmsg.cpp
    Log Message:
    Bounds-check subnum in savemsg()It's possible to use savemsg() (from JS MsgBase.save_msg()) to add a messageto a message base that's not a sub-board (not configured in SCFG->MessageAreas and not the "mail" base), but in that case, savemsg() would dereferencean invalid sub in the scfg->sub array and crash. So use is_valid_subum()to insure the subnum is a valid sub before using as an index.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Windows 11)@1:103/705 to Git commit to main/sbbs/master on Sun Jan 5 21:06:00 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/1dde501fe8ea8dc8bea5e3d4
    Modified Files:
    src/sbbs3/postmsg.cpp
    Log Message:
    Call add_msg_id() *after* opening the message base to send notification email

    notify() was creating wrongly-formated auto-generated Message-IDs:
    e.g. Message-ID: <677B3F17.0.notices@vert.synchro.net>
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Windows 11)@1:103/705 to Git commit to main/sbbs/master on Tue May 20 20:29:14 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/7ea01b60689e299a8a9dd131
    Modified Files:
    src/sbbs3/postmsg.cpp
    Log Message:
    Refactor/order the handling of reply-to-message header data

    Fix CID 551210

    There should not be any change in behavior here other than eliminating a potential null pointer dereference.
    --- SBBSecho 3.27-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)