com php-src: avoid multiple strlen() calls for the same: main/fopen_wrappers.c
| From: | Anatol Belski | Date: | Sat, 13 Sep 2014 21:21:40 +0000 |
| Subject: | com php-src: avoid multiple strlen() calls for the same: main/fopen_wrappers.c | ||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-81558@lists.php.net to get a copy of this message | ||
Commit: 586b863ba754a19495f4abaadd32df85a394616d
Author: Anatol Belski <ab@php.net> Sat, 13 Sep 2014 17:58:47 +0200
Parents: 47fb7a46ac1ff29b065fd5e46b0c3174e7e2b2a0
Branches: master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=586b863ba754a19495f4abaadd32df85a394616d
Log:
avoid multiple strlen() calls for the same
Changed paths:
M main/fopen_wrappers.c
Diff:
diff --git a/main/fopen_wrappers.c b/main/fopen_wrappers.c
index e375bdf..b0eec9a 100644
--- a/main/fopen_wrappers.c
+++ b/main/fopen_wrappers.c
@@ -758,10 +758,11 @@ PHPAPI char *expand_filepath_with_mode(const char *filepath, char *real_path,
co
cwd_state new_state;
char cwd[MAXPATHLEN];
int copy_len;
+ int path_len = strlen(filepath);
if (!filepath[0]) {
return NULL;
- } else if (IS_ABSOLUTE_PATH(filepath, strlen(filepath))) {
+ } else if (IS_ABSOLUTE_PATH(filepath, path_len)) {
cwd[0] = '\0';
} else {
const char *iam = SG(request_info).path_translated;
@@ -784,7 +785,7 @@ PHPAPI char *expand_filepath_with_mode(const char *filepath, char *real_path, co
/* return a relative file path if for any reason
* we cannot cannot getcwd() and the requested,
* relatively referenced file is accessible */
- copy_len = strlen(filepath) > MAXPATHLEN - 1 ? MAXPATHLEN - 1 : strlen(filepath);
+ copy_len = path_len > MAXPATHLEN - 1 ? MAXPATHLEN - 1 : path_len;
if (real_path) {
memcpy(real_path, filepath, copy_len);
real_path[copy_len] = '\0';