curl / Mailing Lists / curl-library / Single Mail
Buy commercial curl support from WolfSSL. We help you work out your issues, debug your libcurl applications, use the API, port to new platforms, add new features and more. With a team lead by the curl founder himself.

tests/server/sws.c: S_ISSOCK is undefined

From: Kevin R. Bulgrien via curl-library <curl-library_at_cool.haxx.se>
Date: Mon, 19 Apr 2021 21:24:05 -0500 (CDT)

On SCO OpenServer 5.0.7, /usr/include/sys/stat.h does not define S_IFSOCK.
This is conditionally required when USE_UNIX_SOCKETS is defined. This
impedes `make tests` on this platform.

Please consider a patch similar to the following:

$ cat ../curl-7.76.0-i686-pc-sco3.2v5.0.7-4.patch
--- curl-7.76.0/tests/server/sws.c.orig 2021-03-30 07:39:13.000000000 -0500
+++ curl-7.76.0/tests/server/sws.c 2021-04-19 19:37:03.000000000 -0500
_at__at_ -2092,11 +2092,13 _at__at_
                unix_socket, errno, strerror(errno));
         goto sws_cleanup;
       }
+#ifdef S_IFSOCK
       if((statbuf.st_mode & S_IFSOCK) != S_IFSOCK) {
         logmsg("Error binding socket, failed to stat %s: (%d) %s",
                unix_socket, error, strerror(error));
         goto sws_cleanup;
       }
+#endif /* S_IFSOCK */
       /* dead socket, cleanup and retry bind */
       rc = unlink(unix_socket);
       if(0 != rc) {

I believe this may be reasonable in that S_IFSOCK appears to only
pedantically check for a type on a socket connection that has
already failed, but do please feel free to correct any
misconception on my part.

When I applied the patch, I was able to run the tests:

TESTDONE: 1419 tests were considered during 783 seconds.
TESTDONE: 1009 tests out of 1069 reported OK: 94%

TESTFAIL: These test cases failed: 574 575 576 586 588 591 592 593 595 596 597 661 706 707 711 712 713 714 715 1000 1003 1005 1006 1010 1013 1014 1022 1023 1028 1036 1037 1038 1039 1047 1055 1057 1062 1086 1233 1388 1389 1390 1391 1392 1393 1405 1414 1444 1449 1501 1569 1570 1913 1914 2000 2001 2002 2003 2039 2045

On the bright side, more tests passed than failed. LOL.

--
Kevin R. Bulgrien
-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette:   https://curl.se/mail/etiquette.html
Received on 2021-04-20