[PMakowski] squizlabs#2.5.1.x-dev: fix unfinite loop if CLI argument …

…is incorrectly quoted
Astinus-Eberhard · Mar 17, 2016 · 3651599 · 3651599
1 parent 7394ee4
commit 3651599
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/CodeSniffer/CLI.php b/CodeSniffer/CLI.php
@@ -726,8 +726,8 @@ public function processLongArgument($arg, $pos)
             } else if (substr($arg, 0, 12) === 'editor-path=') {
                 $value = substr($arg, 12);
                 if (1 === preg_match('/^[\'\"]/', $value, $matches)) {
-                    while (0 === preg_match("/[{$matches[0]}]$/", $value)) {
-                        $value .= ' '.$this->_cliArgs[++$pos];
+                    while ((0 === preg_match("/[{$matches[0]}]$/", $value)) && (isset($this->_cliArgs[++$pos]) === true)) {
+                        $value .= ' '.$this->_cliArgs[$pos];
                         $this->_cliArgs[$pos] = '';
                     }