Skip to content

Commit e2b3ce7

Browse files
committed
Use session_cookie_len rather than calling strlen
1 parent 926fa17 commit e2b3ce7

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

ext/session/session.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1337,13 +1337,14 @@ static int php_session_cache_limiter(void)
13371337

13381338
static void php_session_remove_cookie(void) {
13391339
char *session_cookie;
1340+
size_t session_cookie_len;
13401341
sapi_header_line header_line = {0};
13411342

13421343
ZEND_ASSERT(strpbrk(ZSTR_VAL(PS(session_name)), SESSION_FORBIDDEN_CHARS) == NULL);
13431344
session_cookie_len = spprintf(&session_cookie, 0, "Set-Cookie: %s=", ZSTR_VAL(PS(session_name)));
13441345

13451346
header_line.line = session_cookie;
1346-
header_line.line_len = strlen(session_cookie);
1347+
header_line.line_len = session_cookie_len;
13471348
header_line.header_len = sizeof("Set-Cookie") - 1;
13481349
sapi_header_op(SAPI_HEADER_DELETE_PREFIX, &header_line);
13491350

0 commit comments

Comments
 (0)