https://gitlab.synchro.net/main/sbbs/-/commit/0a1265a00a2f193d1d977577
Modified Files:
src/sbbs3/userdat.c
Log Message:
strListFastDelete() now takes a count argument
--- SBBSecho 3.20-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/c0c3c3ae007cc6f90469d3e0
Modified Files:
src/sbbs3/userdat.c
Log Message:
Check return value of seekuserdat() in putuserdat() and return failure

... trying to get to bottom of user.tab corruption reported in issue #797.

Also, don't need to seek in lockuserdat() since we always call seekuserdat() first.
--- SBBSecho 3.21-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/5adf169677b1dee1819f18ca
Modified Files:
src/sbbs3/userdat.c
Log Message:
putuserdat() no longer calls dirtyuserdat() if the write/update failed
--- SBBSecho 3.21-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/690bb56e025804cf73cfebbf
Modified Files:
src/sbbs3/userdat.c
Log Message:
A file descriptor of 0 is technically valid (though normally, STDIN)

Likely fix for CID 514647
--- SBBSecho 3.21-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/6b5a54d3bf6def6cf0694cd5
Modified Files:
src/sbbs3/userdat.c
Log Message:
Add description comment for find_login_id()
--- SBBSecho 3.22-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/df32524348f6987de16ee0d3
Modified Files:
src/sbbs3/userdat.c
Log Message:
Add random jitter to lockuserdat() and putnodedat() retry delays

so that if multiple entities call these functions at the same time, they
don't retry at exactly the same times.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/ccef9d28aad4782f7879354f
Modified Files:
src/sbbs3/userdat.c
Log Message:
Check return value of writeuserfields() in putuserstr()

Return correct error value upon seek failure.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/2cb39d9a8dc73a02a03df214
Modified Files:
src/sbbs3/userdat.c
Log Message:
Insure ctrl chars/codes are stripped from nodestatus() output

Use strlcpy instead of strncpy
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/4610fc7b454e891fb09f2663
Modified Files:
src/sbbs3/userdat.c
Log Message:
Fix incorrect use of sizeof in node_vstatus and node_activity

This would cause the "Logout" status and multinode chat activity
strings to be truncated to 3 or 7 chars.

CID 515713 and 515715
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/1743d8afe6f8bd73bed45a9b
Modified Files:
src/sbbs3/userdat.c
Log Message:
Remove the config check from openuserdat()

... since all (almost all) callerd do a config check first.

Also, some callers of openuserdat() were expecting -1 on failure (always).

Functions that returned the return value of openuserdat() upon failure, now return USER_OPEN_ERROR instead, to be consistent.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/d0e0e8c902f604f70108cf21
Modified Files:
src/sbbs3/userdat.c
Log Message:
Go back to using non-blocking periodic/polling user.tab lock attempts

This partially reverts commit 03b84df85.

I observed deadlocks on Linux attempting locks of user.tab on Samba share, which also deadlocked my Windows nodes. Interestingly, the Windows nodes
never deadlocked on their own (after a week of testing) when using blocking locks.

Double the frequency of lock retries - this has helped reduce the observed user.tab lock failures on Vertrauen.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)