Fix undefined array key AlterOperation

Signed-off-by: Maurício Meneghini Fauth <[email protected]>

Author: MauricioFauth

src/Statements/AlterStatement.php

@@ -67,7 +67,14 @@ class AlterStatement extends Statement
     public function parse(Parser $parser, TokensList $list)
     {
         ++$list->idx; // Skipping `ALTER`.
-        $this->options = OptionsArray::parse($parser, $list, static::$OPTIONS);
+        $parsedOptions = OptionsArray::parse($parser, $list, static::$OPTIONS);
+        if ($parsedOptions->isEmpty()) {
+            $parser->error('Unrecognized alter operation.', $list->tokens[$list->idx]);
+
+            return;
+        }
+
+        $this->options = $parsedOptions;
         ++$list->idx;
 
         // Parsing affected table.

tests/Misc/BugsTest.php

@@ -25,6 +25,7 @@ public function bugProvider(): array
             ['bugs/fuzz1'],
             ['bugs/fuzz2'],
             ['bugs/fuzz3'],
+            ['bugs/fuzz4'],
             ['bugs/gh9'],
             ['bugs/gh14'],
             ['bugs/gh16'],

tests/data/bugs/fuzz1.out

@@ -61,38 +61,11 @@
         "statements": [
             {
                 "@type": "PhpMyAdmin\\SqlParser\\Statements\\AlterStatement",
-                "table": {
-                    "@type": "PhpMyAdmin\\SqlParser\\Components\\Expression",
-                    "database": null,
-                    "table": null,
-                    "column": null,
-                    "expr": ".",
-                    "alias": null,
-                    "function": null,
-                    "subquery": null
-                },
-                "altered": [
-                    {
-                        "@type": "PhpMyAdmin\\SqlParser\\Components\\AlterOperation",
-                        "options": {
-                            "@type": "PhpMyAdmin\\SqlParser\\Components\\OptionsArray",
-                            "options": []
-                        },
-                        "field": null,
-                        "partitions": null,
-                        "unknown": [
-                            {
-                                "@type": "@4"
-                            }
-                        ]
-                    }
-                ],
-                "options": {
-                    "@type": "PhpMyAdmin\\SqlParser\\Components\\OptionsArray",
-                    "options": []
-                },
+                "table": null,
+                "altered": [],
+                "options": null,
                 "first": 0,
-                "last": 3
+                "last": 0
             }
         ],
         "brackets": 0,
@@ -105,7 +78,14 @@
             [
                 "Unrecognized alter operation.",
                 {
-                    "@type": "@5"
+                    "@type": "@2"
+                },
+                0
+            ],
+            [
+                "Unexpected beginning of statement.",
+                {
+                    "@type": "@4"
                 },
                 0
             ]

tests/data/bugs/fuzz4.in

@@ -0,0 +1 @@
+ALTeR=SET

tests/data/bugs/fuzz4.out

@@ -0,0 +1,98 @@
+{
+    "query": "ALTeR=SET",
+    "lexer": {
+        "@type": "PhpMyAdmin\\SqlParser\\Lexer",
+        "str": "ALTeR=SET",
+        "len": 9,
+        "last": 9,
+        "list": {
+            "@type": "PhpMyAdmin\\SqlParser\\TokensList",
+            "tokens": [
+                {
+                    "@type": "PhpMyAdmin\\SqlParser\\Token",
+                    "token": "ALTeR",
+                    "value": "ALTER",
+                    "keyword": "ALTER",
+                    "type": 1,
+                    "flags": 3,
+                    "position": 0
+                },
+                {
+                    "@type": "PhpMyAdmin\\SqlParser\\Token",
+                    "token": "=",
+                    "value": "=",
+                    "keyword": null,
+                    "type": 2,
+                    "flags": 2,
+                    "position": 5
+                },
+                {
+                    "@type": "PhpMyAdmin\\SqlParser\\Token",
+                    "token": "SET",
+                    "value": "SET",
+                    "keyword": "SET",
+                    "type": 1,
+                    "flags": 11,
+                    "position": 6
+                },
+                {
+                    "@type": "PhpMyAdmin\\SqlParser\\Token",
+                    "token": null,
+                    "value": null,
+                    "keyword": null,
+                    "type": 9,
+                    "flags": 0,
+                    "position": null
+                }
+            ],
+            "count": 4,
+            "idx": 4
+        },
+        "delimiter": ";",
+        "delimiterLen": 1,
+        "strict": false,
+        "errors": []
+    },
+    "parser": {
+        "@type": "PhpMyAdmin\\SqlParser\\Parser",
+        "list": {
+            "@type": "@1"
+        },
+        "statements": [
+            {
+                "@type": "PhpMyAdmin\\SqlParser\\Statements\\AlterStatement",
+                "table": null,
+                "altered": [],
+                "options": null,
+                "first": 0,
+                "last": 0
+            },
+            {
+                "@type": "PhpMyAdmin\\SqlParser\\Statements\\SetStatement",
+                "options": {
+                    "@type": "PhpMyAdmin\\SqlParser\\Components\\OptionsArray",
+                    "options": []
+                },
+                "end_options": null,
+                "set": [],
+                "first": 1,
+                "last": 2
+            }
+        ],
+        "brackets": 0,
+        "strict": false,
+        "errors": []
+    },
+    "errors": {
+        "lexer": [],
+        "parser": [
+            [
+                "Unrecognized alter operation.",
+                {
+                    "@type": "@2"
+                },
+                0
+            ]
+        ]
+    }
+}