soup-multipart: remove extra CRLF before multipart body
authorDan Winship <danw@gnome.org>
Tue, 6 Apr 2010 16:29:01 +0000 (12:29 -0400)
committerDan Winship <danw@gnome.org>
Fri, 9 Apr 2010 23:51:33 +0000 (19:51 -0400)
HTTP multiparts should have no preamble, so it shouldn't have the
leading CRLF either

https://bugzilla.gnome.org/show_bug.cgi?id=614183

libsoup/soup-multipart.c

index f825a9d..9223f1d 100644 (file)
@@ -437,7 +437,8 @@ soup_multipart_to_message (SoupMultipart *multipart,
                part_headers = multipart->headers->pdata[i];
                part_body = multipart->bodies->pdata[i];
 
-               str = g_string_new ("\r\n--");
+               str = g_string_new (i == 0 ? NULL : "\r\n");
+               g_string_append (str, "--");
                g_string_append (str, multipart->boundary);
                g_string_append (str, "\r\n");
                soup_message_headers_iter_init (&iter, part_headers);