struct msgs *
folder_read (char *name, int lockflag)
{
- int msgnum, prefix_len, len, *mi;
+ int msgnum, len, *mi;
struct msgs *mp;
struct dirent *dp;
DIR *dd;
}
/* Allocate the main structure for folder information */
- mp = (struct msgs *) mh_xmalloc ((size_t) sizeof(*mp));
-
+ NEW(mp);
clear_folder_flags (mp);
mp->foldpath = name;
mp->lowmsg = 0;
if (access (name, W_OK) == -1)
set_readonly (mp);
- prefix_len = strlen(BACKUP_PREFIX);
/*
* Allocate a temporary place to record the
default:
/* skip any files beginning with backup prefix */
- if (!strncmp (dp->d_name, BACKUP_PREFIX, prefix_len))
+ if (has_prefix(dp->d_name, BACKUP_PREFIX))
continue;
/* skip the LINK file */
/* for testing, allocate minimal necessary space */
/* mp->hghoff = max (mp->hghmsg, 1); */
+ /*
+ * If for some reason hghoff < lowoff (like we got an integer overflow)
+ * the complain about this now.
+ */
+
+ if (mp->hghoff < mp->lowoff) {
+ adios(NULL, "Internal failure: high message limit < low message "
+ "limit; possible overflow?");
+ }
+
/*
* Allocate space for status of each message.
*/
/*
* Read and initialize the sequence information.
*/
- seq_read (mp, lockflag);
+ if (seq_read (mp, lockflag) == NOTOK) {
+ char seqfile[PATH_MAX];
+
+ /* Failed to lock sequence file. */
+ snprintf (seqfile, sizeof(seqfile), "%s/%s", mp->foldpath, mh_seq);
+ advise (seqfile, "failed to lock");
+
+ return NULL;
+ }
return mp;
}