[EMAIL PROTECTED] writes:

>+    protected boolean validateToolInfo(ToolInfo info)
>+    {
[...]
>+        if (info.getClassname() == null)
>+        {
>+            LOG.error("Tool " + info.getKey() + " has no Class definition!");
>+            return false;
>+        }

If you add that part without the DataInfo changes, then validateToolInfo() will 
nuke with a NPE when you add

<data>
  <key>test</key>
</data>

to the toolbox. It passes the first two checks (object is not null and
key is not empty) and then calls getClassname() which is
data.getClass().getName() with data == null. Boom.

        Best regards
                Henning


-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
[EMAIL PROTECTED]        +49 9131 50 654 0   http://www.intermeta.de/

RedHat Certified Engineer -- Jakarta Turbine Development  -- hero for hire
   Linux, Java, perl, Solaris -- Consulting, Training, Development

                      4 - 8 - 15 - 16 - 23 - 42

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to