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

  • src/sbbs3/sauce.c

    From Rob Swindell@VERT to Git commit to main/sbbs/master on Sat Apr 24 20:23:32 2021
    https://gitlab.synchro.net/main/sbbs/-/commit/291639149ee0964d7052d1e0
    Modified Files:
    src/sbbs3/sauce.c
    Log Message:
    Maybe address Coverity issue.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Fri Mar 22 11:38:39 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/c35dec2fbced851ce25c1627
    Modified Files:
    src/sbbs3/sauce.c
    Log Message:
    Remove gratuitous use of comma operator.

    ---
    ï¿­ 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 Sat Mar 23 16:12:16 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/c8634c7adc3633923d5db58f
    Modified Files:
    src/sbbs3/sauce.c
    Log Message:
    SAFECOPY (now using strlcpy) requires the source is terminated

    ... so use strncpy() and TERMINATE() here (instead of SAFECOPY) to resolve newly reported Coverity issues.

    There may be other places where we're using SAFECOPY() with an unterminated source string that would've been fine before we switched SAFECOPY from using strncpy to strlcpy. So we should reconsider that change.

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