]> granicus.if.org Git - mutt/commit
Remove redundant mbox delimiter check in imap_browse(). (closes #3646)
authorKevin McCarthy <kevin@8t8.us>
Sat, 21 Nov 2015 23:28:57 +0000 (15:28 -0800)
committerKevin McCarthy <kevin@8t8.us>
Sat, 21 Nov 2015 23:28:57 +0000 (15:28 -0800)
commit0864167e8a6ba598cf919fb26c242997518f7386
treea372b5cf17ff10bbd1b4bd6fa2bb89f1ba872a20
parentd9dfd78e9e307f4ab5ac8f71d64114a0214f2db2
Remove redundant mbox delimiter check in imap_browse().  (closes #3646)

imap_fix_path() removes duplicate and trailing delimiters, so the check
below it was redundant.  This also made it appear list.delim could be
used uninitialized.

Remove the check, but add a check to make sure the "fixed" path has
len>0, to prevent oob accesses of mbox[n-1] below.

Lastly, remove a redundant n=strlen(mbox) inside the initial LIST
processing loop.  The mbox isn't changed from above, so there is no
need to rerun strlen.
imap/browse.c