]> granicus.if.org Git - php/commitdiff
Merge branch 'PHP-7.1'
authorJoe Watkins <krakjoe@php.net>
Thu, 19 Jan 2017 10:49:09 +0000 (10:49 +0000)
committerJoe Watkins <krakjoe@php.net>
Thu, 19 Jan 2017 10:49:09 +0000 (10:49 +0000)
* PHP-7.1:
  Update comment, incorrect since 224aaf94

1  2 
ext/session/session.c

index 6323020f68e643746755c9f903ea10cc084f4591,d672b0c2831f571e5b566d784b1c7c0a09d4d644..02f8d8d88166e6deb80aa48dc9b06c9fdfc67bb5
@@@ -435,10 -425,9 +435,9 @@@ static int 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;
 +              return FAILURE;
        }
  
        /* GC must be done after read */