]> granicus.if.org Git - php/commitdiff
Fix Windows build for shared ext/sockets
authorChristoph M. Becker <cmbecker69@gmx.de>
Tue, 8 Sep 2020 20:51:46 +0000 (22:51 +0200)
committerChristoph M. Becker <cmbecker69@gmx.de>
Tue, 8 Sep 2020 20:54:21 +0000 (22:54 +0200)
We can't use `PHPAPI` for extensions which may be built shared.  Thus,
we introduce `PHP_SOCKETS_API`.

ext/sockets/config.w32
ext/sockets/php_sockets.h

index aeb419c4f06ab56dc7c53f81bd14e580d5792aa8..ad9becab2a1705af274d334311d1849e025a796d 100644 (file)
@@ -9,6 +9,7 @@ if (PHP_SOCKETS != "no") {
                EXTENSION('sockets', 'sockets.c multicast.c conversions.c sockaddr_conv.c sendrecvmsg.c', PHP_SOCKETS_SHARED, "/DZEND_ENABLE_STATIC_TSRMLS_CACHE=1");
                AC_DEFINE('HAVE_SOCKETS', 1);
                PHP_INSTALL_HEADERS("ext/sockets", "php_sockets.h windows_common.h");
+               ADD_FLAG("CFLAGS_SOCKETS", "/D PHP_SOCKETS_EXPORTS=1");
        } else {
                WARNING("sockets not enabled; libraries and headers not found");
        }
index 7be3c4e9ea2e58a30d76d5299de608b173efe19d..e31ebc753063f41f9d6a44ff91b3da34f52858d1 100644 (file)
@@ -52,6 +52,18 @@ typedef int PHP_SOCKET;
 typedef SOCKET PHP_SOCKET;
 #endif
 
+#ifdef PHP_WIN32
+#      ifdef PHP_SOCKETS_EXPORTS
+#              define PHP_SOCKETS_API __declspec(dllexport)
+#      else
+#              define PHP_SOCKETS_API __declspec(dllimport)
+#      endif
+#elif defined(__GNUC__) && __GNUC__ >= 4
+#      define PHP_SOCKETS_API __attribute__ ((visibility("default")))
+#else
+#      define PHP_SOCKETS_API
+#endif
+
 /* Socket class */
 
 typedef struct {
@@ -63,7 +75,7 @@ typedef struct {
        zend_object std;
 } php_socket;
 
-extern PHPAPI zend_class_entry *socket_ce;
+extern PHP_SOCKETS_API zend_class_entry *socket_ce;
 
 static inline php_socket *socket_from_obj(zend_object *obj) {
        return (php_socket *)((char *)(obj) - XtOffsetOf(php_socket, std));
@@ -104,7 +116,7 @@ ZEND_BEGIN_MODULE_GLOBALS(sockets)
 #endif
 ZEND_END_MODULE_GLOBALS(sockets)
 
-PHPAPI ZEND_EXTERN_MODULE_GLOBALS(sockets)
+PHP_SOCKETS_API ZEND_EXTERN_MODULE_GLOBALS(sockets)
 #define SOCKETS_G(v) ZEND_MODULE_GLOBALS_ACCESSOR(sockets, v)
 
 enum sockopt_return {
@@ -113,8 +125,8 @@ enum sockopt_return {
        SOCKOPT_SUCCESS
 };
 
-PHPAPI char *sockets_strerror(int error);
-PHPAPI int socket_import_file_descriptor(PHP_SOCKET socket, php_socket *retsock);
+PHP_SOCKETS_API char *sockets_strerror(int error);
+PHP_SOCKETS_API int socket_import_file_descriptor(PHP_SOCKET socket, php_socket *retsock);
 
 #else
 #define phpext_sockets_ptr NULL