Fix imap auth plain

Send the password and `\r\n` as one message.

Fixes #1286

quote imap strings more carefully

Co-authored-by: JerikoOne <jeriko.one@gmx.us>

sanitise cache paths

Co-authored-by: JerikoOne <jeriko.one@gmx.us>

Do not interrupt switch fallthrough

Commit e85c735 introduced the `<check-stats>` function, designated by
`OP_CHECK_STATS`.

This had been inserted below `OP_MAIN_CHANGE_VFOLDER`, which is
`change-vfolder`'s designation; however, `OP_MAIN_CHANGE_VFOLDER` relies
on C's switch fallthrough logic to execute. As a result,
`change-vfolder` executed `check-stats`'s logic instead of the correct
logic.

This commit moves `OP_CHECK_STATS` to above the #ifdef that
`OP_MAIN_CHANGE_VFOLDER` is defined in.

Add alloc fail check in nntp_fetch_headers

Set length modifiers for group and desc

nntp_add_group parses a line controlled by the connected nntp server.
Restrict the maximum lengths read into the stack buffers group, and
desc.

prevent buffer underrun

Thanks @l2dy

reduce config variable setting

Reduce the number of places where the config variables are set.
This will reduce the complexity when setting becomes a function call.

Correct a comment typo

protect some paths from relativity

Some path config variables represent folders which may be non-local.
e.g. `set folder = 'imap://...'

They shouldn't be treated as relative file paths.

Add check-stats function to calculate mailbox statistics.

This allows the statistics to be updated without setting
$mail_check_stats or before $mail_check_stats_interval has passed.

Add a reference to $copy in the $record documentation.

Make path config variables relative

Distinguish between strings, paths and commands

merge: small refactoring

 * Simplify the logic in mutt_addr_write_single()
 * Remove duplicate function prototype
 * Only calculate mutt_fqdn when needed
 * Fix memory leak when creating alias

Fix memory leak when creating alias

Only calculate mutt_fqdn when needed

Remove duplicate function prototype

Simplify the logic in mutt_addr_write_single()

Remove doubled function call in nested if statement

fix mutt_buffy_notify() to properly update BuffyNotify

In commit
184244053 doxy: document buffy.c
the check for whether to call mutt_buffy_list() was changed from
if (mutt_buffy_check(false) && BuffyNotify)
to
if (BuffyNotify && (mutt_buffy_check(false) != 0))

By reversing the check mutt_buffy_check() never executes when
BuffyNotify is zero. However, it is essential to run mutt_buffy_check()
before checking BuffyNotify, as mutt_buffy_check() updates BuffyNotify.

Therefore return the check to its inital form

Fixes #1279

merge: minor improvements

- Boolify some parameters
- Rename some buffers for consistency

mutt_update_recvattach_menu

parse_date_range

match_addrlist

encode_quoted

mutt_prepare_envelope

mutt_is_list_recipient

mutt_fqdn

mutt_rfc822_write_header

mutt_make_message_attach

prepend_curdir

update_content_info

make_from_addr

make_from

quote_simple

mutt_get_body_charset

merge: replace mutt_mktemp() with mutt_file_mkstemp()

 * pgp_class_application_handler()
 * pgp_class_decrypt_mime()
 * pgp_class_encrypted_handler()
 * pgp_class_encrypt_message()
 * pgp_class_traditional_encryptsign()
 * pgp_class_verify_one()
 * pgp_decrypt_part()
 * pgp_find_hash()
 * pgp_gpgme_decrypt_mime()
 * pgp_gpgme_encrypted_handler()
 * pgp_gpgme_extract_keys()
 * smime_class_build_smime_entity()
 * smime_class_decrypt_mime()
 * smime_class_invoke_import()
 * smime_class_verify_one()
 * smime_extract_certificate()
 * smime_extract_signer_certificate()
 * smime_gpgme_application_handler()
 * smime_gpgme_decrypt_mime()
 * smime_handle_cert_email()
 * smime_handle_entity()

Remove errfile from smime_handle_entity

Remove outfile from smime_handle_entity

Remove tmptmpfname from smime_handle_entity

Remove ftmpname from smime_handle_cert_email

Remove tempfile from smime_gpgme_decrypt_mime

Remove tempfile from smime_gpgme_application_handler

Remove tmpfname from smime_extract_signer_certificate

Remove tmpfname from smime_extract_certificate

Remove smimeerrfile from smime_class_verify_one

Remove tmpfname from smime_class_invoke_import

Remove tempfile from smime_class_decrypt_mime

Remove smimeerrfile from smime_class_build_smime_entity

Remove tmpfile from pgp_gpgme_extract_keys

Remove tempfile from pgp_gpgme_encrypted_handler

Remove tempfile from pgp_gpgme_decrypt_mime

Remove tempfile from pgp_find_hash

Remove pgperrfile from pgp_decrypt_part

Remove pgperrfile from pgp_class_verify_one

Remove pgperrfile from pgp_class_traditional_encryptsign

Remove pgperrfile from pgp_class_encrypt_message

Remove tempfile from pgp_class_encrypted_handler

Remove tempfile from pgp_class_decrypt_mime

Remove pgperrfile from pgp_class_application_handler

Remove pgpoutfile from pgp_class_application_handler

Put bit fields together

Use the enum constant to initialise

teach docs about true and false

merge: NeoMutt 2018-06-22

sync translations

clang-format

update contributors

changelog

version 2018-06-22

merge: add doxygen docs

 * document pattern functions
 * document sendlib functions
 * document send functions
 * document recvattach functions

document recvattach functions

document send functions

document sendlib functions

document pattern functions

merge: update translations

 * Update German translation
 * Update ja translation
 * Update zh_CN translation

Update German translation

update German translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update ja translation

Update zh_CN translation

Fix env var quoting in backticks

Issue #1262

ensure Buffers are initialised

merge: upstream fixes

 * Add $pgp_check_gpg_decrypt_status_fd.
 * Add GnuPG status fd checks for inline pgp.
 * Allow larger passphrase timeout values
 * mutt_str_atol: better error handling

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