X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=uip%2Fsendsbr.c;h=f8b415cb12dd692a7ee577934202bf8a9106f69d;hp=c7f4d447238ea7fc1c530607bdd55c98af742fd6;hb=3baec236a39c5c89a9bda8dbd988d643a21decc6;hpb=714b5c530ece27ea2835a313013f5b770163403c diff --git a/uip/sendsbr.c b/uip/sendsbr.c index c7f4d44..f8b415c 100644 --- a/uip/sendsbr.c +++ b/uip/sendsbr.c @@ -30,7 +30,6 @@ int debugsw = 0; /* global */ int forwsw = 1; int inplace = 1; int pushsw = 0; -int unique = 0; int verbsw = 0; char *altmsg = NULL; /* .. */ @@ -52,7 +51,7 @@ static FILE *composition_file; /* composition file pointer */ /* ** external prototypes */ -int sendsbr(char **, int, char *, struct stat *, int, char *, int); +int sendsbr(char **, int, char *, struct stat *, int); char *getusername(void); /* @@ -65,10 +64,10 @@ static void anno(int, struct stat *); static void annoaux(int); static int sendaux(char **, int, char *, struct stat *); -static int attach(char *, char *, int); +static int attach(char *); static void clean_up_temporary_files(void); static int get_line(void); -static void make_mime_composition_file_entry(char *, int); +static void make_mime_composition_file_entry(char *); /* @@ -76,11 +75,10 @@ static void make_mime_composition_file_entry(char *, int); */ int -sendsbr(char **vec, int vecp, char *drft, struct stat *st, - int rename_drft, char *attachment_header_field_name, int attachformat) +sendsbr(char **vec, int vecp, char *drft, struct stat *st, int rename_drft) { int status; - char buffer[BUFSIZ], file[BUFSIZ]; + char buffer[BUFSIZ]; char *original_draft; /* name of original draft file */ char *p; /* string pointer for building file name */ @@ -94,50 +92,29 @@ sendsbr(char **vec, int vecp, char *drft, struct stat *st, original_draft = drft; /* - ** There might be attachments if a header field name for - ** attachments is supplied. Convert the draft to a MIME message. + ** Convert the draft to a MIME message. ** Use the mhbuild composition file for the draft if there was ** a successful conversion because that now contains the MIME ** message. A nice side effect of this is that it leaves the ** original draft file untouched so that it can be retrieved ** and modified if desired. */ + switch (attach(drft)) { + case OK: + drft = composition_file_name; + break; - if (attachment_header_field_name != (char *)0) { - switch (attach(attachment_header_field_name, drft, - attachformat)) { - case OK: - drft = composition_file_name; - break; - - case NOTOK: - return (NOTOK); + case NOTOK: + return (NOTOK); - case DONE: - break; - } + case DONE: + break; } done=armed_done; switch (setjmp(env)) { case OK: - /* - ** If given -push and -unique (which is undocumented), then - ** rename the draft file. I'm not quite sure why. - */ - if (pushsw && unique) { - char *cp = m_mktemp2(drft, invo_name, NULL, NULL); - if (cp == NULL) { - adios("sendsbr", "unable to create temporary file"); - } - if (rename(drft, strncpy(file, cp, sizeof(file))) - == NOTOK) - adios(file, "unable to rename %s to", drft); - drft = file; - } - status = sendaux(vec, vecp, drft, st) ? NOTOK : OK; - /* rename the original draft */ if (rename_drft && status == OK && rename(original_draft, strncpy(buffer, @@ -167,7 +144,7 @@ sendsbr(char **vec, int vecp, char *drft, struct stat *st, if (strlen(composition_file_name) >= sizeof (composition_file_name) - 6) - advise((char *)0, "unable to remove original composition file."); + advise(NULL, "unable to remove original composition file."); else { if ((p = strrchr(composition_file_name, '/')) == NULL) @@ -175,12 +152,12 @@ sendsbr(char **vec, int vecp, char *drft, struct stat *st, else p++; - (void)strcpy(body_file_name, p); + strcpy(body_file_name, p); *p++ = ','; - (void)strcpy(p, body_file_name); - (void)strcat(p, ".orig"); + strcpy(p, body_file_name); + strcat(p, ".orig"); - (void)unlink(composition_file_name); + unlink(composition_file_name); } } @@ -188,80 +165,85 @@ sendsbr(char **vec, int vecp, char *drft, struct stat *st, } static int -attach(char *attachment_header_field_name, char *draft_file_name, - int attachformat) +attach(char *draft_file_name) { char buf[MAXPATHLEN + 6]; /* miscellaneous buffer */ int c; /* current character for body copy */ int has_attachment; /* draft has at least one attachment */ int has_body; /* draft has a message body */ + int non_ascii; /* msg body contains non-ASCII chars */ int length; /* length of attachment header field name */ char *p; /* miscellaneous string pointer */ - /* - ** Open up the draft file. - */ - + /* Open up the draft file. */ if ((draft_file = fopen(draft_file_name, "r")) == (FILE *)0) - adios((char *)0, "can't open draft file `%s'.", + adios(NULL, "can't open draft file `%s'.", draft_file_name); /* ** Allocate a buffer to hold the header components as they're read in. ** This buffer might need to be quite large, so we grow it as needed. */ - field = (char *)mh_xmalloc(field_size = 256); /* - ** Scan the draft file for a header field name that matches the - ** -attach argument. The existence of one indicates that the + ** Scan the draft file for an attachment header field name. + ** The existence of one indicates that the ** draft has attachments. Bail out if there are no attachments ** because we're done. Read to the end of the headers even if ** we have no attachments. */ - - length = strlen(attachment_header_field_name); + length = strlen(attach_hdr); has_attachment = 0; - while (get_line() != EOF && *field != '\0' && *field != '-') - if (strncasecmp(field, attachment_header_field_name, length) - == 0 && field[length] == ':') + while (get_line() != EOF && *field != '\0' && *field != '-') { + if (strncasecmp(field, attach_hdr, length) == 0 && + field[length] == ':') { has_attachment = 1; - - if (has_attachment == 0) - return (DONE); + } + } /* - ** We have at least one attachment. Look for at least one - ** non-blank line in the body of the message which indicates - ** content in the body. + ** Look for at least one non-blank line in the body of the + ** message which indicates content in the body. + ** Check if body contains at least one non-blank (= not empty) + ** and if it contains any non-ASCII chars (= need MIME). */ - has_body = 0; + non_ascii = 0; while (get_line() != EOF) { for (p = field; *p != '\0'; p++) { if (*p != ' ' && *p != '\t') { has_body = 1; - break; + } + if (*p > 127 || *p < 0) { + non_ascii = 1; } } - - if (has_body) + if (has_body && non_ascii) break; } /* - ** Make names for the temporary files. + ** Bail out if there are no attachments and only ASCII text. + ** This means we don't need to convert it to MIME. + */ + if (!has_attachment && non_ascii==0) { + return DONE; + } + + /* + ** Else: mimify */ - (void)strncpy(body_file_name, - m_mktemp(m_maildir(invo_name), NULL, NULL), + /* Make names for the temporary files. */ + strncpy(body_file_name, + m_mktemp(toabsdir(invo_name), NULL, NULL), sizeof (body_file_name)); - (void)strncpy(composition_file_name, - m_mktemp(m_maildir(invo_name), NULL, NULL), + strncpy(composition_file_name, + m_mktemp(toabsdir(invo_name), NULL, NULL), sizeof (composition_file_name)); if (has_body) @@ -269,10 +251,9 @@ attach(char *attachment_header_field_name, char *draft_file_name, composition_file = fopen(composition_file_name, "w"); - if ((has_body && body_file == (FILE *)0) || - composition_file == (FILE *)0) { + if ((has_body && !body_file) || !composition_file) { clean_up_temporary_files(); - adios((char *)0, "unable to open all of the temporary files."); + adios(NULL, "unable to open all of the temporary files."); } /* @@ -280,62 +261,55 @@ attach(char *attachment_header_field_name, char *draft_file_name, ** non-attachment header fields to the temporary composition file. ** Then add the dashed line separator. */ - rewind(draft_file); - - while (get_line() != EOF && *field != '\0' && *field != '-') - if (strncasecmp(field, attachment_header_field_name, length) - != 0 || field[length] != ':') - (void)fprintf(composition_file, "%s\n", field); - - (void)fputs("--------\n", composition_file); - - /* - ** Copy the message body to a temporary file. - */ + while (get_line() != EOF && *field && *field != '-') { + if (strncasecmp(field, attach_hdr, length) != 0 || + field[length] != ':') { + fprintf(composition_file, "%s\n", field); + } + } + fputs("--------\n", composition_file); if (has_body) { - while ((c = getc(draft_file)) != EOF) - putc(c, body_file); + /* Copy the message body to the temporary file. */ + while ((c = getc(draft_file)) != EOF) { + putc(c, body_file); + } + fclose(body_file); - (void)fclose(body_file); + /* Add a mhbuild MIME composition file line for the body */ + /* charset will be discovered/guessed by buildmimeproc */ + fprintf(composition_file, "#text/plain %s\n", body_file_name); } /* - ** Add a mhbuild MIME composition file line for the body if - ** there was one. - */ - - if (has_body) - make_mime_composition_file_entry(body_file_name, attachformat); - - /* ** Now, go back to the beginning of the draft file and look for ** header fields that specify attachments. Add a mhbuild MIME ** composition file for each. */ - rewind(draft_file); - - while (get_line() != EOF && *field != '\0' && *field != '-') { - if (strncasecmp(field, attachment_header_field_name, length) - == 0 && field[length] == ':') { - for (p = field + length + 1; *p == ' ' || *p == '\t'; - p++) - ; - - make_mime_composition_file_entry(p, attachformat); + while (get_line() != EOF && *field && *field != '-') { + if (strncasecmp(field, attach_hdr, length) == 0 && + field[length] == ':') { + for (p = field+length+1; *p==' ' || *p=='\t'; p++) { + continue; + } + if (*p == '+') { + /* forwarded message */ + fprintf(composition_file, "#forw [forwarded message(s)] %s\n", p); + } else { + make_mime_composition_file_entry(p); + } } } - (void)fclose(composition_file); + fclose(composition_file); /* ** We're ready to roll! Run mhbuild on the composition file. ** Note that mhbuild is in the context as buildmimeproc. */ - - (void)sprintf(buf, "%s %s", buildmimeproc, composition_file_name); + sprintf(buf, "%s %s", buildmimeproc, composition_file_name); if (system(buf) != 0) { clean_up_temporary_files(); @@ -348,8 +322,8 @@ attach(char *attachment_header_field_name, char *draft_file_name, static void clean_up_temporary_files(void) { - (void)unlink(body_file_name); - (void)unlink(composition_file_name); + unlink(body_file_name); + unlink(composition_file_name); return; } @@ -370,7 +344,7 @@ get_line(void) for (n = 0, p = field; (c = getc(draft_file)) != EOF; *p++ = c) { if (c == '\n' && (c = getc(draft_file)) != ' ' && c != '\t') { - (void)ungetc(c, draft_file); + ungetc(c, draft_file); c = '\n'; break; } @@ -389,181 +363,49 @@ get_line(void) } static void -make_mime_composition_file_entry(char *file_name, int attachformat) +make_mime_composition_file_entry(char *file_name) { - int binary; /* binary character found flag */ - int c; /* current character */ - char cmd[MAXPATHLEN + 6]; /* file command buffer */ - char *content_type; /* mime content type */ - FILE *fp; /* content and pipe file pointer */ - struct node *np; /* context scan node pointer */ - char *p; /* miscellaneous string pointer */ - struct stat st; /* file status buffer */ - - content_type = (char *)0; - - /* - ** Check the file name for a suffix. Scan the context for that - ** suffix on a mhshow-suffix- entry. We use these entries to - ** be compatible with mhshow, and there's no reason to make the - ** user specify each suffix twice. Context entries of the form - ** "mhshow-suffix-contenttype" in the name have the suffix in - ** the field, including the dot. - */ - - if ((p = strrchr(file_name, '.')) != (char *)0) { - for (np = m_defs; np; np = np->n_next) { - if (strncasecmp(np->n_name, "mhshow-suffix-", 14) == 0 - && mh_strcasecmp(p, np->n_field) == 0) - { - content_type = np->n_name + 14; - break; - } - } - } - - /* - ** No content type was found, either because there was no matching - ** entry in the context or because the file name has no suffix. - ** Open the file and check for non-ASCII characters. Choose the - ** content type based on this check. - */ - - if (content_type == (char *)0) { - if ((fp = fopen(file_name, "r")) == (FILE *)0) { - clean_up_temporary_files(); - adios((char *)0, "unable to access file \"%s\"", - file_name); - } - - binary = 0; - - while ((c = getc(fp)) != EOF) { - if (c > 127 || c < 0) { - binary = 1; - break; - } + FILE *fp; + struct node *np; + char *cp; + char content_type[BUFSIZ]; + char cmdbuf[BUFSIZ]; + char *cmd = mimetypequeryproc; + + for (np = m_defs; np; np = np->n_next) { + if (strcasecmp(np->n_name, mimetypequery)==0) { + cmd = np->n_field; + break; } - - (void)fclose(fp); - - content_type = binary ? - "application/octet-stream" : "text/plain"; } + snprintf(cmdbuf, sizeof cmdbuf, "%s %s", cmd, file_name); - /* - ** Make sure that the attachment file exists and is readable. - ** Append a mhbuild directive to the draft file. This starts with - ** the content type. Append a file name attribute and a private - ** x-unix-mode attribute. Also append a description obtained - ** (if possible) by running the "file" command on the file. - */ - - if (stat(file_name, &st) == -1 || access(file_name, R_OK) != 0) { + if (!(fp = popen(cmdbuf, "r"))) { clean_up_temporary_files(); - adios((char *)0, "unable to access file \"%s\"", file_name); + adios(NULL, "unable to determine content type with `%s'", + cmdbuf); } + if (fgets(content_type, sizeof content_type, fp) && + (cp = strrchr(content_type, '\n'))) { + *cp = '\0'; + } else { + strcpy(content_type, "application/octet-stream"); + admonish(NULL, "problems with `%s', using fall back type `%s'", + cmdbuf, content_type); + } + pclose(fp); - switch (attachformat) { - case 0: - /* Insert name, file mode, and Content-Id. */ - (void)fprintf(composition_file, - "#%s; name=\"%s\"; x-unix-mode=0%.3ho", - content_type, ((p = strrchr(file_name, '/')) - == (char *)0) ? file_name : p + 1, - (unsigned short)(st.st_mode & 0777)); - - if (strlen(file_name) > MAXPATHLEN) { - clean_up_temporary_files(); - adios((char *)0, "attachment file name `%s' too long.", - file_name); - } - - (void)sprintf(cmd, "file '%s'", file_name); - - if ((fp = popen(cmd, "r")) != (FILE *)0 && - fgets(cmd, sizeof (cmd), fp) != (char *)0) { - *strchr(cmd, '\n') = '\0'; - - /* - ** The output of the "file" command is of the form - ** - ** file: description - ** - ** Strip off the "file:" and subsequent white space. - */ - - for (p = cmd; *p != '\0'; p++) { - if (*p == ':') { - for (p++; *p != '\0'; p++) { - if (*p != '\t') - break; - } - break; - } - } - - if (*p != '\0') - /* Insert Content-Description. */ - (void)fprintf(composition_file, " [ %s ]", p); - - (void)pclose(fp); - } - - break; - case 1: - if (stringdex(m_maildir(invo_name), file_name) == 0) { - /* - ** Content had been placed by send into a temp file. - ** Don't generate Content-Disposition header, because - ** it confuses Microsoft Outlook, Build 10.0.6626, at - ** least. - */ - fprintf(composition_file, "#%s <>", content_type); - } else { - /* - ** Suppress Content-Id, insert simple - ** Content-Disposition. - */ - fprintf(composition_file, - "#%s; name=\"%s\" <>{attachment}", - content_type, - ((p = strrchr(file_name, '/')) == (char *)0) ? - file_name : p + 1); - } - - break; - case 2: - if (stringdex(m_maildir(invo_name), file_name) == 0) { - /* - ** Content had been placed by send into a temp file. - ** Don't generate Content-Disposition header, because - ** it confuses Microsoft Outlook, Build 10.0.6626, at - ** least. - */ - fprintf(composition_file, "#%s <>", content_type); - } else { - /* - ** Suppress Content-Id, insert Content-Disposition - ** with modification date. - */ - fprintf(composition_file, - "#%s; name=\"%s\" <>{attachment; modification-date=\"%s\"}", - content_type, - ((p = strrchr(file_name, '/')) == (char *)0) ? - file_name : p + 1, dtime(&st.st_mtime, 0)); - } - - break; - default: - adios((char *)0, "unsupported attachformat %d", attachformat); + /* TODO: don't use access(2) because it checks for ruid, not euid */ + if (access(file_name, R_OK) != 0) { + clean_up_temporary_files(); + adios(NULL, "unable to access file `%s'", file_name); } - /* - ** Finish up with the file name. - */ + fprintf(composition_file, "#%s; name=\"%s\" <>{attachment}", + content_type, + (!(cp = strrchr(file_name, '/'))) ? file_name : cp + 1); - (void)fprintf(composition_file, " %s\n", file_name); + fprintf(composition_file, " %s\n", file_name); return; } @@ -578,26 +420,20 @@ sendaux(char **vec, int vecp, char *drft, struct stat *st) { pid_t child_id; int i, status, fd, fd2; - char backup[BUFSIZ], buf[BUFSIZ]; + char backup[BUFSIZ]; fd = pushsw ? tmp_fd() : NOTOK; fd2 = NOTOK; vec[vecp++] = drft; - if (annotext) { - if ((fd2 = tmp_fd()) != NOTOK) { - vec[vecp++] = "-idanno"; - snprintf(buf, sizeof(buf), "%d", fd2); - vec[vecp++] = buf; - } else { - admonish(NULL, "unable to create file for annotation list"); - } + if (annotext && (fd2 = tmp_fd()) == NOTOK) { + admonish(NULL, "unable to create file for annotation list"); } if (distfile && distout(drft, distfile, backup) == NOTOK) done(1); vec[vecp] = NULL; - for (i = 0; (child_id = vfork()) == NOTOK && i < 5; i++) + for (i = 0; (child_id = fork()) == NOTOK && i < 5; i++) sleep(5); switch (child_id) { @@ -675,48 +511,48 @@ alert(char *file, int out) sleep(5); switch (child_id) { - case NOTOK: - /* oops -- fork error */ - advise("fork", "unable to"); - - case OK: - /* child process -- send it */ - SIGNAL(SIGHUP, SIG_IGN); - SIGNAL(SIGINT, SIG_IGN); - SIGNAL(SIGQUIT, SIG_IGN); - SIGNAL(SIGTERM, SIG_IGN); - if (forwsw) { - if ((in = open(file, O_RDONLY)) == NOTOK) { - admonish(file, "unable to re-open"); - } else { - lseek(out, (off_t) 0, SEEK_END); - strncpy(buf, "\nMessage not delivered to anyone.\n", sizeof(buf)); - write(out, buf, strlen(buf)); - strncpy(buf, "\n------- Unsent Draft\n\n", sizeof(buf)); - write(out, buf, strlen(buf)); - cpydgst(in, out, file, "temporary file"); - close(in); - strncpy(buf, "\n------- End of Unsent Draft\n", sizeof(buf)); - write(out, buf, strlen(buf)); - if (rename(file, strncpy(buf, m_backup(file), sizeof(buf))) == NOTOK) - admonish(buf, "unable to rename %s to", file); - } + case NOTOK: + /* oops -- fork error */ + advise("fork", "unable to"); + + case OK: + /* child process -- send it */ + SIGNAL(SIGHUP, SIG_IGN); + SIGNAL(SIGINT, SIG_IGN); + SIGNAL(SIGQUIT, SIG_IGN); + SIGNAL(SIGTERM, SIG_IGN); + if (forwsw) { + if ((in = open(file, O_RDONLY)) == NOTOK) { + admonish(file, "unable to re-open"); + } else { + lseek(out, (off_t) 0, SEEK_END); + strncpy(buf, "\nMessage not delivered to anyone.\n", sizeof(buf)); + write(out, buf, strlen(buf)); + strncpy(buf, "\n------- Unsent Draft\n\n", sizeof(buf)); + write(out, buf, strlen(buf)); + cpydgst(in, out, file, "temporary file"); + close(in); + strncpy(buf, "\n------- End of Unsent Draft\n", sizeof(buf)); + write(out, buf, strlen(buf)); + if (rename(file, strncpy(buf, m_backup(file), sizeof(buf))) == NOTOK) + admonish(buf, "unable to rename %s to", file); } - lseek(out, (off_t) 0, SEEK_SET); - dup2(out, fileno(stdin)); - close(out); - /* create subject for error notification */ - snprintf(buf, sizeof(buf), "send failed on %s", - forwsw ? "enclosed draft" : file); - - execlp(mailproc, r1bindex(mailproc, '/'), - getusername(), "-subject", buf, NULL); - fprintf(stderr, "unable to exec "); - perror(mailproc); - _exit(-1); - - default: /* no waiting... */ - break; + } + lseek(out, (off_t) 0, SEEK_SET); + dup2(out, fileno(stdin)); + close(out); + /* create subject for error notification */ + snprintf(buf, sizeof(buf), "send failed on %s", + forwsw ? "enclosed draft" : file); + + execlp(mailproc, mhbasename(mailproc), getusername(), + "-subject", buf, NULL); + fprintf(stderr, "unable to exec "); + perror(mailproc); + _exit(-1); + + default: /* no waiting... */ + break; } } @@ -760,34 +596,34 @@ anno(int fd, struct stat *st) child_id = debugsw ? NOTOK : fork(); switch (child_id) { - case NOTOK: /* oops */ - if (!debugsw) - advise(NULL, "unable to fork, so doing annotations by hand..."); - if (cwd == NULL) - cwd = getcpy(pwd()); - - case OK: - /* block a few signals */ - sigemptyset(&set); - sigaddset(&set, SIGHUP); - sigaddset(&set, SIGINT); - sigaddset(&set, SIGQUIT); - sigaddset(&set, SIGTERM); - SIGPROCMASK(SIG_BLOCK, &set, &oset); - - annoaux(fd); - if (child_id == OK) - _exit(0); - - /* reset the signal mask */ - SIGPROCMASK(SIG_SETMASK, &oset, &set); - - chdir(cwd); - break; + case NOTOK: /* oops */ + if (!debugsw) + advise(NULL, "unable to fork, so doing annotations by hand..."); + if (cwd == NULL) + cwd = getcpy(pwd()); - default: /* no waiting... */ - close(fd); - break; + case OK: + /* block a few signals */ + sigemptyset(&set); + sigaddset(&set, SIGHUP); + sigaddset(&set, SIGINT); + sigaddset(&set, SIGQUIT); + sigaddset(&set, SIGTERM); + SIGPROCMASK(SIG_BLOCK, &set, &oset); + + annoaux(fd); + if (child_id == OK) + _exit(0); + + /* reset the signal mask */ + SIGPROCMASK(SIG_SETMASK, &oset, &set); + + chdir(cwd); + break; + + default: /* no waiting... */ + close(fd); + break; } } @@ -806,7 +642,7 @@ annoaux(int fd) admonish(NULL, "$mhfolder not set"); return; } - maildir = m_maildir(folder); + maildir = toabsdir(folder); if (chdir(maildir) == NOTOK) { if (debugsw) admonish(maildir, "unable to change directory to"); @@ -868,7 +704,7 @@ annoaux(int fd) if (is_selected(mp, msgnum)) { if (debugsw) advise(NULL, "annotate message %d", msgnum); - annotate(m_name (msgnum), annotext, cp, inplace, + annotate(m_name(msgnum), annotext, cp, inplace, 1, -2, 0); } }