Index: /trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java	(revision 8422)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java	(revision 8423)
@@ -1014,5 +1014,7 @@
             }
         }
-        regactions.put(toolbar, action);
+        if (toolbar != null) {
+            regactions.put(toolbar, action);
+        }
         return action;
     }
Index: /trunk/test/unit/org/openstreetmap/josm/gui/preferences/ToolbarPreferencesTest.java
===================================================================
--- /trunk/test/unit/org/openstreetmap/josm/gui/preferences/ToolbarPreferencesTest.java	(revision 8422)
+++ /trunk/test/unit/org/openstreetmap/josm/gui/preferences/ToolbarPreferencesTest.java	(revision 8423)
@@ -14,6 +14,6 @@
 import org.junit.Test;
 import org.openstreetmap.josm.actions.ActionParameter;
+import org.openstreetmap.josm.actions.ActionParameter.StringActionParameter;
 import org.openstreetmap.josm.actions.ParameterizedAction;
-import org.openstreetmap.josm.actions.ActionParameter.StringActionParameter;
 import org.openstreetmap.josm.gui.preferences.ToolbarPreferences.ActionDefinition;
 import org.openstreetmap.josm.gui.preferences.ToolbarPreferences.ActionParser;
@@ -61,6 +61,6 @@
         checkAction(parser.loadAction("action(param1=value1,"), "param1", "value1");
         checkAction(parser.loadAction("action(param1=value1"), "param1", "value1");
-        checkAction(parser.loadAction("action(param1="), "param1", "");
-        checkAction(parser.loadAction("action(param1"), "param1", "");
+        checkAction(parser.loadAction("action(param1="));
+        checkAction(parser.loadAction("action(param1"));
         checkAction(parser.loadAction("action("));
         checkAction(parser.loadAction("action"));