1. Forum
  2. DOVE-Net
  3. Synchronet Programming

  • src/xpdev/sockwrap.c

    From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Mon Mar 2 22:42:57 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/2773f92cddcec6b459d6a17b
    Modified Files:
    src/xpdev/sockwrap.c
    Log Message:
    nonblocking_connect() must return non-zero upon failure to connect

    on Windows, after the call to the socket_writable() timed out (failed)
    calling getsockopt(... SO_ERROR ...) was setting the result to 0, indicating a successful connection.

    So if getsockopt() reports the socket errno value is 0, don't use that value for the result, default to ETIMEDOUT (60) instead.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Mon Mar 2 22:53:57 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/05695d68d4d420b7b2f00dbc
    Modified Files:
    src/xpdev/sockwrap.c
    Log Message:
    Fix Borland C compile error

    Yup, still using ancient tools to build xpdev

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Mon Mar 2 23:05:36 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/f0faf99aab62e7855c5de682
    Modified Files:
    src/xpdev/sockwrap.c
    Log Message:
    Minimize the diff of these last couple of commits to nonblocking_connect()

    No functional change.

    We're not retaining the original conect() error value, so don't need the new/additional variable (and associated line changes).

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Tue Mar 3 12:55:09 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/e77ef308c29867a147afce0e
    Modified Files:
    src/xpdev/sockwrap.c
    Log Message:
    getsockopt(... SO_ERROR...) should be called when writable

    Commit 01fc91c816 reversed this logic.

    Leaving the getsockopt() in the failure case for now in case the
    wirder range of results is useful somehow.

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