not the same behaviour as in genwrap.c

compiler.

FreeBSD to fix builds for OSs without alloca() (ie: OpenBSD) that define
__GNUC__

headers are rife with.  Re-enables cross-building with mingw32 (at least for
SyncTERM).

for the static 64-byte sbbs->telnet_cmd buffer, it will bork the session.
(Seen with a long user-name and DISPLAY when using the OS X telnet client).

rather than using a fixed-length buffer.

case where the handle_type becomes invalid somehow (exceptionall unlikely)

stdin in uifcx getstr().

is detected, rather than an uninitialized value!

xp_asprintf_next() as protection against an invalid type being passed in.

fails after mmap() succeeds.

Overhaul of the sample allocation by setting it to NULL when it's copied to
something that will free it, and for Win32, wait for the currently playing
sample (if any) to complete before returning, and free the handle then.

function.

Fix small resource leak in parseEnumList if splitList() returns a zero-length
list (I don't think that's actually possible, but there's code to handle it).

"Attempting to destroy a locked mutex results in undefined behavior."

xp_printf() interpreted the 'l' modifier as 'h' and the 'll' modifier as the
(invalid) 'hl' modifier.

an uninitialized value.

Found by Coverity.

Specifically, if select(),  getsockopt(), or connect() fails, return
ERROR_VALUE instead of SOCKET_ERROR.

Found by Coverity since we never checked the return value of getsockopt()

Thanks Coverity!