Raise error if any key in a mapping is null (#3073)

* Null keys will raise an error
* Raise errors when a key in an object is None

Author: kddejong

src/cfnlint/decode/cfn_yaml.py

@@ -88,6 +88,19 @@ def construct_yaml_map(self, node):
             key = self.construct_object(key_node, False)
             value = self.construct_object(value_node, False)
 
+            if key is None:
+                raise CfnParseError(
+                    self.filename,
+                    [
+                        build_match(
+                            filename=self.filename,
+                            message=f"Null key {key_node.value!r} not supported (line {key_node.start_mark.line + 1})",
+                            line_number=key_node.start_mark.line,
+                            column_number=key_node.start_mark.column,
+                            key=key_node.value,
+                        ),
+                    ],
+                )
             for key_dup in mapping:
                 if key_dup == key:
                     if not matches:

test/unit/module/decode/test_decode.py

@@ -11,7 +11,9 @@
 import yaml
 from yaml.scanner import ScannerError
 
-import cfnlint.decode.decode  # pylint: disable=E0401
+import cfnlint.decode.cfn_json
+import cfnlint.decode.cfn_yaml
+import cfnlint.decode.decode
 
 
 class TestDecode(BaseTestCase):
@@ -125,3 +127,14 @@ def test_decode_yaml_error(self, mock_cfn_yaml):
         self.assertEqual(len(matches), 1)
         self.assertEqual(matches[0].rule.id, "E0000")
         self.assertEqual(matches[0].message, err_msg)
+
+    def test_decode_yaml_null_key(self):
+        err_msg = "Null key 'null' not supported (line 3)"
+        with self.assertRaises(cfnlint.decode.cfn_yaml.CfnParseError) as e:
+            cfnlint.decode.cfn_yaml.loads(
+                """
+                Parameters:
+                    null: test
+            """
+            )
+        self.assertEqual(str(e.exception), err_msg)