diff options
| author | Thomas Lenz <tlenz@iaik.tugraz.at> | 2014-03-14 12:52:35 +0100 | 
|---|---|---|
| committer | Thomas Lenz <tlenz@iaik.tugraz.at> | 2014-03-14 12:52:35 +0100 | 
| commit | 191bd90971b488bff16a5e08d0c1b2306da5876f (patch) | |
| tree | a70dd32de00ec28118d2499a74e2802c9022dd9c /id/server/idserverlib | |
| parent | 6c9e452d5ab984c1ef221c4a2e746454aa41e014 (diff) | |
| download | moa-id-spss-191bd90971b488bff16a5e08d0c1b2306da5876f.tar.gz moa-id-spss-191bd90971b488bff16a5e08d0c1b2306da5876f.tar.bz2 moa-id-spss-191bd90971b488bff16a5e08d0c1b2306da5876f.zip | |
bugfix possible NullPointerException in STORK config
Diffstat (limited to 'id/server/idserverlib')
| -rw-r--r-- | id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/config/stork/STORKConfig.java | 36 | 
1 files changed, 19 insertions, 17 deletions
| diff --git a/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/config/stork/STORKConfig.java b/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/config/stork/STORKConfig.java index ff481b825..b3a4cca78 100644 --- a/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/config/stork/STORKConfig.java +++ b/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/config/stork/STORKConfig.java @@ -58,30 +58,32 @@ public class STORKConfig {  		this.basedirectory = basedirectory;
  		this.props = props;
 -		//create CPEPS map
 -		List<at.gv.egovernment.moa.id.commons.db.dao.config.CPEPS> cpeps = stork.getCPEPS();
 -		
  		cpepsMap = new HashMap<String, CPEPS>();
 +		attr = new ArrayList<StorkAttribute>();
 +		
 +		if (stork != null) {
 +			//create CPEPS map
 +			List<at.gv.egovernment.moa.id.commons.db.dao.config.CPEPS> cpeps = stork.getCPEPS();
 +		
 +			if (cpeps != null) {
 +				for(at.gv.egovernment.moa.id.commons.db.dao.config.CPEPS cpep : cpeps) {
 -		if (cpeps != null) {
 -			for(at.gv.egovernment.moa.id.commons.db.dao.config.CPEPS cpep : cpeps) {
 -
 -				try {
 -					CPEPS moacpep = new CPEPS(cpep.getCountryCode(), new URL(cpep.getURL()));
 +					try {
 +						CPEPS moacpep = new CPEPS(cpep.getCountryCode(), new URL(cpep.getURL()));
 -					cpepsMap.put(cpep.getCountryCode(), moacpep);
 +						cpepsMap.put(cpep.getCountryCode(), moacpep);
 -				} catch (MalformedURLException e) {
 -					Logger.warn("Error in MOA-ID Configuration. CPEP entry for country "
 -							+ cpep.getCountryCode() + " has an invalid URL and is ignored.");
 +					} catch (MalformedURLException e) {
 +						Logger.warn("Error in MOA-ID Configuration. CPEP entry for country "
 +								+ cpep.getCountryCode() + " has an invalid URL and is ignored.");
 +					}	
  				}
  			}
 -		}
 -		attr = new ArrayList<StorkAttribute>();
 -		if (stork.getAttributes() != null) {
 -			for(StorkAttribute current : stork.getAttributes()) {
 -				attr.add(current);
 +			if (stork.getAttributes() != null) {
 +				for(StorkAttribute current : stork.getAttributes()) {
 +					attr.add(current);
 +				}
  			}
  		}
 | 
