• src/sbbs3/xtrn_sec.cpp

    From Rob Swindell@VERT to Git commit to sbbs/master on Friday, October 02, 2020 18:38:01
    https://gitlab.synchro.net/sbbs/sbbs/-/commit/3209fee4ccbc81050824e696
    Modified Files:
    src/sbbs3/xtrn_sec.cpp
    Log Message:
    (more) Safe string handling

    strcpy -> SAFECOPY
    sprintf -> safe_snprintf or SAFEPRINTF2
    strcat -> SAFECAT

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on ChromeOS)@VERT to Git commit to main/sbbs/master on Sunday, March 23, 2025 18:11:53
    https://gitlab.synchro.net/main/sbbs/-/commit/7550784ff7d067f3b418b4e9
    Modified Files:
    src/sbbs3/xtrn_sec.cpp
    Log Message:
    Don't close logfile_fp before running JavaScript externals

    This would've prevented some JS log functions from writing to the node.log file (which gets concatented onto the daily "system" log files).

    This just came up via code review while looking into Keyop's reported error: term Node 2 !ERROR 9 (Bad file descriptor) in main.cpp line 4606 (node_thread)
    truncating "logfile" access=0

    This is likely unrelated since the "logfile" is immediately re-opened after executing the external program/script.

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