X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/81a21a9a97d8633f6d6231e31fdb6e328d0d3ff2..d0656bf2442041891c2c67214c0a4637cbba9ce0:/sbr/folder_read.c?ds=inline diff --git a/sbr/folder_read.c b/sbr/folder_read.c index 8e3a3ffa..e4bac904 100644 --- a/sbr/folder_read.c +++ b/sbr/folder_read.c @@ -2,8 +2,6 @@ /* * folder_read.c -- initialize folder structure and read folder * - * $Id$ - * * This code is Copyright (c) 2002, by the authors of nmh. See the * COPYRIGHT file in the root directory of the nmh distribution for * complete copyright information. @@ -25,13 +23,14 @@ */ struct msgs * -folder_read (char *name) +folder_read (char *name, int lockflag) { int msgnum, prefix_len, len, *mi; struct msgs *mp; - struct stat st; struct dirent *dp; DIR *dd; + bvector_t *v; + size_t i; name = m_mailpath (name); if (!(dd = opendir (name))) { @@ -39,14 +38,8 @@ folder_read (char *name) return NULL; } - if (stat (name, &st) == -1) { - free (name); - return NULL; - } - /* 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; @@ -56,6 +49,8 @@ folder_read (char *name) mp->hghsel = 0; mp->numsel = 0; mp->nummsg = 0; + mp->seqhandle = NULL; + mp->seqname = NULL; if (access (name, W_OK) == -1) set_readonly (mp); @@ -76,10 +71,7 @@ folder_read (char *name) */ if (mp->nummsg >= len) { len += NUMMSGS; - if (!(mi = (int *) realloc (mi, - (size_t) (len * sizeof(*mi))))) { - adios (NULL, "unable to allocate storage"); - } + mi = (int *) mh_xrealloc (mi, (size_t) (len * sizeof(*mi))); } /* Check if this is the first message we've seen */ @@ -104,9 +96,6 @@ folder_read (char *name) switch (dp->d_name[0]) { case '.': case ',': -#ifdef MHE - case '+': -#endif /* MHE */ continue; default: @@ -134,10 +123,26 @@ folder_read (char *name) /* 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. */ - mp->msgstats = mh_xmalloc (MSGSTATSIZE(mp, mp->lowoff, mp->hghoff)); + mp->num_msgstats = MSGSTATNUM (mp->lowoff, mp->hghoff); + mp->msgstats = mh_xmalloc (MSGSTATSIZE(mp)); + for (i = 0, v = mp->msgstats; i < mp->num_msgstats; ++i, ++v) { + *v = bvector_create (0); + } + + mp->msgattrs = svector_create (0); /* * Clear all the flag bits for all the message @@ -159,7 +164,15 @@ folder_read (char *name) /* * Read and initialize the sequence information. */ - seq_read (mp); + 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; }