mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-01 22:25:25 +02:00
Code Issues Wiki Activity

[fix] Bug 283844: NPE on restoring property set if persistent data is corrupted

Browse source
This commit is contained in:
Uwe Stieber 2009-07-17 14:32:10 +00:00
parent 5461401e09
commit c0f678f895
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/persistence/dom/RSEDOMImporter.java
View file

@ -26,6 +26,7 @@
* David McKnight (IBM) - [245198] [dstore] ServerLauncherProperties not restored
* David McKnight (IBM) - [267052] need to be able to create subsystems-after-the-fact
* David McKnight (IBM) - [271243] [files] Switching service type brings up TWO file subsystems after restart
* Uwe Stieber (Wind River) - [283844] NPE on restoring property set if persistent data is corrupted
********************************************************************************/
package org.eclipse.rse.internal.persistence.dom;
@ -559,8 +560,13 @@ public class RSEDOMImporter {
set.setDescription(attribute.getValue());
} else {
String typeStr = attribute.getType();
IPropertyType type = PropertyType.fromString(typeStr);
set.addProperty(attribute.getKey(), attribute.getValue(), type);
// We keep getting reports throwing NPE in PropertyType.fromString(...).
// If the data is corrupted and the type cannot be determined, it is better
// to just drop the single property than the whole property set
if (typeStr != null) {
IPropertyType type = PropertyType.fromString(typeStr);
set.addProperty(attribute.getKey(), attribute.getValue(), type);
}
}
}
// properties are now stored as children, get those next