BadFunctions/EasyRFI: bug fix - fix detecting of start/end of the statement [2]

PHPCS can be run from within an IDE during live coding. Similarly PHPCS can be run over files containing parse errors.

With that in mind, it is best practice to bow out in those cases.
Parse error detection should catch those errors. That is not the responsibility of this sniff.

Author: jrfnl

Security/Sniffs/BadFunctions/EasyRFISniff.php

@@ -39,6 +39,10 @@ public function register() {
 	*/
 	public function process(File $phpcsFile, $stackPtr) {
 		$closer = $phpcsFile->findNext(T_SEMICOLON, ($stackPtr + 1));
+		if ($closer === false) {
+			// Live coding or parse error.
+			return;
+		}
 
 		$utils  = \PHPCS_SecurityAudit\Security\Sniffs\UtilsFactory::getInstance();
 		$tokens = $phpcsFile->getTokens();

Security/Tests/BadFunctions/EasyRFIUnitTest.0.inc

@@ -15,3 +15,6 @@ include arg(2) . drupal_get_query_parameters()['param'];
 
 // Prevent false positives on safe $_SERVER variables.
 include $_SERVER['DOCUMENT_ROOT'] . '/filename.php';
+
+// Intentional parse error. This should be the last test in the file.
+require_once

Security/Tests/BadFunctions/EasyRFIUnitTest.1.inc

@@ -15,3 +15,6 @@ include arg(2) . drupal_get_query_parameters()['param']; // Warning x 2.
 
 // Prevent false positives on safe $_SERVER variables.
 include $_SERVER['DOCUMENT_ROOT'] . '/filename.php'; // Error.
+
+// Intentional parse error. This should be the last test in the file.
+require $_GET['path']

Security/Tests/BadFunctions/EasyRFIUnitTest.Drupal7.1.inc

@@ -12,3 +12,6 @@ include getenv('PATHTOFILE'); // Error.
 // Drupal 7.
 include ( 'path/to/' . $form['filename'] ); // Error.
 include arg(2) . drupal_get_query_parameters()['param']; // Error x 2.
+
+// Intentional parse error. This should be the last test in the file.
+include_once ( $_GET['path']