Bugfix: NullPointerException if no PEPS is configured

1 files changed, 31 insertions, 28 deletions

diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/moaconfig/StorkConfigValidator.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/moaconfig/StorkConfigValidator.java
index 318b3b3e7..d8b0ac6de 100644
--- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/moaconfig/StorkConfigValidator.java
+++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/moaconfig/StorkConfigValidator.java
@@ -22,37 +22,40 @@ public class StorkConfigValidator {
 		log.debug("Validate general STORK configuration");
 
 		// check peps list
-		for(CPEPS current : form.getCpepslist()) {
-			// check country code
-			String check = current.getCountryCode();
-			if (MiscUtil.isNotEmpty(check)) {
-				if (ValidationHelper.containsPotentialCSSCharacter(check, false)) {
-					log.warn("CPEPS config countrycode contains potentail XSS characters: " + check);
-					errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.cc",
-							new Object[] {ValidationHelper.getPotentialCSSCharacter(false)} ));
-				}
-				if(!check.toLowerCase().matches("^[a-z][a-z]$")) {
-						log.warn("CPEPS config countrycode does not comply to ISO 3166-2 : " + check);
+		
+		if (form.getCpepslist() != null) {
+			for(CPEPS current : form.getCpepslist()) {
+				// check country code
+				String check = current.getCountryCode();
+				if (MiscUtil.isNotEmpty(check)) {
+					if (ValidationHelper.containsPotentialCSSCharacter(check, false)) {
+						log.warn("CPEPS config countrycode contains potentail XSS characters: " + check);
 						errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.cc",
-								new Object[] {check} ));
+								new Object[] {ValidationHelper.getPotentialCSSCharacter(false)} ));
+					}
+					if(!check.toLowerCase().matches("^[a-z][a-z]$")) {
+							log.warn("CPEPS config countrycode does not comply to ISO 3166-2 : " + check);
+							errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.cc",
+									new Object[] {check} ));
+					}
+				} else {
+					log.warn("CPEPS config countrycode is empty : " + check);
+					errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.empty",
+							new Object[] {check} ));
 				}
-			} else {
-				log.warn("CPEPS config countrycode is empty : " + check);
-				errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.empty",
-						new Object[] {check} ));
-			}
-
-			// check url
-			check = current.getURL();
-			if (MiscUtil.isNotEmpty(check)) {
-				if (!ValidationHelper.validateURL(check)) {
-					log.info("CPEPS config URL is invalid : " + check);
-					errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.url"));
+	
+				// check url
+				check = current.getURL();
+				if (MiscUtil.isNotEmpty(check)) {
+					if (!ValidationHelper.validateURL(check)) {
+						log.info("CPEPS config URL is invalid : " + check);
+						errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.url"));
+					}
+				} else {
+					log.warn("CPEPS config url is empty : " + check);
+					errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.empty",
+							new Object[] {check} ));
 				}
-			} else {
-				log.warn("CPEPS config url is empty : " + check);
-				errors.add(LanguageHelper.getErrorString("validation.stork.cpeps.empty",
-						new Object[] {check} ));
 			}
 		}