diff options
author | Tjerk Meesters <datibbaw@php.net> | 2014-08-23 09:18:02 +0800 |
---|---|---|
committer | Ferenc Kovacs <tyrael@php.net> | 2014-08-27 04:27:26 +0200 |
commit | e60d39581a8fb7e0d8cabd5c2d44d35e01d2969c (patch) | |
tree | 8b9ec9518c9a440fcf15afd4b5aaab58e6c7b6f1 | |
parent | 7ac17d0d794bc85a98bc6e4e1919c4a7660810ed (diff) | |
download | php-git-e60d39581a8fb7e0d8cabd5c2d44d35e01d2969c.tar.gz |
Fixed #67694: Regression in session_regenerate_id()
-rw-r--r-- | ext/session/session.c | 2 | ||||
-rw-r--r-- | ext/session/tests/bug67694.phpt | 41 |
2 files changed, 42 insertions, 1 deletions
diff --git a/ext/session/session.c b/ext/session/session.c index d1a7a073a5..d440e6fdd9 100644 --- a/ext/session/session.c +++ b/ext/session/session.c @@ -1994,7 +1994,7 @@ static PHP_FUNCTION(session_regenerate_id) RETURN_FALSE; } efree(PS(id)); - PS(id) = NULL; + memset(PS(session_data_hash),'\0', 16); } PS(id) = PS(mod)->s_create_sid(&PS(mod_data), NULL TSRMLS_CC); diff --git a/ext/session/tests/bug67694.phpt b/ext/session/tests/bug67694.phpt new file mode 100644 index 0000000000..08a296923f --- /dev/null +++ b/ext/session/tests/bug67694.phpt @@ -0,0 +1,41 @@ +--TEST-- +Bug #67694 Regression in session_regenerate_id() +--INI-- +session.save_handler = files +session.name=PHPSESSID +session.auto_start = 0 +session.use_cookies = 0 +session.cache_limiter = +date.timezone = UTC +--SKIPIF-- +<?php include('skipif.inc'); ?> +--FILE-- +<?php + +$data = date('r'); + +session_start(); +$id = session_id(); +$_SESSION['init'] = $data; + +session_write_close(); + +session_id($id); +session_start(); + +session_regenerate_id(false); + +$newid = session_id(); +var_dump($newid != $id); + +session_write_close(); + +unset($_SESSION); + +session_id($newid); +session_start(); + +var_dump(isset($_SESSION['init']) && $data == $_SESSION['init']); +--EXPECT-- +bool(true) +bool(true) |