X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=webcit-ng%2Fserver%2Fmessages.c;h=f176156a51d4ccdda7728acf21464348adb80098;hb=9207704c8f44bce5bf21b18906aa23151a177ef0;hp=9fdb678b4da088e7822cf52c28f3eb0fd4346020;hpb=09186f9e04c7de4d95979376c459e89b111f0113;p=citadel.git

diff --git a/webcit-ng/server/messages.c b/webcit-ng/server/messages.c
index 9fdb678b4..f176156a5 100644
--- a/webcit-ng/server/messages.c
+++ b/webcit-ng/server/messages.c
@@ -208,7 +208,6 @@ void dav_put_message(struct http_transaction *h, struct ctdlsession *c, char *eu
 		ctdl_printf(c, "MIME-Version: 1.0\r");
 		ctdl_printf(c, "Content-Type: multipart/mixed; boundary=\"%s\"\r", mime_boundary);
 		ctdl_write(c, HKEY("\r\n"));
-		ctdl_write(c, HKEY("\r\n"));
 		ctdl_printf(c, "--%s\r", mime_boundary);	// start of message body
 	}
 
@@ -231,7 +230,11 @@ void dav_put_message(struct http_transaction *h, struct ctdlsession *c, char *eu
 		for (i=0; i<num_attachments; ++i) {
 			extract_token(attid, att, i, ',', sizeof(attid));
 			one_att = pop_upload(attid);
-			syslog(LOG_DEBUG, "💥 attachment: %s", one_att.filename);
+
+			// After calling pop_upload(), the attachment is no longer in the global list.
+			// The file descriptor has zero links, so when we close it, the filesystem will remove it from disk.
+
+			syslog(LOG_DEBUG, "💥 attachment: %s , len=%ld", one_att.filename, one_att.length);
 			ctdl_printf(c, "--%s--\r", mime_boundary);
 			ctdl_printf(c, "Content-Type: %s; name=\"%s\"\r", one_att.content_type, one_att.filename);
 			ctdl_printf(c, "Content-Disposition: attachment; filename=\"%s\"\r", one_att.filename);
@@ -240,10 +243,20 @@ void dav_put_message(struct http_transaction *h, struct ctdlsession *c, char *eu
 
 			char *raw_att = malloc(one_att.length);
 			if (raw_att) {
-				fread(raw_att, one_att.length, 1, one_att.fp);
+				rewind(one_att.fp);
+				if (fread(raw_att, one_att.length, 1, one_att.fp) != 1) {
+					syslog(LOG_ERR, "messages: %m");
+				}
 
 				// now encode it
-
+				char *encoded_att = malloc((one_att.length * 150) / 100);
+				if (encoded_att) {
+					size_t encoded_length = CtdlEncodeBase64(encoded_att, raw_att, one_att.length, BASE64_YES_LINEBREAKS);
+					ctdl_write(c, encoded_att, encoded_length);
+					syslog(LOG_DEBUG, "Encoded attachment: len=%ld", encoded_length);
+					free(encoded_att);
+				}
+				free(raw_att);
 			}
 
 			fclose(one_att.fp);