check nullValue instead of address. Since nullValue can only make address null

martijnvg · martijnvg · commit 77c30b14f664 · 2025-02-24T16:44:19.000+01:00
diff --git a/server/src/main/java/org/elasticsearch/index/mapper/IpFieldMapper.java b/server/src/main/java/org/elasticsearch/index/mapper/IpFieldMapper.java
@@ -582,11 +582,11 @@ protected void parseCreateField(DocumentParserContext context) throws IOExceptio
                 throw e;
             }
         }
-        if (address != null) {
+        if (nullValue != null) {
             indexValue(context, address);
         }
         if (offsetsFieldName != null && context.isImmediateParentAnArray() && context.canAddIgnoredField()) {
-            if (address != null) {
+            if (nullValue != null) {
                 BytesRef sortableValue = new BytesRef(InetAddressPoint.encode(address));
                 context.getOffSetContext().recordOffset(offsetsFieldName, sortableValue);
             } else {

Original file line number	Diff line number	Diff line change
`@@ -582,11 +582,11 @@ protected void parseCreateField(DocumentParserContext context) throws IOExceptio`
`582`	`582`	`throw e;`
`583`	`583`	`}`
`584`	`584`	`}`
`585`		`- if (address != null) {`
	`585`	`+ if (nullValue != null) {`
`586`	`586`	`indexValue(context, address);`
`587`	`587`	`}`
`588`	`588`	`if (offsetsFieldName != null && context.isImmediateParentAnArray() && context.canAddIgnoredField()) {`
`589`		`- if (address != null) {`
	`589`	`+ if (nullValue != null) {`
`590`	`590`	`BytesRef sortableValue = new BytesRef(InetAddressPoint.encode(address));`
`591`	`591`	`context.getOffSetContext().recordOffset(offsetsFieldName, sortableValue);`
`592`	`592`	`} else {`