mutt_str_atol: better error handling

Reset errno to 0 before calling strtol and testing it in mutt_atol.

Otherwise providing LONG_MAX+1 then LONG_MAX gave an error for LONG_MAX too.

* Detect overflow on negative numbers too (< LONG_MIN).

* Distinguish between format error and number overflow as already
  expected (e.g. for pgp_timeout and smime_timeout).

Allow larger passphrase timeout values

This came up in the comp.mail.mutt newsgroup where a user wasn't
satisfied with the SHORT_MAX seconds ~9 hours limit on passphrase
timeouts.

For the first time made it necessary for the options parser to be
able to parse numbers as long values. Also, introduced
mutt_add_timeout() to detect possible overflow before adding a
timeout to a time_t value and truncate to TIME_T_MAX instead.

Add GnuPG status fd checks for inline pgp.

The difficulty is that "BEGIN PGP MESSAGE" could be a signed and
armored part, so we can't fail hard if it isn't encrypted.

Change pgp_check_decryption_okay() to return more status codes, with
>=0 indicating an actual decryption; -2 and -1 indicating plaintext
found; and -3 indicating an actual DECRYPTION_FAILED status code seen.

Fail hard on -3, but change the message for -2 and -1 to indicate the
message was not encrypted.

Add $pgp_check_gpg_decrypt_status_fd.

If set (the default) mutt performs more thorough checking of the
$pgp_decrypt_command status output for GnuPG result codes.

Ticket #39 revealed that GnuPG (currently) does not protect against
messages that have been manipulated to contain an empty encryption
packet followed by a plaintext packet.

A huge thanks to Marcus Brinkmann for researching this issue, taking
the time to report it to us (and the GnuPG team), and taking even more
time to clarify exactly what needed to be checked for.   

tidy init.h values

strip down .gitignore

Get rid of all the things we don't see any more, like autoconf.

Refactor resize.c into separate functions

Splits up mutt_resize_screen() into its separate function to improve readability.

Closes #1149

merge: minor improvements

 * light tidy
 * coverity: fix defects
 * Put variable declaration inside case-break into braces.
 * Trailing spaces
 * Insert extern
 * One definition is enough
 * External linkage was not intended
 * Remove doubled #include
 * Redundant if clause

Redundant if clause

Remove doubled #include

External linkage was not intended

One definition is enough

Insert extern

The variable was already defined in "git_ver.h".

Trailing spaces

Put variable declaration inside case-break into braces.

coverity: fix defects

light tidy

Fix for missing fingerprint

If GPGME doesn't return a key fingerprint, don't crash.

Fix off-by-one in index jump

In an index of 'n' items, you couldn't jump to 'n', only (n-1).

Fixes #1261

acutest: sync to upstream

This file is from the Acutest project:

- https://github.com/mity/acutest

Specifically, from commit: `03885e23`

fcc_attach: Don't ask every time

Option fcc_attach used to ask in any case, if there were attachment nad if there weren't.
In case there is no attachment, there is no need to ask.

fix non-empty buffers

Insert a few `mutt_buffer_reset()`s.

Commit 9d6a2418 replaced a lot of `snprintf()`s with `mutt_buffer_printf()`.
Unfortunately, a few of the original lines were relying on `snprintf()`
writing over the beginning of the buffer.

doxygen: document mh functions

doxygen: document menu functions

drop memset for simple neomutt structs

drop memset for simple arrays

merge: more ncrypt tidying

 * drop unused param in pgp_make_key_attachment()
 * drop unused param in pgp_extract_keys_from_attachment_list()
 * boolify need_passphrase
 * boolify params
 * boolify retval

boolify retval

boolify params

boolify need_passphrase

drop unused param in pgp_extract_keys_from_attachment_list()

The function wasn't being use to look for tagged messages.
So, it's been renamed to pgp_extract_key_from_attachment()

drop unused param in pgp_make_key_attachment()

It was unused by any of the callers of the API wrapper:
crypt_pgp_make_key_attachment().

merge: mass tidy-up of ncrypt

 * drop unnec typedefs
 * fold CryptModuleFunctions into CryptModuleSpecs
 * drop redirect functions
 * tidy headers
 * doxygen: Document the encryption API
 * doxygen: document api implementors
 * unify the naming of all the ncrypt API functions
 * doxygen: document the ncrypt api wrappers

doxygen: document the ncrypt api wrappers

unify the naming of all the ncrypt API functions

doxygen: document api implementors

doxygen: Document the encryption API

tidy headers

drop redirect functions

fold CryptModuleFunctions into CryptModuleSpecs

drop unnec typedefs

merge: minor improvements

 * tidy bcache
 * boolify function params
 * doxy: add function comments
 * fix comment order
 * tidy hcache definitions

tidy hcache definitions

fix comment order

doxy: add function comments

boolify function params

tidy bcache

revert dodgy type changes

Some of the type changes I introduced in 309ecbf4 would cause problems
when strtol() and atoi() return errors.

fix trash mailbox

The second `for` loop was using the value of `i` as a starting place.
By changing the first loop, the second loop wasn't doing any work.

