• src/sbbs3/bat_xfer.cpp

    From Rob Swindell@1:103/705 to Git commit to main/sbbs/master on Sun Jun 6 00:04:44 2021
    https://gitlab.synchro.net/main/sbbs/-/commit/b417647b95c4fa921624034c
    Modified Files:
    src/sbbs3/bat_xfer.cpp
    Log Message:
    Check batch-uploaded filenames for illegal characters/sequences/namesThe BadFilename text.dat is now a format string (prints the bad filename).Use the nice progress() display status while searching for duplicate filenames.
    --- SBBSecho 3.14-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to Git commit to main/sbbs/master on Thu Jun 10 13:27:44 2021
    https://gitlab.synchro.net/main/sbbs/-/commit/0c5c140eceeb8012a8083f6a
    Modified Files:
    src/sbbs3/bat_xfer.cpp
    Log Message:
    Call smb_freefilemem() instead of smb_msgmem() for filesNot that it really matters, but perhaps it'll matter in the future. And the code reads better.
    --- SBBSecho 3.14-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to Git commit to main/sbbs/master on Sat Oct 23 10:06:56 2021
    https://gitlab.synchro.net/main/sbbs/-/commit/7dc1bc798e5a06e2278711ea
    Modified Files:
    src/sbbs3/bat_xfer.cpp
    Log Message:
    Fix segfault after batch-upload when no "uploads" dir specifiedWhen there's no "uploads" directory configured by the sysop, cfg.upload_dir will be set to INVALID_DIR, which cannot be used as an index into cfg.dir[] to determine if the time used for uploading the files should be "given back" to the user.In v3.18, we used the first file in the upload queue, if there was one, else fell back to the "uploads" dir (which had to have been defined if there were no files in the queue). So this illegal array indexing was a regression in v3.19.Reported by Zoltán Gábor on Facebook
    --- SBBSecho 3.14-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)