You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by bv...@apache.org on 2023/05/21 15:56:50 UTC

[camel] branch main updated: avoid potential NPE

This is an automated email from the ASF dual-hosted git repository.

bvahdat pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new 9b63c3eedca avoid potential NPE
9b63c3eedca is described below

commit 9b63c3eedca7a18581f471cf75c17c4ea0aa1663
Author: Babak Vahdat <bv...@apache.org>
AuthorDate: Sun May 21 17:56:41 2023 +0200

    avoid potential NPE
---
 .../camel/component/jsonvalidator/JsonValidationException.java     | 3 ++-
 .../camel/component/jsonvalidator/JsonValidationExceptionTest.java | 7 +++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/components/camel-json-validator/src/main/java/org/apache/camel/component/jsonvalidator/JsonValidationException.java b/components/camel-json-validator/src/main/java/org/apache/camel/component/jsonvalidator/JsonValidationException.java
index 0284ac1a310..0856d8205d0 100644
--- a/components/camel-json-validator/src/main/java/org/apache/camel/component/jsonvalidator/JsonValidationException.java
+++ b/components/camel-json-validator/src/main/java/org/apache/camel/component/jsonvalidator/JsonValidationException.java
@@ -16,6 +16,7 @@
  */
 package org.apache.camel.component.jsonvalidator;
 
+import java.util.Collections;
 import java.util.Set;
 import java.util.stream.Collectors;
 
@@ -40,7 +41,7 @@ public class JsonValidationException extends ValidationException {
     public JsonValidationException(Exchange exchange, JsonSchema schema, Exception e) {
         super(e.getMessage(), exchange, e);
         this.schema = schema;
-        this.errors = null;
+        this.errors = Collections.EMPTY_SET;
     }
 
     public JsonSchema getSchema() {
diff --git a/components/camel-json-validator/src/test/java/org/apache/camel/component/jsonvalidator/JsonValidationExceptionTest.java b/components/camel-json-validator/src/test/java/org/apache/camel/component/jsonvalidator/JsonValidationExceptionTest.java
index aa1b81225cd..06d511e0ca8 100644
--- a/components/camel-json-validator/src/test/java/org/apache/camel/component/jsonvalidator/JsonValidationExceptionTest.java
+++ b/components/camel-json-validator/src/test/java/org/apache/camel/component/jsonvalidator/JsonValidationExceptionTest.java
@@ -17,6 +17,7 @@
 package org.apache.camel.component.jsonvalidator;
 
 import java.text.MessageFormat;
+import java.util.Collections;
 import java.util.LinkedHashSet;
 import java.util.Set;
 
@@ -24,6 +25,7 @@ import com.networknt.schema.ValidationMessage;
 import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
 
 public class JsonValidationExceptionTest {
 
@@ -37,6 +39,11 @@ public class JsonValidationExceptionTest {
                 new JsonValidationException(null, null, errors).getMessage());
     }
 
+    @Test
+    void testErrorsEmpty() {
+        assertSame(Collections.EMPTY_SET, new JsonValidationException(null, null, new Exception()).getErrors());
+    }
+
     private ValidationMessage createError(String msg) {
         return new ValidationMessage.Builder().format(new MessageFormat(msg)).build();
     }