iliaa           Tue Dec 26 16:55:56 2006 UTC

  Modified files:              (Branch: PHP_5_2)
    /php-src/ext/curl   interface.c 
    /php-src    NEWS 
  Log:
  Fixed bug #39878 (CURL doesn't compile on Sun Studio Pro).
  
  
http://cvs.php.net/viewvc.cgi/php-src/ext/curl/interface.c?r1=1.62.2.14.2.16&r2=1.62.2.14.2.17&diff_format=u
Index: php-src/ext/curl/interface.c
diff -u php-src/ext/curl/interface.c:1.62.2.14.2.16 
php-src/ext/curl/interface.c:1.62.2.14.2.17
--- php-src/ext/curl/interface.c:1.62.2.14.2.16 Tue Dec 19 21:41:25 2006
+++ php-src/ext/curl/interface.c        Tue Dec 26 16:55:56 2006
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: interface.c,v 1.62.2.14.2.16 2006/12/19 21:41:25 mike Exp $ */
+/* $Id: interface.c,v 1.62.2.14.2.17 2006/12/26 16:55:56 iliaa Exp $ */
 
 #define ZEND_INCLUDE_FULL_WINDOWS_HEADERS
 
@@ -156,6 +156,12 @@
 #define CAAS(s, v) add_assoc_string_ex(return_value, s, sizeof(s), (char *) v, 
1);
 #define CAAZ(s, v) add_assoc_zval_ex(return_value, s, sizeof(s), (zval *) v);
 
+#if defined(PHP_WIN32) || defined(__GNUC__)
+ #define php_curl_ret(__ret) RETVAL_FALSE; return __ret;
+#else
+ #define php_curl_ret(__ret) RETVAL_FALSE; return;
+#endif
+
 #define PHP_CURL_CHECK_OPEN_BASEDIR(str, len, __ret)                           
                                                                        \
        if (((PG(open_basedir) && *PG(open_basedir)) || PG(safe_mode)) &&       
                                         \
            strncasecmp(str, "file:", sizeof("file:") - 1) == 0)                
                                                \
@@ -164,22 +170,19 @@
                                                                                
                                        \
                if (!(tmp_url = php_url_parse_ex(str, len))) {                  
                                                                \
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid 
URL '%s'", str);                           \
-                       RETVAL_FALSE;                                           
                                        \
-                       return __ret;                                           
                                        \
-               }                                                               
                                                                                
                                \
+                       php_curl_ret(__ret);                                    
                                                \
+               }                                                               
                                        \
                                                                                
                                        \
                if (php_memnstr(str, tmp_url->path, strlen(tmp_url->path), str 
+ len)) {                                \
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "URL '%s' 
contains unencoded control characters.", str);    \
-                       RETVAL_FALSE;                                           
                                        \
-                       return __ret;                                           
                                        \
+                       php_curl_ret(__ret);                                    
                                                \
                }                                                               
                                        \
                                                                                
                                                                                
                                \
                if (tmp_url->query || tmp_url->fragment || 
php_check_open_basedir(tmp_url->path TSRMLS_CC) ||                              
                                     \
                        (PG(safe_mode) && !php_checkuid(tmp_url->path, "rb+", 
CHECKUID_CHECK_MODE_PARAM))       \
                ) {                                                             
                                                                                
                        \
                        php_url_free(tmp_url);                                  
                                                                                
        \
-                       RETVAL_FALSE;                                           
                                        \
-                       return __ret;                                           
                                        \
+                       php_curl_ret(__ret);                                    
                                                \
                }                                                               
                                                                                
                                \
                php_url_free(tmp_url);                                          
                                                                                
        \
        }
http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.462&r2=1.2027.2.547.2.463&diff_format=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.547.2.462 php-src/NEWS:1.2027.2.547.2.463
--- php-src/NEWS:1.2027.2.547.2.462     Tue Dec 26 09:52:04 2006
+++ php-src/NEWS        Tue Dec 26 16:55:56 2006
@@ -29,6 +29,7 @@
   install path). (Tony)
 - Fixed bug #39884 (ReflectionParameter::getClass() throws exception for type
   hint self). (thekid at php dot net)
+- Fixed bug #39878 (CURL doesn't compile on Sun Studio Pro). (Ilia)
 - Fixed bug #39873 (number_format() breaks with locale & decimal points).
   (Ilia)
 - Fixed bug #39869 (safe_read does not initialize errno).

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to