Changeset 7300 in josm


Ignore:
Timestamp:
2014-07-09T22:30:40+02:00 (4 years ago)
Author:
Don-vip
Message:

fix #10251 - NPE with malformed validator rule (invalid fixAdd/fixRemove)

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java

    r7298 r7300  
    216216                    } else if ("fixAdd".equals(ai.key)) {
    217217                        final PrimitiveToTag toTag = PrimitiveToTag.ofMapCSSObject(ai.val, false);
    218                         check.change.add(toTag);
     218                        if (toTag != null) {
     219                            check.change.add(toTag);
     220                        } else {
     221                            Main.warn("Invalid value for "+ai.key+": "+ai.val);
     222                        }
    219223                    } else if ("fixRemove".equals(ai.key)) {
    220224                        CheckParameterUtil.ensureThat(!(ai.val instanceof String) || !(val != null && val.contains("=")),
    221225                                "Unexpected '='. Please only specify the key to remove!");
    222226                        final PrimitiveToTag toTag = PrimitiveToTag.ofMapCSSObject(ai.val, true);
    223                         check.change.add(toTag);
     227                        if (toTag != null) {
     228                            check.change.add(toTag);
     229                        } else {
     230                            Main.warn("Invalid value for "+ai.key+": "+ai.val);
     231                        }
    224232                    } else if ("fixChangeKey".equals(ai.key) && val != null) {
    225233                        CheckParameterUtil.ensureThat(val.contains("=>"), "Separate old from new key by '=>'!");
Note: See TracChangeset for help on using the changeset viewer.