[Yaml] Fix error handling in parser

chalasr · chalasr · commit 71cec8203f77 · 2017-03-22T11:47:48.000+01:00
diff --git a/src/Symfony/Component/Yaml/Parser.php b/src/Symfony/Component/Yaml/Parser.php
@@ -341,7 +341,7 @@ public function parse($value, $flags = 0)
                         try {
                             $parsedLine = Inline::parse($line, $flags, $this->refs);
 
-                            if (!is_string($value)) {
+                            if (!is_string($parsedLine)) {
                                 $parseError = true;
                                 break;
                             }
diff --git a/src/Symfony/Component/Yaml/Tests/ParserTest.php b/src/Symfony/Component/Yaml/Tests/ParserTest.php
@@ -1687,6 +1687,21 @@ public function testComplexMappingNestedInSequenceThrowsParseException()
         $this->parser->parse($yaml);
     }
 
+    /**
+     * @expectedException        \Symfony\Component\Yaml\Exception\ParseException
+     * @expectedExceptionMessage Unable to parse at line 1 (near "[parameters]").
+     */
+    public function testParsingIniThrowsException()
+    {
+        $ini = <<<INI
+[parameters]
+  foo = bar
+  bar = %foo%
+INI;
+
+        $this->parser->parse($ini);
+    }
+
     private function loadTestsFromFixtureFiles($testsFile)
     {
         $parser = new Parser();

Original file line number	Diff line number	Diff line change
`@@ -341,7 +341,7 @@ public function parse($value, $flags = 0)`
`341`	`341`	`try {`
`342`	`342`	`$parsedLine = Inline::parse($line, $flags, $this->refs);`
`343`	`343`
`344`		`- if (!is_string($value)) {`
	`344`	`+ if (!is_string($parsedLine)) {`
`345`	`345`	`$parseError = true;`
`346`	`346`	`break;`
`347`	`347`	`}`