Skip to content

Commit 7d0f320

Browse files
committed
Add null checks to config setters
1 parent e42706e commit 7d0f320

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

components/org.wso2.carbon.identity.api.server.common/src/main/java/org/wso2/carbon/identity/api/server/common/file/FileSerializationConfig.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,9 @@ public DefaultFormat getSerializeDefault() {
8383

8484
public void setSerializeDefault(DefaultFormat serializeDefault) {
8585

86+
if (serializeDefault == null) {
87+
throw new IllegalArgumentException("serializeDefault cannot be null");
88+
}
8689
this.serializeDefault = serializeDefault;
8790
}
8891

@@ -92,6 +95,9 @@ public DefaultFormat getDeserializeDefault() {
9295

9396
public void setDeserializeDefault(DefaultFormat deserializeDefault) {
9497

98+
if (deserializeDefault == null) {
99+
throw new IllegalArgumentException("deserializeDefault cannot be null");
100+
}
95101
this.deserializeDefault = deserializeDefault;
96102
}
97103
}

components/org.wso2.carbon.identity.api.server.common/src/main/java/org/wso2/carbon/identity/api/server/common/file/YamlConfig.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,9 @@ public List<String> getAdditionalTrustedClassNames() {
8484
public void setAdditionalTrustedClasses(Class<?>... classes) {
8585

8686
this.additionalTrustedClassNames = new ArrayList<>();
87+
if (classes == null) {
88+
return;
89+
}
8790
for (Class<?> clazz : classes) {
8891
this.additionalTrustedClassNames.add(clazz.getName());
8992
}

0 commit comments

Comments
 (0)