Fixes #1253

merge: update translations

 * add missing 'Plural-Forms' statement
 * sync translations
 * trans: de-dupe an ssl string
 * Update Czech translation
 * Update ja translation
 * Update Slovak translation
 * Update zh_CN translation
 * use consistent expression for memory streams

sync translations

Update zh_CN translation

use consistent expression for memory streams

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update ja translation

Update ja translation

trans: de-dupe an ssl string

Update Slovak translation

add missing 'Plural-Forms' statement

Update ja translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update Czech translation

Update zh_CN translation

merge: increase and unify buffer sizes

 * adjust the size of some general buffers
 * Use sizeof() rather than constants
 * Create a large buffer for command strings
 * Enlarge buffers from _POSIX_PATH_MAX to PATH_MAX
 * Enlarge buffers from LONG_STRING to PATH_MAX

Enlarge buffers from LONG_STRING to PATH_MAX

Every buffer that has a filesystem path copied into it

Enlarge buffers from _POSIX_PATH_MAX to PATH_MAX

Every buffer that has a filesystem path copied into it

Create a large buffer for command strings

Use sizeof() rather than constants

merge: small improvements

 * tidy handler.c
 * tidy for loops
 * insert {}s
 * doxy: document buffy.c
 * tidy return statements
 * move new functions to reduce header deps
 * add enum for 'set' commands
 * tidy the regex types
 * minor tidying

minor tidying

- rename parameters
- change member types
- fix doxygen typo
- make variable const

tidy the regex types

- remove unnecessary casts
- fix variable types
- remove once-used macro (REGEXEC)

add enum for 'set' commands

The 'set' family of commands are mutually exclusive, so change the
defines to an enumeration.

move new functions to reduce header deps

Having the `new` functions in the header creates a requirement on the
mutt library.

tidy return statements

Add ()s to return statements where the logic is non-trivial.

doxy: document buffy.c

insert {}s

Insert {}s where the conditional or body take up more than one line.

tidy for loops

- Add {}s where the body of a `for` loop isn't obvious.
- Add ()s where the logic isn't clear

tidy handler.c

- Move the function prototypes to a new file `handler.h`
- Rearrange the functions

There are no changes to the code.

tidy hcache

- Drop the typedefs -- they were only used once
- Move the backend list `HCACHE_BACKEND_LIST` into hcache.c
- Update all the source docs

merge: tidy the mailbox API

 * unify parameter naming
 * reorder functions
 * rename functions
 * rename implementation functions
 * document all implementation functions
 * swap mx_commit_message() params
 * rename mx functions to match api

mx api: rename mx functions to match api

mx api: swap mx_commit_message() params

for consistency with the other api functions

mx api: document all implementation functions

mx api: rename implementation functions

mx api: rename functions

mx api: reorder functions

mx api: unify parameter naming

Do not (try to) install / uninstall nonexistent man pages

See #1247

merge: drop pgpring references

 * remove pgpring from docs
 * remove last references to pgpring
 * drop obsolete pgp notes

drop obsolete pgp notes

remove last references to pgpring

remove pgpring from docs

Remove PgpSignature

The PgpSignature is only used by the pgpring tool.
The pgpring tool is only needed by users of PGP2 and PGP5.
PGP5 is over twenty years old.

- remove PgpSignature from ncrypt/pgplib.[ch]
- remove pgppubring.c
- update Makefile.autosetup
- remove contrib/pgp2.rc
- remove contrib/pgp5.rc
- remove contrib/pgp6.rc
- update contrib/Makefile.autosetup

pager: index must be rebuilt on MUTT_REOPENED

When we call mx_check_mailbox() and something change
header in Context->hdrs may have been updated.

In case of MUTT_NEW_MAIL, the page rebuild the index, and fixup
rd.index->current/max and extra->hdr. If the extra->hdr change
like when an above the opened one, or the selected one, is deleted, the
pager is closed.

In case of MUTT_REOPENED we do nothing. Making rd.index->current/max
wrong and extra->hdr perhaps pointing to something that doesn't exist
anymore.

This change makes the MUTT_REOPENED behavior similar to MUTT_NEW_MAIL.

Note that we this the pager have good change to be closed with the
mailbox is modified externally, just like the current MUTT_NEW_MAIL
behavior.

Closes #796

Handle a BAD response in AUTH PLAIN w/o initial response (#1237)

The incorrect assumption was that the first line of an AUTH PLAIN w/o
I-R was always successful, so the client could write the second line and
end up with a readable socket that would be activated by the server
sending the final OK or BAD response.
If however the first line results in a BAD response already, the server
wouldn't write anything else and the client would be left polling for an
additional response that never arrives, effectively hanging the process.

Issue #1236

Avoid regenerating hcache/hcversion.h after building objects in hcache

Specifying the hcache directory asqa dependency for hcache/hcversion.h
means that the latter will be regenerated whenever the hcache directory
is changed, such as when object files are written into it. The net
effect is that "make; make" regenerates hcache/hcversion.h twice.

This commit fixes this problem by explicitely creating the hcache
directory as part of the hcache/hcversion.h target.