svn commit: r724005 - /webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/attachments/Attachments.java

[veithen]

Author: veithen
Date: Sat Dec  6 08:41:02 2008
New Revision: 724005

URL: http://svn.apache.org/viewvc?rev=724005&view=rev
Log:
WSCOMMONS-386: Throw an exception if the "boundary" parameter is not found. 
Indeed, it makes no sense to let the MIME parser look for "--null".

Modified:
    
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/attachments/Attachments.java

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/attachments/Attachments.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/attachments/Attachments.java?rev=724005&r1=724004&r2=724005&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/attachments/Attachments.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/attachments/Attachments.java
 Sat Dec  6 08:41:02 2008
@@ -191,8 +191,11 @@
             if(encoding == null || encoding.length()==0){
                 encoding = "UTF-8";
             }
-            this.boundary = ("--" + contentType.getParameter("boundary"))
-                    .getBytes(encoding);
+            String boundaryParam = contentType.getParameter("boundary");
+            if (boundaryParam == null) {
+                throw new OMException("Content-type has no 'boundary' 
parameter");
+            }
+            this.boundary = ("--" + boundaryParam).getBytes(encoding);
             if (log.isDebugEnabled()) {
                 log.debug("boundary=" + new String(this.boundary));
             }