-
Notifications
You must be signed in to change notification settings - Fork 280
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MinGW Compile fix #329
MinGW Compile fix #329
Conversation
When an ICMP message is received on Windows 10 for a UDP socket, WSAECONNRESET is reported as an error. In this case, just read again. Thanks to nxrighthere for reporting the issue and helping to nail it down. This fixes sctplab#309.
Two Links missed leading `http(s)://` and were broken. All links to `tools.ietf.org` now consistently use https.
SCTP_CONNECT_X and SCTP_CONNECT_X_DELAYED.
# Conflicts: # usrsctplib/netinet/sctp_usrreq.c # usrsctplib/netinet/sctputil.c
# Conflicts: # programs/ekr_loop.c
- Set minimum Windows Version to Windows 7
usrsctplib/netinet/sctp_pcb.c
Outdated
@@ -6852,7 +6852,9 @@ sctp_pcb_init() | |||
#endif | |||
#if defined(__Userspace__) | |||
mbuf_initialize(NULL); | |||
#if !defined(__MINGW32__) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't insert an ifdef
here. Let us call atomic_init()
in any case for the userland stack.
Please replace in user_atomic.h
#if defined(__Userspace_os_Windows)
#if !defined(__MINGW32__)
static void atomic_init(void) {} /* empty when we are not using atomic_mtx */
#endif
#else
static inline void atomic_init(void) {} /* empty when we are not using atomic_mtx */
#endif
by
#if defined(__Userspace_os_Windows)
static void atomic_init(void) {} /* empty when we are not using atomic_mtx */
#else
static inline void atomic_init(void) {} /* empty when we are not using atomic_mtx */
#endif
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
(Hopefully) solves #276