fix leaks in lua get/set options

fix some resource leaks

- CID 169545
- CID 169546
- CID 171381
- CID 171382

Block SIGWINCH during connect(). (closes #3941)

FreeBSD's connect() does not respect SA_RESTART, so a SIGWINCH will
end up interrupting the connect.

If this code is needed in other places, it should be moved into
signal.c.  For this one place, inlining the sigprocmask() seemed okay.

fix whitespace errors

tidy the version strings

sync to upstream's "encrypt-to-self"

NeoMutt's variables:
    pgp_encrypt_self
    smime_encrypt_self

have been superceded by Mutt's:
    pgp_self_encrypt
    pgp_self_encrypt_as
    smime_self_encrypt
    smime_self_encrypt_as

Note: NeoMutt's variables will be deprecated at a later date.

More sophisticated check for BDB version + support for DB6 (non default)

This commit enhances configure.ac to actually parse (as in CPP) and
extract the values of DB_VERSION_MAJOR and DB_VERSION_MINOR from the
Berkeley DB include files. The previous approach to just use grep failed
on some system where the includes are just wrappers around the real bdb
headers.

Also, add support for the current db6 version (6.2), although db5 is
still the first one looked for.

Issue #613

fix: initialise debug_file

Make sure that `debug_file` is initialised early enough.
Also, call `mutt_expand_path()` to enable ~/ shortcuts.

Related to issue #654

contrib: add Mutt logos

Issue #655

revert part of 40f81d1 Add debug_level/debug_file options

This patch moves the set_default/restore_default couple after the defaults have
been set by the init function; this was correctly placed in 1.7.2 and somehow it
got moved to the wrong place, which means that tmpdir (and many other variable)
are not correctly set with their new default value determined from environment
variables (or other logic), so when they get resetted with a 'reset all' call in
.muttrc, they will never have the correct default to fall back to and they will
use the wrong one in init.h

This patch fixes:
- http://bugs.debian.org/865822
- http://bugs.debian.org/865842

Issue #654

score address pattern do not match personal name

Fixes #646
Debian BTS 383769

open attachments in read-only mode

Fixes #647
Debian BTS 528233
Mutt trac #3261

Add Cc, In-Reply-To, and References to default mailto_allow

These headers are quite useful for replying to mailing lists
via mailto links, as they don't break threading.

Debian has been shipping these in the /etc/Muttrc file since
September 2016 without much issues.

Fixes #115

merge: trivial code changes

 * build: don't delete non-existent files
 * remove unused option
 * docs: fix typos
 * update mailmap

update mailmap

docs: fix typos

Fixes #648

remove unused option

build: don't delete non-existent files

merge: upstream fixes (mutt/default)

 * Change the compose menu fields to be dynamically padded.
 * Rename 'sign as' to 'Sign as'; makes compose menu more consistent.
 * Make GPGME key selection behavior the same as classic-PGP. (see #3950)
 * Force hard redraw after $sendmail instead of calling mutt_endwin. (closes #3952) (see #3948)
 * Enable all header fields in the compose menu to be translated.
 * Auto-pad translation for the GPGME key selection "verify key" headers.
 * Improve the L10N comment about Sign as.

Improve the L10N comment about Sign as.

Make it clearer what the indentation behavior is, and add suggestions
about the width.

Thanks to TAKAHASHI Tamotsu for the original patch suggesting changes
to the L10N comments.

Auto-pad translation for the GPGME key selection "verify key" headers.

Remove the ridiculous need for the translators to pad the translation
strings themselves.

Enable all header fields in the compose menu to be translated.

While it is suggested they need not be, for some locales it might be
preferable to be able to localize the fields.

For instance, fr_FR may want to change the punctuation to have a space
in front of the colon.  ja_JP may be more comfortable with a native
character set description of the field when using it every day.

Force hard redraw after $sendmail instead of calling mutt_endwin. (closes #3952) (see #3948)

Adding a mutt_endwin() seemed like a clean solution to allowing
ncurses pinentry for $sendmail, but it leaves other users watching a
blank screen.  This change is extremely likely to generate a large
number of complaints and bug reports.  So instead, force a hard
refresh afterwards.

Make GPGME key selection behavior the same as classic-PGP. (see #3950)

Classic-PGP key selection auto-selected a match if there was a single
strong, valid, address-match result, even if there were other weak
matches.

GPGME was prompting in the same situation, if there were other weak
matches.

Change GPGME to match the classic behavior, as this is more useful and
matches the manual description.

Rename 'sign as' to 'Sign as'; makes compose menu more consistent.

Change the compose menu fields to be dynamically padded.

Pad based on the maximum width of the fields.  Note that this is a bit
of a mess, because some of the fields are translated while others are
not.

Update documentation for modify-labels-then-hide

This is a follow up to #602, documenting the new
behavior.

make modify-labels-then-hide hide or unhide

If the message is no longer in the current query after modifying
the label it will be hidden as before, but if it is in the current
query (again), it will be unhidden.

Related to #601

Add reorganised collapse_flagged patch

Decode CRLF line endings to LF when copying headers

This fixes display issues with files that consist of CRLF
line endings rather than just LF line endings, for example,
emails exported directly from the gmail API.

This only kicks in when CH_DECODE is set which might be a bit
strange, because that does RFC2047 header decoding, but it seems
a safe place to place this line end decoding without potentially
affecting anything else.

Include pattern for broken threads

References #616

From Frederic Duval:
http://cedricduval.free.fr/mutt/patches/download/patch-1.5.1.cd.pattern_broken.1

Improve search for mime.types

- Make the search order sane
- Print an error if mime.types is not found

Search order is:
    /etc/mime.types
    SYSCONFDIR/mime.types
    PKGDATADIR/mime.types
    ~/.mime.types

The last matching file wins.

fix: maildir creation

- added missing recursive directory creation after user prompt
- checked for trailing '/' when saving mbox

merge: start tidying Buffy handling

 * tidy some buffy code
 * split mutt_parse_mailboxes into mutt_parse_unmailboxes
 * drop VirtIncoming

drop VirtIncoming

Change the notmuch code to use Incoming rather than its own VirtIncoming.
These changes have no visible effect to the user.

This removes a lot of duplicate code and simplifies the logic in the
sidebar.

split mutt_parse_mailboxes into mutt_parse_unmailboxes

Split mutt_parse_mailboxes into two functions: one to add, one to remove.
There's not much overlap in the two functions and separating them
simplifies the logic.

Where unmailboxes used to use two loops, there's now only one.

tidy some buffy code

As a prelude to some notmuch changes, tidy up some buffy-handling
functions.  There are no functional changes.

- rename some variables
- reduce the scope of variables
- add some bools

CODE_OF_CONDUCT: remove jekyll code

[ci skip]

docs: add code of conduct

[ci skip]

merge: trivial code changes

 * reduce scope of for loop variables
 * replace variables with bools
 * rename static function
 * strip out once-used macro
 * while (1) -> while (true)

fix mailmap

while (1) -> while (true)

strip out once-used macro

rename static function

replace variables with bools

Each variable is limited to a single function.
Each was only set to 0 or 1.

Fucntions:
address_header_decode
alternative_handler
attach_forward_bodies
check_host
ci_send_message
clean_references
copy_clearsigned
crypt_getkeybyaddr
crypt_select_key
crypt_write_signed
disable_coredumps
display_line
do_uncolor
draw_sidebar
eat_message_range
eat_range
envelope_defaults
expand_aliases_r
get_token
handle_docline
imap_auth_sasl
imap_browse
imap_conn_find
imap_fast_trash
imap_fetch_message
imap_make_msg_set
imap_pretty_mailbox
imap_read_headers
imap_read_literal
init_common
init_menu
maildir_check_mailbox
main
mbox_check_mailbox
menu_redraw_index
mh_rewrite_message
mix_check_message
mix_make_chain
mutt_account_hook
mutt_alias_menu
mutt_attach_reply
mutt_body_handler
mutt_buffy_check
mutt_builtin_editor
mutt_check_simple
mutt_compose_attachment
mutt_copy_hdr
mutt_count_body_parts
mutt_display_message
mutt_draw_statusline
mutt_edit_attachment
mutt_edit_headers
mutt_expand_fmt
mutt_FormatString
mutt_index_menu
mutt_is_application_smime
mutt_multi_choice
mutt_pager
mutt_parse_color
mutt_parse_hook
mutt_parse_mono
mutt_parse_multipart
mutt_print_attachment
mutt_progress_update
mutt_remove_duplicates
mutt_sasl_start
mutt_signed_handler
mutt_view_attachment
mutt_write_fcc
mutt_write_rfc822_header
mutt_yesorno
nm_check_mailbox
nm_nonctx_get_count
nm_sync_mailbox
nntp_capabilities
nntp_hcache_update
nntp_newsrc_gen_entries
nntp_open_connection
parse_parameters
parse_pub_line
parse_regex
pgpring_dump_keyblock
pgp_application_pgp_handler
pgp_copy_clearsigned
pgp_decrypt_mime
pgp_find_keys
pgp_getkeybyaddr
pgp_getkeybystr
pgp_gpgme_application_handler
pgp_gpgme_decrypt_mime
pgp_read_packet
pgp_select_key
pgp_sign_message
pgp_traditional_encryptsign
pop_fetch_headers
print_smime_keyinfo
qp_decode_line
query_menu
reopen_mailbox
resolve_color
resolve_types
rfc1524_expand_filename
rfc2047_decode
rfc2231_decode_parameters
select_msg
show_one_sig_status
shrink_histfile
smime_getkeys
smime_select_key
ssl_init
tls_init
utf8_to_utf7
_mutt_expand_path
_mutt_parse_color
_mutt_traverse_thread

reduce scope of for loop variables

Each change is limited to a single function:
    attach_forward_bodies
    attach_forward_msgs
    attach_reply_envelope_defaults
    check_all_msg
    find_parent
    mutt_attach_bounce
    mutt_attach_reply
    mutt_attach_resend
    mutt_builtin_editor
    mutt_gen_attach_list

translation: update Chinese (Simplified)

init.c: allow sourcing of multiple files

Credits: Cedric Duval,
http://cedricduval.free.fr/mutt/patches/download/patch-1.5.4.cd.source_multiple.2

Issue #616

docs: fix ifdef examples

Fix trailing whitespaces

merge: drop obsolete options / docs

update gitignore

Update .gitignore to match all the latest build changes.

Completely remove dotlock

The last-ditch attempt to lock a file, dating back to the times when NFS
and SMB file-locking wasn't reliable.

drop UPDATING files

Drop:
    UPDATING    (from upstream)
    UPDATING.gz

The information in here is also in our ChangeLog.

One ChangeLog to bind them all

Drop files:
    ChangeLog      (from upstream)
    ChangeLog.nntp

Rename:
    ChangeLog.neomutt -> ChangeLog.md

drop configure options --enable-exact-address

Unused option.

drop configure option --with-exec-shell

Using /bin/sh is safe.

drop configure option --enable-nfs-fix

No longer needed.

drop configure option --disable-warnings

Always enable the warnings.

remove stamp-h.in

This file is generated by configure and shouldn't be committed to the
repo.

remove references to synonyms

Undocumented           | Real variable
-----------------------|-------------------------
edit_hdrs              | edit_headers
envelope_from          | use_envelope_from
forw_decode            | forward_decode
forw_decrypt           | forward_decrypt
forw_format            | forward_format
forw_quote             | forward_quote
hdr_format             | index_format
indent_str             | indent_string
mime_fwd               | mime_forward
msg_format             | message_format
pgp_autoencrypt        | crypt_autoencrypt
pgp_autosign           | crypt_autosign
pgp_auto_traditional   | pgp_replyinline
pgp_create_traditional | pgp_autoinline
pgp_replyencrypt       | crypt_replyencrypt
pgp_replysign          | crypt_replysign
pgp_replysignencrypted | crypt_replysignencrypted
pgp_verify_sig         | crypt_verify_sig
post_indent_str        | post_indent_string
print_cmd              | print_command
smime_sign_as          | smime_default_key
xterm_icon             | ts_icon_format
xterm_set_titles       | ts_enabled
xterm_title            | ts_status_format

Remove refs to deleted configure options

Obsolete configure options:
    --enable-hcache
    --enable-imap
    --enable-mailtool
    --enable-nntp
    --enable-pop
    --enable-smtp
    --with-included-gettext
    --with-regex

cleanup hdr_format_str

Sort options in alphabetical order for better overview.
Also while at it, removed a few superfluous blank lines.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

doc: fix spaces in the manual

Thanks to Bo YU (@yuzibo) and Floyd Anderson (@floand)

These were introduced when `tidy` was run.

docs: fix typos

build: remove another reference to devel-notes.txt

NeoMutt 2017-06-09 (1.8.3)

update translations

update version

fix colours properly

new contributors to mailmap

Revert "Move setting of GPG_TTY to mutt_init(). (see #3948)"

This reverts commit 89ddfa61c180de00769f6e1798579ec80349a4eb.

This commit was created by upstream and then reverted citing possible
security problems leaking the GPG_TTY env var.

Fix latest coverity issues (#624)

* Fix latest coverity issues

ncrypt/crypt.c
CID 170494:  Resource leaks  (RESOURCE_LEAK)
Variable "from" going out of scope leaks the storage it points to.

nntp.c
CID 170495:  Uninitialized variables  (UNINIT)
Declaring variable "buf" without initializer.

parse.c
CID 76957:  Error handling issues  (NEGATIVE_RETURNS)
"loc" is passed to a parameter that cannot be negative.

fix: don't pass colour-codes to filters

If the index_format string is being passed to an external filter, then
don't embed our secret colour-codes.

Fixes: #221

docs: explain binding warnings

Offer suggestion on binding problems

Ref: #621

fix: Don't set a colour unless it's been defined.

Some SETCOLOR's were being used without checking if the user had set the
colour, first.

If the user set the "normal" colours, then SETCOLOR would leave blocks
of unexpected colours.

Fixes #610

include mutt_regex.h in globals.h

Error observered
In file included from main.c:40:0:
globals.h:56:20: error: storage size of 'AttachKeyword' isn't known
 WHERE struct Regex AttachKeyword;
                    ^~~~~~~~~~~~~
globals.h:288:20: error: storage size of 'PgpGoodSign' isn't known
 WHERE struct Regex PgpGoodSign;
                    ^~~~~~~~~~~
Adding header include for mutt_regex.h.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

check return value of fgets

Following warning was observed:
 init.c: In function 'mutt_init':
 mutt.h:53:15: warning: ignoring return value of 'fgets_unlocked', declared with attribute warn_unused_result [-Wunused-result]
  #define fgets fgets_unlocked

 init.c:4053:7: note: in expansion of macro 'fgets'
        fgets(buffer, sizeof(buffer), f);
        ^~~~~
Added code that checks return value.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

devel-notes.txt replaced by docs on neomutt.org

The prepare script mentions doc/devel-notes.txt, which no longer exists.
Point to https://www.neomutt.org/dev/newbie-tutorial instead.

merge: upstream fixes (mutt/default)

 * Fix "format string is not a string literal" warnings. (closes #3949)
 * Enable TEXTDOMAINDIR override to make translation testing easier.
 * Change km_dokey() to return -2 on a timeout/sigwinch.
 * Fix tag-prefix to not abort on $timeout.
 * Restore setenv function.
 * Move setting of GPG_TTY to mutt_init(). (see #3948)
 * Add a mutt_endwin() before invoking $sendmail. (closes #3948)

Add a mutt_endwin() before invoking $sendmail. (closes #3948)

This is to support invoking a program such as msmtp that can use gpg
to decrypt a password.  The ncurses pinentry can corrupt the screen
unless we exit curses.

Move setting of GPG_TTY to mutt_init(). (see #3948)

This allows other programs mutt runs to use the ncurses pinentry if needed.

Restore setenv function.

Partially revert fa1192803257, restoring setenv.  It turns out we
still need the function.

Fix tag-prefix to not abort on $timeout.

If $timeout is set very low, then it can cancel the tag-prefix before
the user has time to press the desired command.

Change the code to set the tag flag, and then recontinue with normal
event processing instead.  Cancel on an abort, but continue with a timeout.

Thanks to Lauri Tirkkonen for reporting the issue.

Change km_dokey() to return -2 on a timeout/sigwinch.

In some cases, such as tag-prefix or _mutt_enter_string(), it is
desirable to be able to distinguish between a timeout/sigwinch event
and an input error/abort/ctrl-g.

Enable TEXTDOMAINDIR override to make translation testing easier.

If set, TEXTDOMAINDIR will point to a translation directory instead of
using the mutt installation directory.

Fix "format string is not a string literal" warnings. (closes #3949)

Mutt calls msgfmt with '-c' to verify that translation format strings
match, but it is still safer to indirect strings with no formatting
through %s.

fix translations

update Japanese translation

fix some remaining translation problems

build: fix make git_ver.h

By separating the lines, the version variable was getting lost.

build: fix make dist

small refactoring

* replace ESC_CHAR macro with function
* added braces to if logic to improve readability

github: simplify PR template

In a lot of PR's, either the question "Why is this PR needed" is removed
or answered very similar to "What does this PR do?". Therefore, i think
this can be removed.

[ci skip]

merge: Remove legacy support / notes

 * remove AIX code, as it's unsupported
 * doc: remove Ultrix note, as it's unsupported
 * remove notes about SunOS as it's discontinued
 * INSTALL: remove mutt notes for supported platforms

INSTALL: remove mutt notes for supported platforms

remove notes about SunOS as it's discontinued

doc: remove Ultrix note, as it's unsupported

remove AIX code, as it's unsupported