#9444 throw StrictDynamicMappingException exception if dynamic is 'strict' and undeclared field value is NULL, test for this

Fixes #9445

src/main/java/org/elasticsearch/index/mapper/object/ObjectMapper.java

@@ -587,6 +587,10 @@ public class ObjectMapper implements Mapper, AllFieldMapper.IncludeInAll {
                 }
             }
             mapper.parse(context);
+        } else {
+            if (dynamic == Dynamic.STRICT) {
+                throw new StrictDynamicMappingException(fullPath, lastFieldName);
+            }
         }
     }
 

src/test/java/org/elasticsearch/index/mapper/dynamic/DynamicMappingTests.java

@@ -98,6 +98,17 @@ public class DynamicMappingTests extends ElasticsearchSingleNodeTest {
         } catch (StrictDynamicMappingException e) {
             // all is well
         }
+
+        try {
+            defaultMapper.parse("type", "1", XContentFactory.jsonBuilder()
+                    .startObject()
+                    .field("field1", "value1")
+                    .field("field2", (String) null)
+                    .bytes());
+            fail();
+        } catch (StrictDynamicMappingException e) {
+            // all is well
+        }
     }
 
     @Test