Fix GH-8563

With memory streams if we get a NULL buffer we must not instantiate an empty line

Author: Girgias

ext/spl/spl_directory.c

@@ -1883,22 +1883,21 @@ static zend_result spl_filesystem_file_read_ex(spl_filesystem_object *intern, bo
 	}
 
 	if (!buf) {
-		intern->u.file.current_line = estrdup("");
-		intern->u.file.current_line_len = 0;
-	} else {
-		if (SPL_HAS_FLAG(intern->flags, SPL_FILE_OBJECT_DROP_NEW_LINE)) {
-			if (line_len > 0 && buf[line_len - 1] == '\n') {
+		return FAILURE;
+	}
+
+	if (SPL_HAS_FLAG(intern->flags, SPL_FILE_OBJECT_DROP_NEW_LINE)) {
+		if (line_len > 0 && buf[line_len - 1] == '\n') {
+			line_len--;
+			if (line_len > 0 && buf[line_len - 1] == '\r') {
 				line_len--;
-				if (line_len > 0 && buf[line_len - 1] == '\r') {
-					line_len--;
-				}
-				buf[line_len] = '\0';
 			}
+			buf[line_len] = '\0';
 		}
-
-		intern->u.file.current_line = buf;
-		intern->u.file.current_line_len = line_len;
 	}
+
+	intern->u.file.current_line = buf;
+	intern->u.file.current_line_len = line_len;
 	intern->u.file.current_line_num += line_add;
 
 	return SUCCESS;

ext/spl/tests/SplFileObject_fgetcsv_basic.phpt renamed to ext/spl/tests/SplFileObject/SplFileObject_fgetcsv_basic.phpt

@@ -13,6 +13,7 @@ fclose($fp);
 
 $fo = new SplFileObject('SplFileObject__fgetcsv1.csv');
 var_dump($fo->fgetcsv());
+var_dump($fo->fgetcsv());
 ?>
 --CLEAN--
 <?php
@@ -29,3 +30,4 @@ array(4) {
   [3]=>
   string(1) "5"
 }
+bool(false)

ext/spl/tests/SplFileObject_key_error001.phpt renamed to ext/spl/tests/SplFileObject/SplFileObject_key_error001.phpt

@@ -18,5 +18,5 @@ var_dump($s->key());
 var_dump($s->valid());
 ?>
 --EXPECT--
-int(14)
+int(12)
 bool(false)

ext/spl/tests/SplFileObject_key_error002.phpt renamed to ext/spl/tests/SplFileObject/SplFileObject_key_error002.phpt

@@ -18,5 +18,5 @@ var_dump($s->key());
 var_dump($s->valid());
 ?>
 --EXPECT--
-int(13)
+int(12)
 bool(false)

ext/spl/tests/SplFileObject/gh8563.phpt

@@ -0,0 +1,48 @@
+--TEST--
+Bug GH-8563: Different results for seek() on SplFileObject and SplTempFileObject
+--FILE--
+<?php
+
+const LINES = 5;
+const INDEX = 10;
+
+$file_01 = new SplFileObject(__DIR__ . '/gh8563.txt', 'w+');
+$file_02 = new SplTempFileObject(-1);
+$file_03 = new SplTempFileObject(0);
+$file_04 = new SplTempFileObject();
+
+// write to files
+for ($i = 0; $i < LINES; $i++) {
+    $file_01->fwrite("line {$i}" . PHP_EOL);
+    $file_02->fwrite("line {$i}" . PHP_EOL);
+    $file_03->fwrite("line {$i}" . PHP_EOL);
+    $file_04->fwrite("line {$i}" . PHP_EOL);
+}
+
+// reset
+$file_01->rewind();
+$file_02->rewind();
+$file_03->rewind();
+$file_04->rewind();
+
+// seek
+$file_01->seek(INDEX);
+$file_02->seek(INDEX);
+$file_03->seek(INDEX);
+$file_04->seek(INDEX);
+
+// show results
+echo 'file_01: ' . $file_01->key(), PHP_EOL;
+echo 'file_02: ' . $file_02->key(), PHP_EOL;
+echo 'file_03: ' . $file_03->key(), PHP_EOL;
+echo 'file_04: ' . $file_04->key(), PHP_EOL;
+?>
+--CLEAN--
+<?php
+unlink(__DIR__ . '/gh8563.txt');
+?>
+--EXPECT--
+file_01: 4
+file_02: 4
+file_03: 4
+file_04: 4

ext/spl/tests/fileobject_001.phpt

@@ -70,7 +70,7 @@ string(1) "4"
 int(5)
 string(1) "5"
 int(6)
-string(0) ""
+bool(false)
 ===B===
 int(0)
 string(1) "0"