From: Sascha Schumann Date: Thu, 2 Nov 2000 15:47:02 +0000 (+0000) Subject: Const'ify part of sapi_request_info. X-Git-Tag: php-4.0.4RC3~384 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=43ff395b65234ebc606549720b24387ec5a79d06;p=php Const'ify part of sapi_request_info. query_string, post_data, path_translated, request_uri, auth_user, auth_password and argv0 are modified, so they cannot become const. --- diff --git a/ext/session/session.c b/ext/session/session.c index 1ed71dace0..ffe7cd89c0 100644 --- a/ext/session/session.c +++ b/ext/session/session.c @@ -641,7 +641,7 @@ static void strcat_gmt(char *ubuf, time_t *when) static void last_modified(void) { - char *path; + const char *path; struct stat sb; char buf[MAX_STR + 1]; SLS_FETCH(); diff --git a/main/SAPI.h b/main/SAPI.h index e551a32165..31e6872ab8 100644 --- a/main/SAPI.h +++ b/main/SAPI.h @@ -61,17 +61,17 @@ extern sapi_module_struct sapi_module; /* true global */ typedef struct { - char *request_method; + const char *request_method; char *query_string; char *post_data; - char *cookie_data; + const char *cookie_data; long content_length; uint post_data_length; char *path_translated; char *request_uri; - char *content_type; + const char *content_type; unsigned char headers_only; diff --git a/main/main.c b/main/main.c index 73e1b3e54d..3a8af10da3 100644 --- a/main/main.c +++ b/main/main.c @@ -1140,10 +1140,9 @@ static void php_build_argv(char *s, zval *track_vars_array ELS_DC PLS_DC) PHPAPI int php_handle_special_queries(SLS_D PLS_DC) { if (SG(request_info).query_string && SG(request_info).query_string[0]=='=' - && PG(expose_php)) { - if(php_info_logos(SG(request_info).query_string+1)) - { - return 1; + && PG(expose_php)) { + if (php_info_logos(SG(request_info).query_string+1)) { + return 1; } else if (!strcmp(SG(request_info).query_string+1, PHP_CREDITS_GUID)) { php_print_credits(PHP_CREDITS_ALL); return 1; diff --git a/main/php_logos.c b/main/php_logos.c index 94a0830c2f..aa66843d60 100644 --- a/main/php_logos.c +++ b/main/php_logos.c @@ -68,7 +68,7 @@ int php_shutdown_info_logos(void) } #define CONTENT_TYPE_HEADER "Content-Type: " -int php_info_logos(char *logo_string) +int php_info_logos(const char *logo_string) { php_info_logo *logo_image; char *content_header; diff --git a/main/php_logos.h b/main/php_logos.h index d079adf5df..54c61d2632 100644 --- a/main/php_logos.h +++ b/main/php_logos.h @@ -5,6 +5,6 @@ PHPAPI int php_register_info_logo(char *logo_string, char *mimetype, unsigned ch PHPAPI int php_unregister_info_logos(char *logo_string); int php_init_info_logos(void); int php_shutdown_info_logos(void); -int php_info_logos(char *logo_string); +int php_info_logos(const char *logo_string); #endif /* _PHP_LOGOS_H */ diff --git a/main/php_variables.c b/main/php_variables.c index 4187d00eba..bff78e4a79 100644 --- a/main/php_variables.c +++ b/main/php_variables.c @@ -212,6 +212,7 @@ SAPI_POST_HANDLER_FUNC(php_std_post_handler) void php_treat_data(int arg, char *str, zval* destArray ELS_DC PLS_DC SLS_DC) { char *res = NULL, *var, *val; + const char *c_var; pval *array_ptr; int free_buffer=0; char *strtok_buf = NULL; @@ -246,17 +247,17 @@ void php_treat_data(int arg, char *str, zval* destArray ELS_DC PLS_DC SLS_DC) } if (arg == PARSE_GET) { /* GET data */ - var = SG(request_info).query_string; - if (var && *var) { - res = (char *) estrdup(var); + c_var = SG(request_info).query_string; + if (c_var && *c_var) { + res = (char *) estrdup(c_var); free_buffer = 1; } else { free_buffer = 0; } } else if (arg == PARSE_COOKIE) { /* Cookie data */ - var = SG(request_info).cookie_data; - if (var && *var) { - res = (char *) estrdup(var); + c_var = SG(request_info).cookie_data; + if (c_var && *c_var) { + res = (char *) estrdup(c_var); free_buffer = 1; } else { free_buffer = 0; diff --git a/sapi/apache2filter/sapi_apache2.c b/sapi/apache2filter/sapi_apache2.c index 00f2b92284..d7d899b031 100644 --- a/sapi/apache2filter/sapi_apache2.c +++ b/sapi/apache2filter/sapi_apache2.c @@ -298,9 +298,9 @@ static int php_output_filter(ap_filter_t *f, ap_bucket_brigade *bb) /* XXX: Lots of startup crap. Should be moved into its own func */ PG(during_request_startup) = 0; SG(sapi_headers).http_response_code = 200; - SG(request_info).content_type = (char *) apr_table_get(f->r->headers_in, "Content-Type"); + SG(request_info).content_type = apr_table_get(f->r->headers_in, "Content-Type"); SG(request_info).query_string = f->r->args; - SG(request_info).request_method = (char *) f->r->method; + SG(request_info).request_method = f->r->method; SG(request_info).request_uri = f->r->uri; f->r->no_cache = f->r->no_local_copy = 1; content_type = sapi_get_default_content_type(SLS_C);