TZUTC: -0800
MSGID: 53776.syncprog@1:103/705 2daabdcc
PID: Synchronet 3.21a-Linux master/48922a15c Dec 17 2025 GCC 12.2.0
TID: SBBSecho 3.33-Linux master/48922a15c Dec 17 2025 GCC 12.2.0
BBSID: VERT
CHRS: ASCII 1
FORMAT: flowed
https://gitlab.synchro.net/main/sbbs/-/commit/130d267d095b28c5c829b2d3
Modified Files:
	src/sbbs3/js_socket.cpp
Log Message:
Code clean-up

  - TRUE -> true
  - Eliminate unnecessary free(NULL) checks
  - Fix the potential socket leaks in error handling in socket constructors
    as reported by ChatGPT via Nelgin

Something I noticed while doing this: functions in this file call
call_socket_open_callback() from main.cpp either directly or via a callback
from xpms API calls. This only work if the terminal server is running and has
been initialized. So ... running other servers in their own process without
the terminal server (as some sysops do) won't count open sockets accurately.

--- SBBSecho 3.33-Linux
 * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
SEEN-BY: 10/0 1 102/401 103/705 105/81 106/201 124/5016 128/187 129/14
SEEN-BY: 153/7715 154/110 214/22 218/0 1 215 700 810 226/30 227/114
SEEN-BY: 229/110 134 206 317 400 426 428 470 700 705 266/512 280/464
SEEN-BY: 291/111 301/1 320/219 322/757 342/200 396/45 460/58 633/280
SEEN-BY: 712/848 902/26 5075/35
PATH: 103/705 218/700 229/426