]> granicus.if.org Git - neomutt/commit
Fix improper signed int conversion of IMAP uid and msn values.
authorKevin McCarthy <kevin@8t8.us>
Sat, 6 Jan 2018 04:39:50 +0000 (20:39 -0800)
committerRichard Russon <rich@flatcap.org>
Sun, 7 Jan 2018 01:51:16 +0000 (01:51 +0000)
commitac54945b250f90c59d18c6852a7899093df22721
tree654ed4b82be0a044fda10791048c6b1bcfcb783f
parent5820043d06a047caa5aca7547e4839fb3bbc7bc6
Fix improper signed int conversion of IMAP uid and msn values.

Several places in the imap code, when parsing "number" and "nz-number"
values from the IMAP data, use atoi() and strtol().  This is
incorrect, and can result in failures when a uid value happens to be
larger than 2^31.

Create a helper function, mutt_atoui() and use that instead.  One
place was using strtol() and relying on the endptr parameter, and so
was changed to use strtoul() instead.

Thanks to Paul Saunders for the bug report and original patch, which
this commit is based on.
imap/command.c
imap/imap.c
imap/message.c
muttlib.c
protos.h