1
0
mirror of https://github.com/php/php-src.git synced 2026-04-10 17:43:13 +02:00

Merge branch 'PHP-5.6'

* PHP-5.6:
  Fixed session module is sending multiple set-cookie headers when session.use_strict_mode=1
This commit is contained in:
Yasuo Ohgaki
2014-01-15 11:07:51 +09:00
3 changed files with 13 additions and 3 deletions

View File

@@ -342,6 +342,7 @@ PS_READ_FUNC(files)
PS(send_cookie) = 1;
}
php_session_reset_id(TSRMLS_C);
PS(session_status) = php_session_active;
}
ps_files_open(data, PS(id) TSRMLS_CC);

View File

@@ -371,6 +371,7 @@ PS_READ_FUNC(mm)
PS(send_cookie) = 1;
}
php_session_reset_id(TSRMLS_C);
PS(session_status) = php_session_active;
}
sd = ps_sd_lookup(data, PS(id), 0);

View File

@@ -492,18 +492,26 @@ static void php_session_initialize(TSRMLS_D) /* {{{ */
}
}
php_session_reset_id(TSRMLS_C);
PS(session_status) = php_session_active;
/* Set session ID for compatibility for older/3rd party save handlers */
if (!PS(use_strict_mode)) {
php_session_reset_id(TSRMLS_C);
PS(session_status) = php_session_active;
}
/* Read data */
php_session_track_init(TSRMLS_C);
if (PS(mod)->s_read(&PS(mod_data), PS(id), &val, &vallen TSRMLS_CC) == FAILURE) {
/* Some broken save handler implementation returns FAILURE for non-existent session ID */
/* It's better to rase error for this, but disabled error for better compatibility */
/* It's better to raise error for this, but disabled error for better compatibility */
/*
php_error_docref(NULL TSRMLS_CC, E_NOTICE, "Failed to read session data: %s (path: %s)", PS(mod)->s_name, PS(save_path));
*/
}
/* Set session ID if session read didn't activated session */
if (PS(use_strict_mode) && PS(session_status) != php_session_active) {
php_session_reset_id(TSRMLS_C);
PS(session_status) = php_session_active;
}
if (val) {
PHP_MD5_CTX context;