Should store the disposition-type without Content-Disposition paramet…

…ers.
thunderbird · Apr 3, 2011 · 37303d6 · jca02266 · Apr 9, 2011 · jca02266
1 parent 39e3aec
commit 37303d6
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/com/fsck/k9/mail/store/LocalStore.java b/src/com/fsck/k9/mail/store/LocalStore.java
@@ -2368,6 +2368,14 @@ public Void doDbWork(final SQLiteDatabase db) throws WrappedException, Unavailab
                             String contentId = MimeUtility.getHeaderParameter(attachment.getContentId(), null);
 
                             String contentDisposition = MimeUtility.unfoldAndDecode(attachment.getDisposition());
+                            String dispositionType = contentDisposition;
+
+                            int pos = dispositionType.indexOf(';');
+                            if (pos != -1) {
+                                // extract the disposition-type, "attachment", "inline" or extension-token (see the RFC 2183)
+                                dispositionType = dispositionType.substring(0, pos);
+                            }
+
                             if (name == null && contentDisposition != null) {
                                 name = MimeUtility.getHeaderParameter(contentDisposition, "filename");
                             }
@@ -2380,7 +2388,7 @@ public Void doDbWork(final SQLiteDatabase db) throws WrappedException, Unavailab
                                 cv.put("name", name);
                                 cv.put("mime_type", attachment.getMimeType());
                                 cv.put("content_id", contentId);
-                                cv.put("content_disposition", contentDisposition);
+                                cv.put("content_disposition", dispositionType);
 
                                 attachmentId = db.insert("attachments", "message_id", cv);
                             } else {