• src/sbbs3/services.cpp

    From Rob Swindell (on Debian Linux)@1:103/705 to Git commit to main/sbbs/master on Thu Jun 4 19:01:13 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/cfa6fe9e126476a9f3db8384
    Modified Files:
    src/sbbs3/services.cpp
    Log Message:
    services: don't apply JS disconnection check to static services

    ead5ccf16 (song-11-earn) added a disconnection check to the Services
    server's js_OperationCallback() that aborts a script once its client
    socket has been gone for 10 consecutive callbacks (socket_check() of client->socket fails). This is correct for per-connection services
    (accepted client socket), but static services (SERVICE_OPT_STATIC -
    e.g. the IRC daemon, MRC connector) run via js_static_service_thread()
    with a zero-initialized service_client whose .socket is never set (0). socket_check(0, ...) returns false, so auto_terminate static services
    were wrongly warned "Disconnected" and aborted after 10 callbacks, then
    (being STATIC_LOOP) immediately restarted - cycling endlessly.

    Gate the disconnection check on !(client->service->options & SERVICE_OPT_STATIC) so it only runs for real per-client connections.

    Fixes #1156 (reported by Accession).

    Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Debian Linux)@1:103/705 to Git commit to main/sbbs/master on Thu Jun 4 19:13:32 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/55a7e431ba36a37bb5b64c79
    Modified Files:
    src/sbbs3/services.cpp
    Log Message:
    services: init static-service client socket to INVALID_SOCKET, not 0

    js_static_service_thread() memset()s its service_client to zero and never assigns .socket, leaving it 0 - a valid file descriptor (stdin), not the codebase's "no socket" sentinel. The thread already passes INVALID_SOCKET
    to js_initcx(), but js_initcx() doesn't store its sock arg into the struct,
    so the field silently stayed 0. Set it to INVALID_SOCKET explicitly so no
    code path mistakes fd 0 for a live client socket.

    Hygiene follow-up to cfa6fe9e1 (bolt-11-banner) / #1156; not a behavior
    change (socket_check() already returns false for both 0 and INVALID_SOCKET,
    so the SERVICE_OPT_STATIC guard remains the actual fix).

    Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)