#include <unistd.h>
#include <sys/stat.h>
#include <locale.h>
+#include <sysexits.h>
#ifdef HAVE_SYS_PARAM_H
# include <sys/param.h>
#define DROPGROUPPRIVS() \
if (setegid(getgid()) != 0) { \
advise ("setegid", "unable to set group to %ld", (long) getgid()); \
- _exit (-1); \
+ _exit (EX_OSERR); \
}
#define GETGROUPPRIVS() \
if (setegid(return_gid) != 0) { \
advise ("setegid", "unable to set group to %ld", (long) return_gid); \
- _exit (-1); \
+ _exit (EX_OSERR); \
}
#define SAVEGROUPPRIVS() return_gid = getegid()
#else
char *maildir_copy = NULL;
if (atexit(inc_done) != 0) {
- adios(NULL, "atexit failed");
+ adios(EX_OSERR, NULL, "atexit failed");
}
-/*
-** absolutely the first thing we do is save our privileges,
-** and drop them if we can.
-*/
+ /*
+ ** absolutely the first thing we do is save our privileges,
+ ** and drop them if we can.
+ */
SAVEGROUPPRIVS();
TRYDROPGROUPPRIVS();
setlocale(LC_ALL, "");
invo_name = mhbasename(argv[0]);
- /* read user profile/context */
context_read();
arguments = getarguments(invo_name, argc, argv, 1);
switch (smatch(++cp, switches)) {
case AMBIGSW:
ambigsw(cp, switches);
- /* sysexits.h EX_USAGE */
- exit(1);
+ exit(EX_USAGE);
case UNKWNSW:
- adios(NULL, "-%s unknown", cp);
+ adios(EX_USAGE, NULL, "-%s unknown", cp);
case HELPSW:
snprintf(buf, sizeof(buf), "%s [+folder] [switches]", invo_name);
print_help(buf, switches, 1);
- exit(0);
+ exit(argc == 2 ? EX_OK : EX_USAGE);
case VERSIONSW:
print_version(invo_name);
- exit(0);
+ exit(argc == 2 ? EX_OK : EX_USAGE);
case AUDSW:
if (!(cp = *argp++) || *cp == '-')
- adios(NULL, "missing argument to %s", argp[-2]);
+ adios(EX_USAGE, NULL, "missing argument to %s", argp[-2]);
audfile = getcpy(expanddir(cp));
continue;
case NAUDSW:
case FILESW:
if (!(cp = *argp++) || *cp == '-')
- adios(NULL, "missing argument to %s",
+ adios(EX_USAGE, NULL, "missing argument to %s",
argp[-2]);
from = getcpy(expanddir(cp));
case FORMSW:
if (!(form = *argp++) || *form == '-')
- adios(NULL, "missing argument to %s",
+ adios(EX_USAGE, NULL, "missing argument to %s",
argp[-2]);
continue;
case WIDTHSW:
if (!(cp = *argp++) || *cp == '-')
- adios(NULL, "missing argument to %s",
+ adios(EX_USAGE, NULL, "missing argument to %s",
argp[-2]);
width = atoi(cp);
continue;
}
if (*cp == '+' || *cp == '@') {
if (folder)
- adios(NULL, "only one folder at a time!");
+ adios(EX_USAGE, NULL, "only one folder at a time!");
else
folder = getcpy(expandfol(cp));
} else {
- adios(NULL, "usage: %s [+folder] [switches]",
+ adios(EX_USAGE, NULL, "usage: %s [+folder] [switches]",
invo_name);
}
}
newmail = concat(mailspool, "/", getusername(), NULL);
}
if (stat(newmail, &s1) == NOTOK || s1.st_size == 0)
- adios(NULL, "no mail to incorporate");
+ adios(EX_DATAERR, NULL, "no mail to incorporate");
if ((cp = strdup(newmail)) == NULL)
- adios(NULL, "error allocating memory to copy newmail");
+ adios(EX_OSERR, NULL, "error allocating memory to copy newmail");
newmail = cp;
maildir = toabsdir(folder);
if ((maildir_copy = strdup(maildir)) == NULL)
- adios(maildir, "error allocating memory to copy maildir");
+ adios(EX_OSERR, maildir, "error allocating memory to copy maildir");
create_folder(maildir, noisy ? 0 : 1, exit);
if (chdir(maildir) == NOTOK)
- adios(maildir, "unable to change directory to");
+ adios(EX_OSERR, maildir, "unable to change directory to");
- /* read folder and create message structure */
if (!(mp = folder_read(folder)))
- adios(NULL, "unable to read folder %s", folder);
+ adios(EX_IOERR, NULL, "unable to read folder %s", folder);
if (access(newmail, W_OK) != NOTOK) {
locked++;
in = lkfopen(newmail, "r");
DROPGROUPPRIVS();
if (in == NULL)
- adios(NULL, "unable to lock and fopen %s", newmail);
+ adios(EX_IOERR, NULL, "unable to lock and fopen %s", newmail);
fstat(fileno(in), &s1);
} else {
trnflag = 0;
if ((in = fopen(newmail, "r")) == NULL)
- adios(newmail, "unable to read");
+ adios(EX_IOERR, newmail, "unable to read");
}
/* This shouldn't be necessary but it can't hurt. */
if ((i = stat(audfile, &st)) == NOTOK)
advise(NULL, "Creating Receive-Audit: %s", audfile);
if ((aud = fopen(audfile, "a")) == NULL)
- adios(audfile, "unable to append to");
+ adios(EX_IOERR, audfile, "unable to append to");
else if (i == NOTOK)
chmod(audfile, m_gmprot());
fflush(stdout);
}
+ /* check if readable and nonempty */
+ if (!fgets(buf, sizeof(buf), in)) {
+ if (ferror(in)) {
+ advise("read", "unable to");
+ incerr = SCNFAT;
+ } else {
+ incerr = SCNEOF;
+ }
+ goto giveup;
+ }
+ if (strncmp("From ", buf, 5)!=0) {
+ advise(NULL, "not in mbox format");
+ incerr = SCNFAT;
+ goto giveup;
+ }
+
/*
** Get the mail from file (usually mail spool)
*/
- thisisanmbox(in);
hghnum = msgnum = mp->hghmsg;
for (;;) {
/*
*/
break;
}
+giveup:;
free(maildir_copy);
if (incerr < 0) { /* error */
} else {
fclose(in); in = NULL;
}
- adios(NULL, "failed");
+ adios(EX_SOFTWARE, NULL, "failed");
}
if (aud)
fclose(in); in = NULL;
}
- seq_setunseen(mp, 1); /* add new msgs to unseen sequences */
- seq_save(mp); /* synchronize sequences */
- context_save(); /* save the context file */
+ seq_setunseen(mp, 1);
+ seq_save(mp);
+ context_save();
return 0;
}