From acaf9c5227d75321def26ec5631def2fbb58feb2 Mon Sep 17 00:00:00 2001 From: Felipe Pena Date: Sat, 28 Jan 2012 01:24:54 +0000 Subject: [PATCH] - Fixed bug #60860 (session.save_handler=user without defined function core dumps) --- ext/session/mod_user.c | 7 +++++++ ext/session/tests/bug60860.phpt | 17 +++++++++++++++++ 2 files changed, 24 insertions(+) create mode 100644 ext/session/tests/bug60860.phpt diff --git a/ext/session/mod_user.c b/ext/session/mod_user.c index cf13e4b46ef..2ff5302f782 100644 --- a/ext/session/mod_user.c +++ b/ext/session/mod_user.c @@ -80,6 +80,13 @@ PS_OPEN_FUNC(user) { zval *args[2]; STDVARS; + + if (PSF(open) == NULL) { + php_error_docref(NULL TSRMLS_CC, E_WARNING, + "user session functions not defined"); + + return FAILURE; + } SESS_ZVAL_STRING((char*)save_path, args[0]); SESS_ZVAL_STRING((char*)session_name, args[1]); diff --git a/ext/session/tests/bug60860.phpt b/ext/session/tests/bug60860.phpt new file mode 100644 index 00000000000..12310205aa7 --- /dev/null +++ b/ext/session/tests/bug60860.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #60860 (session.save_handler=user without defined function core dumps) +--SKIPIF-- + +--INI-- +session.save_handler=user +--FILE-- + +--EXPECTF-- +Warning: session_start(): user session functions not defined in %s on line 3 + +Fatal error: session_start(): Failed to initialize storage module: user (path: ) in %s on line 3