You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ode.apache.org by bo...@apache.org on 2007/04/24 16:16:35 UTC

svn commit: r531947 - /incubator/ode/trunk/axis2/src/main/java/org/apache/ode/axis2/service/DeploymentWebService.java

Author: boisvert
Date: Tue Apr 24 07:16:34 2007
New Revision: 531947

URL: http://svn.apache.org/viewvc?view=rev&rev=531947
Log:
Better error reporting

Modified:
    incubator/ode/trunk/axis2/src/main/java/org/apache/ode/axis2/service/DeploymentWebService.java

Modified: incubator/ode/trunk/axis2/src/main/java/org/apache/ode/axis2/service/DeploymentWebService.java
URL: http://svn.apache.org/viewvc/incubator/ode/trunk/axis2/src/main/java/org/apache/ode/axis2/service/DeploymentWebService.java?view=diff&rev=531947&r1=531946&r2=531947
==============================================================================
--- incubator/ode/trunk/axis2/src/main/java/org/apache/ode/axis2/service/DeploymentWebService.java (original)
+++ incubator/ode/trunk/axis2/src/main/java/org/apache/ode/axis2/service/DeploymentWebService.java Tue Apr 24 07:16:34 2007
@@ -119,6 +119,9 @@
                         throw new OdeFault("Your message should contain a part named 'package' with a zip element");
 
                     OMText binaryNode = (OMText) zip.getFirstOMChild();
+                    if (binaryNode == null) {
+                        throw new OdeFault("Empty binary node under <zip> element");
+                    }
                     binaryNode.setOptimize(true);
                     try {
                         // We're going to create a directory under the deployment root and put