andrei Thu, 18 Mar 2010 22:37:25 +0000
Revision: http://svn.php.net/viewvc?view=revision&revision=296362
Log:
I am sorry I tried fixing PHP without extensive discussion on the mailing list.
I am sorry I tried fixing PHP without extensive discussion on the mailing list.
I am sorry I tried fixing PHP without extensive discussion on the mailing list.
Hope all the relevant parties are satisfied.
Changed paths:
U php/php-src/branches/PHP_5_2/configure.in
U php/php-src/branches/PHP_5_2/ext/standard/file.c
U php/php-src/branches/PHP_5_2/main/SAPI.c
U php/php-src/branches/PHP_5_2/main/SAPI.h
U php/php-src/branches/PHP_5_2/main/rfc1867.c
U php/php-src/branches/PHP_5_3/configure.in
U php/php-src/branches/PHP_5_3/ext/standard/file.c
U php/php-src/branches/PHP_5_3/main/SAPI.c
U php/php-src/branches/PHP_5_3/main/SAPI.h
U php/php-src/branches/PHP_5_3/main/rfc1867.c
Modified: php/php-src/branches/PHP_5_2/configure.in
===================================================================
--- php/php-src/branches/PHP_5_2/configure.in 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_2/configure.in 2010-03-18 22:37:25 UTC (rev 296362)
@@ -896,12 +896,6 @@
AC_MSG_RESULT([using system default])
fi
-PHP_ARG_ENABLE(large-uploads-fix, whether to enable large files (2G-4G) uploads fix,
-[ --enable-large-uploads-fix Enable large files (2G-4G) uploads fix], no, no)
-if test "$PHP_LARGE_UPLOADS_FIX" = "yes"; then
- AC_DEFINE(HAVE_LARGE_UPLOADS_FIX, 1, [Whether to enable large files (2G-4G) uploads fix])
-fi
-
divert(5)
dnl ## In diversion 5 we check which extensions should be compiled.
Modified: php/php-src/branches/PHP_5_2/ext/standard/file.c
===================================================================
--- php/php-src/branches/PHP_5_2/ext/standard/file.c 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_2/ext/standard/file.c 2010-03-18 22:37:25 UTC (rev 296362)
@@ -579,8 +579,7 @@
char *filename;
int filename_len;
zval *data;
- size_t numbytes = 0;
- int result = SUCCESS;
+ int numbytes = 0;
long flags = 0;
zval *zcontext = NULL;
php_stream_context *context = NULL;
@@ -623,7 +622,7 @@
case IS_RESOURCE: {
size_t len;
if (php_stream_copy_to_stream_ex(srcstream, stream, PHP_STREAM_COPY_ALL, &len) != SUCCESS) {
- result = FAILURE;
+ numbytes = -1;
} else {
numbytes = len;
}
@@ -641,7 +640,7 @@
numbytes = php_stream_write(stream, Z_STRVAL_P(data), Z_STRLEN_P(data));
if (numbytes != Z_STRLEN_P(data)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Only %d of %d bytes written, possibly out of free disk space", numbytes, Z_STRLEN_P(data));
- result = FAILURE;
+ numbytes = -1;
}
}
break;
@@ -667,7 +666,7 @@
} else {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Only %d of %d bytes written, possibly out of free disk space", bytes_written, Z_STRLEN_PP(tmp));
}
- result = FAILURE;
+ numbytes = -1;
break;
}
}
@@ -684,19 +683,19 @@
numbytes = php_stream_write(stream, Z_STRVAL(out), Z_STRLEN(out));
if (numbytes != Z_STRLEN(out)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Only %d of %d bytes written, possibly out of free disk space", numbytes, Z_STRLEN(out));
- result = FAILURE;
+ numbytes = -1;
}
zval_dtor(&out);
break;
}
}
default:
- result = FAILURE;
+ numbytes = -1;
break;
}
php_stream_close(stream);
- if (result == FAILURE) {
+ if (numbytes < 0) {
RETURN_FALSE;
}
Modified: php/php-src/branches/PHP_5_2/main/SAPI.c
===================================================================
--- php/php-src/branches/PHP_5_2/main/SAPI.c 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_2/main/SAPI.c 2010-03-18 22:37:25 UTC (rev 296362)
@@ -191,8 +191,8 @@
SAPI_API SAPI_POST_READER_FUNC(sapi_read_standard_form_data)
{
- size_t read_bytes;
- size_t allocated_bytes=SAPI_POST_BLOCK_SIZE+1;
+ int read_bytes;
+ int allocated_bytes=SAPI_POST_BLOCK_SIZE+1;
if (SG(request_info).content_length > SG(post_max_size)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "POST Content-Length of %ld bytes exceeds the limit of %ld bytes",
Modified: php/php-src/branches/PHP_5_2/main/SAPI.h
===================================================================
--- php/php-src/branches/PHP_5_2/main/SAPI.h 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_2/main/SAPI.h 2010-03-18 22:37:25 UTC (rev 296362)
@@ -118,11 +118,7 @@
void *server_context;
sapi_request_info request_info;
sapi_headers_struct sapi_headers;
-#ifdef HAVE_LARGE_UPLOADS_FIX
- size_t read_post_bytes;
-#else
int read_post_bytes;
-#endif
unsigned char headers_sent;
struct stat global_stat;
char *default_mimetype;
Modified: php/php-src/branches/PHP_5_2/main/rfc1867.c
===================================================================
--- php/php-src/branches/PHP_5_2/main/rfc1867.c 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_2/main/rfc1867.c 2010-03-18 22:37:25 UTC (rev 296362)
@@ -765,9 +765,8 @@
{
char *boundary, *s = NULL, *boundary_end = NULL, *start_arr = NULL, *array_index = NULL;
char *temp_filename = NULL, *lbuf = NULL, *abuf = NULL;
- int boundary_len = 0, cancel_upload = 0, is_arr_upload = 0, array_len = 0;
- int skip_upload = 0, anonindex = 0, is_anonymous;
- size_t total_bytes = 0, max_file_size = 0;
+ int boundary_len = 0, total_bytes = 0, cancel_upload = 0, is_arr_upload = 0, array_len = 0;
+ int max_file_size = 0, skip_upload = 0, anonindex = 0, is_anonymous;
zval *http_post_files = NULL;
HashTable *uploaded_files = NULL;
#if HAVE_MBSTRING && !defined(COMPILE_DL_MBSTRING)
Modified: php/php-src/branches/PHP_5_3/configure.in
===================================================================
--- php/php-src/branches/PHP_5_3/configure.in 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_3/configure.in 2010-03-18 22:37:25 UTC (rev 296362)
@@ -916,12 +916,6 @@
AC_MSG_RESULT([using system default])
fi
-PHP_ARG_ENABLE(large-uploads-fix, whether to enable large files (2G-4G) uploads fix,
-[ --enable-large-uploads-fix Enable large files (2G-4G) uploads fix], no, no)
-if test "$PHP_LARGE_UPLOADS_FIX" = "yes"; then
- AC_DEFINE(HAVE_LARGE_UPLOADS_FIX, 1, [Whether to enable large files (2G-4G) uploads fix])
-fi
-
divert(5)
dnl ## In diversion 5 we check which extensions should be compiled.
Modified: php/php-src/branches/PHP_5_3/ext/standard/file.c
===================================================================
--- php/php-src/branches/PHP_5_3/ext/standard/file.c 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_3/ext/standard/file.c 2010-03-18 22:37:25 UTC (rev 296362)
@@ -584,8 +584,7 @@
char *filename;
int filename_len;
zval *data;
- size_t numbytes = 0;
- int result = SUCCESS;
+ int numbytes = 0;
long flags = 0;
zval *zcontext = NULL;
php_stream_context *context = NULL;
@@ -635,7 +634,7 @@
case IS_RESOURCE: {
size_t len;
if (php_stream_copy_to_stream_ex(srcstream, stream, PHP_STREAM_COPY_ALL, &len) != SUCCESS) {
- result = FAILURE;
+ numbytes = -1;
} else {
numbytes = len;
}
@@ -653,7 +652,7 @@
numbytes = php_stream_write(stream, Z_STRVAL_P(data), Z_STRLEN_P(data));
if (numbytes != Z_STRLEN_P(data)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Only %d of %d bytes written, possibly out of free disk space", numbytes, Z_STRLEN_P(data));
- result = FAILURE;
+ numbytes = -1;
}
}
break;
@@ -679,7 +678,7 @@
} else {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Only %d of %d bytes written, possibly out of free disk space", bytes_written, Z_STRLEN_PP(tmp));
}
- result = FAILURE;
+ numbytes = -1;
break;
}
}
@@ -696,19 +695,19 @@
numbytes = php_stream_write(stream, Z_STRVAL(out), Z_STRLEN(out));
if (numbytes != Z_STRLEN(out)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Only %d of %d bytes written, possibly out of free disk space", numbytes, Z_STRLEN(out));
- result = FAILURE;
+ numbytes = -1;
}
zval_dtor(&out);
break;
}
}
default:
- result = FAILURE;
+ numbytes = -1;
break;
}
php_stream_close(stream);
- if (result == FAILURE) {
+ if (numbytes < 0) {
RETURN_FALSE;
}
Modified: php/php-src/branches/PHP_5_3/main/SAPI.c
===================================================================
--- php/php-src/branches/PHP_5_3/main/SAPI.c 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_3/main/SAPI.c 2010-03-18 22:37:25 UTC (rev 296362)
@@ -191,8 +191,8 @@
SAPI_API SAPI_POST_READER_FUNC(sapi_read_standard_form_data)
{
- size_t read_bytes;
- size_t allocated_bytes=SAPI_POST_BLOCK_SIZE+1;
+ int read_bytes;
+ int allocated_bytes=SAPI_POST_BLOCK_SIZE+1;
if (SG(request_info).content_length > SG(post_max_size)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "POST Content-Length of %ld bytes exceeds the limit of %ld bytes",
Modified: php/php-src/branches/PHP_5_3/main/SAPI.h
===================================================================
--- php/php-src/branches/PHP_5_3/main/SAPI.h 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_3/main/SAPI.h 2010-03-18 22:37:25 UTC (rev 296362)
@@ -120,11 +120,7 @@
void *server_context;
sapi_request_info request_info;
sapi_headers_struct sapi_headers;
-#ifdef HAVE_LARGE_UPLOADS_FIX
- size_t read_post_bytes;
-#else
int read_post_bytes;
-#endif
unsigned char headers_sent;
struct stat global_stat;
char *default_mimetype;
Modified: php/php-src/branches/PHP_5_3/main/rfc1867.c
===================================================================
--- php/php-src/branches/PHP_5_3/main/rfc1867.c 2010-03-18 22:07:51 UTC (rev 296361)
+++ php/php-src/branches/PHP_5_3/main/rfc1867.c 2010-03-18 22:37:25 UTC (rev 296362)
@@ -764,9 +764,8 @@
{
char *boundary, *s = NULL, *boundary_end = NULL, *start_arr = NULL, *array_index = NULL;
char *temp_filename = NULL, *lbuf = NULL, *abuf = NULL;
- int boundary_len = 0, cancel_upload = 0, is_arr_upload = 0, array_len = 0;
- int skip_upload = 0, anonindex = 0, is_anonymous;
- size_t total_bytes = 0, max_file_size = 0;
+ int boundary_len = 0, total_bytes = 0, cancel_upload = 0, is_arr_upload = 0, array_len = 0;
+ int max_file_size = 0, skip_upload = 0, anonindex = 0, is_anonymous;
zval *http_post_files = NULL;
HashTable *uploaded_files = NULL;
#if HAVE_MBSTRING && !defined(COMPILE_DL_MBSTRING)
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
