felipe Fri Jan 30 14:38:13 2009 UTC
Modified files: (Branch: PHP_5_3)
/php-src/sapi/apache_hooks php_apache.c
Log:
MFH:
- WS, CS
- Added missing param. checking
- Fixed type for 'l'
http://cvs.php.net/viewvc.cgi/php-src/sapi/apache_hooks/php_apache.c?r1=1.19.2.3.2.5.2.11&r2=1.19.2.3.2.5.2.12&diff_format=u
Index: php-src/sapi/apache_hooks/php_apache.c
diff -u php-src/sapi/apache_hooks/php_apache.c:1.19.2.3.2.5.2.11
php-src/sapi/apache_hooks/php_apache.c:1.19.2.3.2.5.2.12
--- php-src/sapi/apache_hooks/php_apache.c:1.19.2.3.2.5.2.11 Tue Jan 27
21:59:30 2009
+++ php-src/sapi/apache_hooks/php_apache.c Fri Jan 30 14:38:12 2009
@@ -17,7 +17,7 @@
| David Sklar <sk...@student.net> |
+----------------------------------------------------------------------+
*/
-/* $Id: php_apache.c,v 1.19.2.3.2.5.2.11 2009/01/27 21:59:30 iliaa Exp $ */
+/* $Id: php_apache.c,v 1.19.2.3.2.5.2.12 2009/01/30 14:38:12 felipe Exp $ */
#include "php_apache_http.h"
@@ -63,12 +63,6 @@
ZEND_ARG_INFO(0, filename)
ZEND_END_ARG_INFO()
-ZEND_BEGIN_ARG_INFO(arginfo_apachehooks_request_headers, 0)
-ZEND_END_ARG_INFO()
-
-ZEND_BEGIN_ARG_INFO(arginfo_apachehooks_response_headers, 0)
-ZEND_END_ARG_INFO()
-
ZEND_BEGIN_ARG_INFO_EX(arginfo_apachehooks_setenv, 0, 0, 2)
ZEND_ARG_INFO(0, variable)
ZEND_ARG_INFO(0, value)
@@ -79,16 +73,7 @@
ZEND_ARG_INFO(0, uri)
ZEND_END_ARG_INFO()
-ZEND_BEGIN_ARG_INFO(arginfo_apachehooks_get_version, 0)
-ZEND_END_ARG_INFO()
-
-ZEND_BEGIN_ARG_INFO(arginfo_apachehooks_get_modules, 0)
-ZEND_END_ARG_INFO()
-
-ZEND_BEGIN_ARG_INFO(arginfo_apachehooks_request_run, 0)
-ZEND_END_ARG_INFO()
-
-ZEND_BEGIN_ARG_INFO(arginfo_apachehooks_child_terminate, 0)
+ZEND_BEGIN_ARG_INFO(arginfo_apachehooks__void, 0)
ZEND_END_ARG_INFO()
ZEND_BEGIN_ARG_INFO_EX(arginfo_apachehooks_note, 0, 0, 1)
@@ -98,14 +83,14 @@
const zend_function_entry apache_functions[] = {
PHP_FE(virtual,
arginfo_apachehooks_virtual)
- PHP_FE(apache_request_headers,
arginfo_apachehooks_request_headers)
+ PHP_FE(apache_request_headers,
arginfo_apachehooks__void)
PHP_FE(apache_note,
arginfo_apachehooks_note)
PHP_FE(apache_lookup_uri,
arginfo_apachehooks_lookup_uri)
- PHP_FE(apache_child_terminate,
arginfo_apachehooks_child_terminate)
+ PHP_FE(apache_child_terminate,
arginfo_apachehooks__void)
PHP_FE(apache_setenv,
arginfo_apachehooks_setenv)
- PHP_FE(apache_response_headers,
arginfo_apachehooks_response_headers)
- PHP_FE(apache_get_version,
arginfo_apachehooks_get_version)
- PHP_FE(apache_get_modules,
arginfo_apachehooks_get_modules)
+ PHP_FE(apache_response_headers,
arginfo_apachehooks__void)
+ PHP_FE(apache_get_version,
arginfo_apachehooks__void)
+ PHP_FE(apache_get_modules,
arginfo_apachehooks__void)
PHP_FALIAS(getallheaders, apache_request_headers,
arginfo_apachehooks_request_headers)
{NULL, NULL, NULL}
};
@@ -208,8 +193,9 @@
s = *(char **)((char*)r + offset);
- if (s)
+ if (s) {
RETURN_STRING(s, 1);
+ }
RETURN_EMPTY_STRING();
}
@@ -600,8 +586,9 @@
zend_bool replace = 0;
HashPosition pos;
- if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z|zb", &first,
&second, &replace) == FAILURE)
+ if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z|zb", &first,
&second, &replace) == FAILURE) {
RETURN_FALSE;
+ }
if (Z_TYPE_P(first) == IS_ARRAY) {
switch(ZEND_NUM_ARGS()) {
@@ -621,11 +608,11 @@
}
convert_to_string_ex(value);
- if (replace)
+ if (replace) {
ap_table_set(t,
string_key, Z_STRVAL_PP(value));
- else
+ } else {
ap_table_merge(t, string_key, Z_STRVAL_PP(value));
-
+ }
break;
case HASH_KEY_IS_LONG:
default:
@@ -640,23 +627,22 @@
WRONG_PARAM_COUNT;
break;
}
- }
- else if (Z_TYPE_P(first) == IS_STRING) {
+ } else if (Z_TYPE_P(first) == IS_STRING) {
switch(ZEND_NUM_ARGS()) {
case 2:
case 3:
convert_to_string_ex(&second);
- if (replace)
+ if (replace) {
ap_table_set(t, Z_STRVAL_P(first),
Z_STRVAL_P(second));
- else
+ } else {
ap_table_merge(t, Z_STRVAL_P(first),
Z_STRVAL_P(second));
+ }
break;
default:
WRONG_PARAM_COUNT;
break;
}
- }
- else {
+ } else {
RETURN_FALSE;
}
}
@@ -674,8 +660,9 @@
APREQ_GET_REQUEST(id, r);
- if (ZEND_NUM_ARGS() > 0)
+ if (ZEND_NUM_ARGS() > 0) {
add_header_to_table(r->headers_out,
INTERNAL_FUNCTION_PARAM_PASSTHRU);
+ }
apache_table_to_zval(r->headers_out, 0, return_value);
}
@@ -692,8 +679,9 @@
APREQ_GET_REQUEST(id, r);
- if (ZEND_NUM_ARGS() > 0)
+ if (ZEND_NUM_ARGS() > 0) {
add_header_to_table(r->err_headers_out,
INTERNAL_FUNCTION_PARAM_PASSTHRU);
+ }
apache_table_to_zval(r->err_headers_out, 0, return_value);
}
@@ -747,8 +735,9 @@
res = (char *)ap_get_remote_host(r->connection, r->per_dir_config,
type);
- if (res)
+ if (res) {
RETURN_STRING(res, 1);
+ }
RETURN_EMPTY_STRING();
}
@@ -760,7 +749,7 @@
{
zval *id;
request_rec *r;
- int mtime = 0;
+ long mtime = 0;
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|l", &mtime) ==
FAILURE) {
return;
@@ -768,7 +757,7 @@
APREQ_GET_REQUEST(id, r);
- RETURN_LONG(ap_update_mtime(r, mtime));
+ RETURN_LONG(ap_update_mtime(r, (int) mtime));
}
/* }}} */
@@ -910,8 +899,9 @@
APREQ_GET_REQUEST(id, r);
t = (char *)ap_auth_type(r);
- if (!t)
+ if (!t) {
RETURN_NULL();
+ }
RETURN_STRING(t, 1);
}
@@ -932,8 +922,9 @@
APREQ_GET_REQUEST(id, r);
t = (char *)ap_auth_name(r);
- if (!t)
+ if (!t) {
RETURN_NULL();
+ }
RETURN_STRING(t, 1);
}
@@ -948,12 +939,8 @@
const char *pw;
long status;
- if (ZEND_NUM_ARGS() != 1) {
- WRONG_PARAM_COUNT;
- }
-
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &zpw) ==
FAILURE) {
- RETURN_NULL();
+ return;
}
if (!PZVAL_IS_REF(zpw)) {
@@ -961,16 +948,15 @@
RETURN_NULL();
}
-
APREQ_GET_REQUEST(id, r);
pw = NULL;
status = ap_get_basic_auth_pw(r, &pw);
if (status == OK && pw) {
ZVAL_STRING(zpw, (char *)pw, 1);
- }
- else
+ } else {
ZVAL_NULL(zpw);
+ }
RETURN_LONG(status);
}
/* }}} */
@@ -985,8 +971,10 @@
char *type = NULL;
int typelen;
- if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &type,
&typelen) == FAILURE)
+ if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &type,
&typelen) == FAILURE) {
return;
+ }
+
APREQ_GET_REQUEST(id, r);
if(type) {
r->content_type = pstrdup(r->pool, type);
@@ -1002,6 +990,10 @@
zval *id;
request_rec *r;
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
APREQ_GET_REQUEST(id, r);
ap_basic_http_header((request_rec *)SG(server_context));
@@ -1015,6 +1007,10 @@
zval *id;
request_rec *r;
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
APREQ_GET_REQUEST(id, r);
ap_send_http_trace((request_rec *)SG(server_context));
@@ -1028,6 +1024,10 @@
zval *id;
request_rec *r;
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
APREQ_GET_REQUEST(id, r);
ap_send_http_options((request_rec *)SG(server_context));
@@ -1040,20 +1040,20 @@
{
zval *id;
request_rec *r;
- int rec = 0;
+ long rec = 0;
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|l", &rec) ==
FAILURE) {
return;
}
APREQ_GET_REQUEST(id, r);
- ap_send_error_response(r, rec);
+ ap_send_error_response(r, (int) rec);
RETURN_TRUE;
}
PHP_FUNCTION(apache_request_set_content_length)
{
- int length;
+ long length;
zval *id;
request_rec *r;
@@ -1071,6 +1071,11 @@
{
zval *id;
request_rec *r;
+
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
APREQ_GET_REQUEST(id, r);
ap_set_keepalive(r);
RETURN_TRUE;
@@ -1136,7 +1141,7 @@
zval *id;
char *z_errstr;
int z_errstr_len;
- int facility = APLOG_ERR;
+ long facility = APLOG_ERR;
request_rec *r;
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|l", &z_errstr,
&z_errstr_len, &facility) == FAILURE) {
@@ -1144,7 +1149,7 @@
}
APREQ_GET_REQUEST(id, r);
- ap_log_error(APLOG_MARK, facility, r->server, "%s", z_errstr);
+ ap_log_error(APLOG_MARK, (int) facility, r->server, "%s", z_errstr);
RETURN_TRUE;
}
/* }}} */
@@ -1237,9 +1242,14 @@
request_rec *r;
int status;
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
APREQ_GET_REQUEST(id, r);
- if(!r || ap_is_initial_req(r))
+ if (!r || ap_is_initial_req(r)) {
RETURN_FALSE;
+ }
status = ap_run_sub_req(r);
ap_destroy_sub_req(r);
RETURN_LONG(status);
@@ -1781,6 +1791,10 @@
Fetch all HTTP request headers */
PHP_FUNCTION(apache_request_headers)
{
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
apache_table_to_zval(((request_rec *)SG(server_context))->headers_in,
PG(safe_mode), return_value);
}
/* }}} */
@@ -1789,6 +1803,10 @@
Fetch all HTTP response headers */
PHP_FUNCTION(apache_response_headers)
{
+ if (zend_parse_parameters_none() == FAILURE) {
+ return;
+ }
+
apache_table_to_zval(((request_rec *) SG(server_context))->headers_out,
0, return_value);
}
/* }}} */
@@ -1807,7 +1825,9 @@
}
while(top) {
- if(r->prev) r = r->prev;
+ if (r->prev) {
+ r = r->prev;
+ }
else break;
}
@@ -1828,7 +1848,6 @@
return;
}
-
if(!(rr = sub_req_lookup_uri(filename, ((request_rec *)
SG(server_context))))) {
php_error(E_WARNING, "URI lookup failed", filename);
RETURN_FALSE;
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
