Fix NPE when no Content-Disposition header is present.

simple/simple-http/src/main/java/org/simpleframework/http/message/BufferPart.java

@@ -69,7 +69,11 @@ public BufferPart(Segment segment, Buffer buffer) {
     * @return this returns true if the associated part is a file
     */
    public boolean isFile() {
-      return getDisposition().isFile();
+      ContentDisposition disposition = getDisposition();
+      if (disposition == null) {
+         return false;
+      }
+      return disposition.isFile();
    }
 
    /**
@@ -80,7 +84,11 @@ public boolean isFile() {
     * @return this returns the name of the associated part
     */
    public String getName() {
-      return getDisposition().getName();
+      ContentDisposition disposition = getDisposition();
+      if (disposition == null) {
+         return null;
+      }
+      return disposition.getName();
    }
 
    /**
@@ -91,7 +99,11 @@ public String getName() {
     * @return this returns the file name of the associated part
     */
    public String getFileName() {
-      return getDisposition().getFileName();
+      ContentDisposition disposition = getDisposition();
+      if (disposition == null) {
+         return null;
+      }
+      return disposition.getFileName();
    }
 
    /**

simple/simple-http/src/test/java/org/simpleframework/http/message/BufferPartTest.java

@@ -0,0 +1,14 @@
+package org.simpleframework.http.message;
+
+import junit.framework.TestCase;
+import org.simpleframework.common.buffer.ArrayAllocator;
+
+public class BufferPartTest extends TestCase {
+
+    public void testNoDispositionContextNPE() throws Exception {
+        BufferPart bufferPart = new BufferPart(new MockSegment(), new ArrayAllocator().allocate());
+        assertNull(bufferPart.getFileName());
+        assertNull(bufferPart.getName());
+        assertFalse(bufferPart.isFile());
+    }
+}