]> granicus.if.org Git - php/commitdiff
Update comment, incorrect since 224aaf94
authorSjonHortensius <SjonHortensius@users.noreply.github.com>
Thu, 19 Jan 2017 10:41:02 +0000 (11:41 +0100)
committerJoe Watkins <krakjoe@php.net>
Thu, 19 Jan 2017 10:48:54 +0000 (10:48 +0000)
In 224aaf94; the warning was enabled; making the comment above incorrect. I've updated the comment to reflect the current code

ext/session/session.c

index c06c5a3a0db80c6039761c6a2e869c62d5620b7d..d672b0c2831f571e5b566d784b1c7c0a09d4d644 100644 (file)
@@ -425,8 +425,7 @@ static void php_session_initialize(void) /* {{{ */
        php_session_track_init();
        if (PS(mod)->s_read(&PS(mod_data), PS(id), &val, PS(gc_maxlifetime)) == FAILURE) {
                php_session_abort();
-               /* Some broken save handler implementation returns FAILURE for non-existent session ID */
-               /* It's better to raise error for this, but disabled error for better compatibility */
+               /* FYI: Some broken save handlers return FAILURE for non-existent session ID, this is incorrect */
                php_error_docref(NULL, E_WARNING, "Failed to read session data: %s (path: %s)", PS(mod)->s_name, PS(save_path));
                return;
        }