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

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

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

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

https://gitlab.synchro.net/main/sbbs/-/commit/d2f5d55133310a6c03b91057
Modified Files:
src/xpdev/ini_file.c
Log Message:
No strtoull() for Borland

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

https://gitlab.synchro.net/main/sbbs/-/commit/cea188d03b824bafaf7b4622
Modified Files:
src/xpdev/ini_file.c
Log Message:
Fix reversed logic of previous commit.

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

https://gitlab.synchro.net/main/sbbs/-/commit/4fd02c4efa228d777db94785
Modified Files:
src/xpdev/ini_file.c
Log Message:
Partial fix of iniGetSString() and iniReadSString()

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

https://gitlab.synchro.net/main/sbbs/-/commit/b29d43752c47abff892b5083
Modified Files:
src/xpdev/ini_file.c
Log Message:
Finish fixing ini*SString().

The first fix was to copy the right value, the second fix was to
ensure zero-length strings are terminated.

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

https://gitlab.synchro.net/main/sbbs/-/commit/ddcb536755e002a0985a8f48
Modified Files:
src/xpdev/ini_file.c
Log Message:
Fix (incorrect) warning from gcc12

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

https://gitlab.synchro.net/main/sbbs/-/commit/fbd2315bd2647948b18e26ae
Modified Files:
src/xpdev/ini_file.c
Log Message:
And now Coverity gets into the false warning game.

Fix incorrect Coverity issue that speculates this may return fval.

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

https://gitlab.synchro.net/main/sbbs/-/commit/fd41db08baa65e68618163f9
Modified Files:
src/xpdev/ini_file.c
Log Message:
Cast away warning...

This is why I returned ret. :D

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

https://gitlab.synchro.net/main/sbbs/-/commit/2995f851439e440f60587b87
Modified Files:
src/xpdev/ini_file.c
Log Message:
Bugfix and more paranoia in ini*SString() functions.

Bugfis: Be sure to terminate value if returning deflt.
Paranoia: Terminate value if iniGetString() returns NULL

I think the only way to trigger the paranoia is to pass NULL as the
default when reading a key that's not present.

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

https://gitlab.synchro.net/main/sbbs/-/commit/92088f9983ef38fc376208c2
Modified Files:
src/xpdev/ini_file.c
Log Message:
For name ints/enums, allow the value to be longer than the name

This allows me to shorten "Informational" to "Info" and "Debugging" to "Debug" and be backward compatible with existing .ini files. I think "Info" and
"Debug" are universally known and take up less space (e.g. in SCFG).

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