1. Forum
  2. Fido
  3. SYNC PROGRAMMING

  • src/xpdev/sockwrap.c

    From Rob Swindell (on Windows 11)@1:103/705 to Git commit to main/sbbs/master on Mon Mar 2 22:42:56 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.
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Windows 11)@1:103/705 to Git commit to main/sbbs/master on Mon Mar 2 22:53:56 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
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Windows 11)@1:103/705 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).
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Deucе@1:103/705 to Git commit to main/sbbs/master on Tue Mar 3 12:55:08 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.
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)