• src/xpdev/ini_file.c

    From rswindell@VERT to CVS commit on Monday, April 30, 2018 21:58:00
    src/xpdev ini_file.c 1.162 1.163
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv29284

    Modified Files:
    ini_file.c
    Log Message:
    iniWriteFile(): don't truncate the file to 0-bytes before writing the contents, instead truncate to the new length after writing the contents. This should
    make .ini files more tolerate of "out of disk space" situations and reduce
    the risk of lost data (0-byte .ini files) due to low/no disk space.


    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Tuesday, August 28, 2018 21:01:00
    src/xpdev ini_file.c 1.164 1.165
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv11353

    Modified Files:
    ini_file.c
    Log Message:
    If any non-NULL style arguments were passed with non-NULL elements, those elements (member variables) would be changed to point to a string constant. Don't do that - instead just use local stack variables for those style
    element default values. This leads the way to constifying the style arguments in the future and allows the caller to use non-NULL style elements as indicator of something that should be freed if the caller allocated some memory for it.



    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Tuesday, August 28, 2018 22:12:00
    src/xpdev ini_file.c 1.165 1.166
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv32379

    Modified Files:
    ini_file.c
    Log Message:
    Fix mistaken reverse logic in previous commit: would write "(null)" instead
    of default style elements.



    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From deuce@VERT to CVS commit on Wednesday, July 10, 2019 19:41:47
    src/xpdev ini_file.c 1.166 1.167
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv4986

    Modified Files:
    ini_file.c
    Log Message:
    Fix obvious typo in iniSetBitField() which could result in a NULL
    dereference. (found by Coverity)




    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From deuce@VERT to CVS commit on Tuesday, July 16, 2019 18:26:00
    src/xpdev ini_file.c 1.167 1.168
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv11153

    Modified Files:
    ini_file.c
    Log Message:
    If ftell() fails, it returns -1. We don't want to truncate there.

    Found by Coverity.




    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From deuce@VERT to CVS commit on Tuesday, July 16, 2019 20:35:53
    src/xpdev ini_file.c 1.168 1.169
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv29520

    Modified Files:
    ini_file.c
    Log Message:
    Coverity issues. Not serious (strcat() -> SAFECAT())




    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From deuce@VERT to CVS commit on Tuesday, July 16, 2019 20:38:54
    src/xpdev ini_file.c 1.169 1.170
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv30014

    Modified Files:
    ini_file.c
    Log Message:
    More SAFECATS()
    - Coverity.




    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Sunday, January 24, 2021 21:52:29
    https://gitlab.synchro.net/main/sbbs/-/commit/4eeaaf726c2d356e29201bb7
    Modified Files:
    src/xpdev/ini_file.c
    Log Message:
    Support !include wildcard-pattern

    This allows auto-inclusion of sub-directories of config files (e.g. !include modopts.d/*.ini). This, along with the next commit to js_file.c, allows module authors (or sysops) to keep their module-specific settings in a module-specific file (rather than always merging-with/managing the monolithic/shared modopts.ini). The [modname] section heading is still required in the included .ini file.

    Replaced use of non-thread-safe STRERROR() with safe_strerror().

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Sunday, January 24, 2021 22:24:48
    https://gitlab.synchro.net/main/sbbs/-/commit/30d4aa79220f8f7481bad092
    Modified Files:
    src/xpdev/ini_file.c
    Log Message:
    Fix Borland C++ compile failure.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Thursday, January 28, 2021 07:17:50
    https://gitlab.synchro.net/main/sbbs/-/commit/902e7a51e969f3c9361c4953
    Modified Files:
    src/xpdev/ini_file.c
    Log Message:
    Fix the root root cause of TypeError: obj is not an object

    iniGetNamedStringList() was returning NULL when attempting to get the list of keys in the "root" (unnamed) section and the file is actually empty/blank. If there were *any* characters in the file, it would return an empty list (not NULL). Now, always return the empty list, even when the file is blank.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Tuesday, May 04, 2021 08:35:40
    https://gitlab.synchro.net/main/sbbs/-/commit/fe022fecc758985541b9c457
    Modified Files:
    src/xpdev/ini_file.c
    Log Message:
    Remove unused variable added in previous commit.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Wednesday, May 05, 2021 05:17:59
    https://gitlab.synchro.net/main/sbbs/-/commit/ec0e0ee47efb1427ef894920
    Modified Files:
    src/xpdev/ini_file.c
    Log Message:
    Fix a couple of resource leaks in iniSortSections()

    One in an error path (CID 331353), the other not.

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