https://gitlab.synchro.net/main/sbbs/-/commit/984f1eb68d22b45de08fd723
Modified Files:
src/xpdev/genwrap.c
Log Message:
Insure all builds of safe_strerror() write the string to the passed buf

The GNU_SOURCE build of this function was (sometimes?) just returning the string and not actually copying it to the passed buf. This is consistent
with the GNU manpage on strerror_r():
"This may be either a pointer to a string that the function stores in buf ..." but was inconsistent with all the other build types of this function.

Also updated to use strlcpy and write the problematic error number to the default string (if unknown).

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

https://gitlab.synchro.net/main/sbbs/-/commit/0d60813bb295908d35ad556e
Modified Files:
src/xpdev/genwrap.c
Log Message:
Fix issue in last commit

Need to update the parameter names as well.

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

https://gitlab.synchro.net/main/sbbs/-/commit/78ae0fd80623b830fe86b6d6
Modified Files:
src/xpdev/genwrap.c
Log Message:
Change spaces to tabs in new strlcpy()

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

src/xpdev genwrap.c 1.116 1.117
Update of /cvsroot/sbbs/src/xpdev
In directory cvs:/home/rswindell/sbbs/src/xpdev

Modified Files:
genwrap.c
Log Message:
Fix safe_strerror() for GNU builds (must use the return value of strerror_r()).



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

src/xpdev genwrap.c 1.117 1.118
Update of /cvsroot/sbbs/src/xpdev
In directory cvs:/tmp/cvs-serv32696

Modified Files:
genwrap.c
Log Message:
Provide a (non-ideal) implementation of safe_strerror() for Borland C builds that don't have strerror_s:
Error: Unresolved external '_strerror_s' referenced from genwrap



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

https://gitlab.synchro.net/sbbs/sbbs/-/commit/678627dbcfc2b6eab9a7a7cb
Modified Files:
src/xpdev/genwrap.c
Log Message:
Use the crappy strerror() wrapper for all _WIN32 that's not MSC,
not just for Borland.

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

https://gitlab.synchro.net/main/sbbs/-/commit/746ccfd57d43877a95aaded3
Modified Files:
src/xpdev/genwrap.c
Log Message:
Attempt to fix musl builds

The non-standard strerror_r() is glibc specific, musl doesn't do
that. It *appears* that __USE_GNU implies glibc.

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

https://gitlab.synchro.net/main/sbbs/-/commit/8084997ba19ed4b9c1773003
Modified Files:
src/xpdev/genwrap.c
Log Message:
Use RtlGetVersion to get/report Windows 11 (and build number) correctly

This is fun Microsoft. If Windows 11 is actually Windows 10.22000+, what will Windows 12 be? No one can guess.

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

https://gitlab.synchro.net/main/sbbs/-/commit/d58cc13fde38ff3a965dc838
Modified Files:
src/xpdev/genwrap.c
Log Message:
Move the #pragma warning to just before the call to GetVersionEx()

Needed for building sbbsexec.dll

I guess #pramga warning only affects the following source line.

Apparently we're disabling this warning effectively via other means in all other MSVC projects.

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

https://gitlab.synchro.net/main/sbbs/-/commit/0c385ef21ba3488214dfa917
Modified Files:
src/xpdev/genwrap.c
Log Message:
Only use RtlGetVersion for Windows 10/11

This function appears to truncate the service pack info for Windows 7 (6.1): "Windows NT Version 6.1 (Build 7601) Service Pack 1 x86" became:
"Windows NT Version 6.1 (Build 7601) S x86"

Don't close the handle to ntdll.dll (hey, that's stupid filename, Microsoft!) since the module could be unloaded from the address space and then a call to the captured procedure address could/would crash. This handle will be closed when the process terminates anyway.

While we're here, correct the Windows 6.1 -> 7.0 numbering. That looks better: "Windows NT Version 7.0 (Build 7601) Service Pack 1 x86"

Something should probably be done for Windows 6.2 -> 8.0 numbering too, but I don't have a VM handy. Is anyone actually still running Windows 8.x?

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

https://gitlab.synchro.net/main/sbbs/-/commit/3046bd7ae4c3a503d9c5762d
Modified Files:
src/xpdev/genwrap.c
Log Message:
Simplify the Windows version detection a bit, don't use NetWkstaGetInfo()

I started seeing Windows 10.0 reported (without a build number) again on Windows 11 (after a regular update), so changed the logic to always try/use RtlGetVersion() except for Windows 7 where I saw it truncated the Service
Pack details.

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

https://gitlab.synchro.net/main/sbbs/-/commit/0158699aff72ded56de68882
Modified Files:
src/xpdev/genwrap.c
Log Message:
Don't return -1 from safe_snprintf() when size is zero.

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

https://gitlab.synchro.net/main/sbbs/-/commit/f5a033dd24945fdd2406c66f
Modified Files:
src/xpdev/genwrap.c
Log Message:
Annotate risky call to random()

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

https://gitlab.synchro.net/main/sbbs/-/commit/4439557bfaef6655d2979e78
Modified Files:
src/xpdev/genwrap.c
Log Message:
Seconds not milliseconds.

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

https://gitlab.synchro.net/main/sbbs/-/commit/97434544be0ea793c297bd42
Modified Files:
src/xpdev/genwrap.c
Log Message:
Actually, CLOCK_SECOND is better than CLOCK_MONOTONIC_FAST

It's basically CLOCK_REALTIME with tv_nsec set to zero.

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

https://gitlab.synchro.net/main/sbbs/-/commit/cd8e57212957a107d820d7f2
Modified Files:
src/xpdev/genwrap.c
Log Message:
If we're older than Vista, use GetTickCount()

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

https://gitlab.synchro.net/main/sbbs/-/commit/68dffbe20c3e00ea9c37b61c
Modified Files:
src/xpdev/genwrap.c
Log Message:
Check _WIN32_WINNT, not WINVER

Borland hates WINVER

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

https://gitlab.synchro.net/main/sbbs/-/commit/b02975d7b18d659f1ad0fef3
Modified Files:
src/xpdev/genwrap.c
Log Message:
Hard-code the _WIN32_WINNT_VISTA value.

Maybe Borland is also using pre-vista headers?

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

https://gitlab.synchro.net/main/sbbs/-/commit/8f919a6c118cd86c5b672030
Modified Files:
src/xpdev/genwrap.c
Log Message:
More fixups for clock selection.

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