Add libidn2 support

Merge branch 'stable'

automatic post-release commit for mutt-1.9.5

Merge branch 'stable'

Improve gss debug printing of status_string.

Commit f52ee2f7 ensured the debug strings were properly '\0'
terminated.  However, it did not prevent the strncpy from reading past
the end of the status_string.value data; it simply capped it
afterwards.  Improve the code so it only reads up to
status_string.length without overwriting the buffer.

Merge branch 'stable'

Remove trailing null count from gss_buffer_desc.length field.

RFC 2744 section 3.2.2 clearly states that trailing NULL characters
are not to be included in the length field, and are not to be assumed
to be present in the value field.

Thanks to Greg Hudson, who recently debugged this same issue with
fetchmail, and kindly took the time to look at Mutt's code too.

Merge branch 'stable'

Rename _regex.h to _mutt_regex.h to avoid name collision on Macs.

On Macs, <regex.h> includes <_regex.h>.  Because Mutt defines '-I .'
during compilation, our bundled version of _regex.h was being used
instead of the system one.

I have no idea how it managed to work before, but starting in Xcode
9.3, the differences in struct size and fields started to produce a
crash and other strange behavior.

The real issue is our use of '-I .' during compilation, which allows
our local headers to override system ones.  An easier fix for now is
to rename the header.

Many thanks to Charles Diza, Christian Ebert, and Fabian Groffen for
their help trying things out and helping to debug the problem.

Also, a huge thanks to Steve Karmeinsky for allowing me to ssh in to
his Mac so I could track down the underlying issue.

Merge branch 'stable'

configure: check for tinfo matching ncurses

When we selected ncursesw, look for tinfow as well.  Since ncurses-6.1
the binary compatibility between tinfo and tinfow disappeared, resulting
in crashes and other odd behaviour.

This change checks for tinfo or tinfow based on which ncurses we found
to continue with.

https://bugs.gentoo.org/651552

[Thanks to Fabian Groffen for the patch.  I made a minor modification
to fall back to tinfo because pre-6.1 systems may not have a tinfow
but still need tinfo to compile.]

Updated Polish translation.

pager specifc "show incoming mailboxes list" macro

Exiting the pager before change-folder avoids 'Not available in this
menu' error when selecting a different entry in some circumstances.

Increase account.user/login size to 128.

Like the pass field in commit 52949004, the existing size was
insufficient for the reporter of issue #18.  For now, just bump up the
size, but I'll move up looking into making these dynamically sized.

Remove dead code in pgp.c

pgp_extract_keys_from_messages() has been subsumed by
crypt_extract_keys_from_messages().  The function had incorrect
multi-bit flag comparisons, but since it's dead code, remove it
instead.

Fix comparison of flags with multiple bits set.

PGPENCRYPT, PGPKEY, SMIMEENCRYPT, and SMIMEOPAQUE are all combination
flags, with multiple bits set.  In a few places these flags were
bitwise-and'ed incorrectly: expecting a non-zero result to indicate
all the bits in the flag were set.  Change those to explicitly compare
the result against the original flag.

Updated French translation.

Change prompt string for $crypt_verify_sig

Thanks to Michael Tatge for pointing out that the prompt is used for
both PGP and S/MIME.

Merge branch 'stable'

Fix s/mime non-detached signature handling.

This fix is based on stbuehler's patch from
https://dev.gnupg.org/T2919.

Recent versions of gpgme seem to terminate the connection for a
protocol error.  stbuehler's analysis is that this is actually a
gpgme bug, but recreating the context works around the problem.

Change mutt_error call in mutt_gpgme_set_sender() to dprint.

The message is not an error message, and flies by so quickly it has
little informative content.  Change it to use dprint instead.

Thanks to Darac Marjal for reporting the issue.

Updated French translation.

Improve the error message when a signature is missing.

Thanks to Vincent Lefèvre for reporting the issue and for the wording
suggestion!

Updated French translation.

Add $abort_noattach and $abort_noattach_regexp options.

$abort_noattach_regexp is matched against the body of a message.  If
so, the $abort_noattach quadoption is used to prompt whether to abort
sending.

Thanks to Antonio Radici for bringing the original version of the
patch to our attention, which he pulled from
https://github.com/tlvince/pkgbuild/blob/master/mutt-kiss/mutt-attach.patch

This version was rewritten to use a regexp, to fix a few issues, and
to better fit in with Mutt styles and conventions.

Merge branch 'stable'

automatic post-release commit for mutt-1.9.4

Merge branch 'stable'

Add subjectrx section to the muttrc man page.

Fix subjectrx example in the manual.

Avoid a potential integer overflow if a Content-Length value is huge.

Merge branch 'stable'

Fix is_from() year parsing to abort on year overflow.

Unlike mutt_parse_date(), is_from() was not checking for overflow, and
could end up passing a negative year to mutt_mktime().

It should perhaps be changed to use mutt_atoi(), which does better
range checking, but that requires mutt_atoi() being changed to allow
trailing characters and its callers return value checks being
updated.  I'll put that on the todo list.

Cap parsed years at 9999 when converting to time_t.

Large year values, even those less than INT_MAX, apparently can cause
gmtime() and localtime() to return NULL.  Mutt needs larger changes
checking and handling those errors, but this will prevent the
immediately triggerable issue.

Improve index color cache flushing behavior.

Flush when redefining the colors for an existing color line, in
addition to when defining a new color line.

Flush only when uncoloring the index.

Avoid a potential integer overflow if a Content-Length value is huge.

Change mkreldate.sh to use the UTC date with git, so that it is increasing.

Fix document version and date for git build.

Change mkreldate.sh to output just the date, so it can be used in the
manual too.

Use git to generate the release date if available.

For now, fall back to the ChangeLog.  Perhaps the release date should
be generated from a file, similar to VERSION, for those cases instead.

Add a blank line between the compile options and the contact info for mutt -v

Updated French translation.

Put gitlab issues url on a separate line.

This removes the need to use brackets, and makes it easier for
translations to fit within 80 columns.

Thanks to Derek Martin for the suggested wording, and to Vincent
Lefèvre for proofreading and noticing a small error.

Fix history menu title.

Remove an accidental leading space.  Thanks to Vincent Lefèvre for
noticing the problem and providing a patch!

Updated French translation.

Add history-search function, bound to ctrl-r.

Create a very basic "search history" functionality in the line editor.
It uses the current input, and searches backward through history.  If
there is one match, it immediately uses that otherwise it pops up a
simple menu of matches.

Merge branch 'stable'

Add stub flea and muttbug scripts back.

It was rightfully pointed out that the removal was too abrupt.  These
programs have been around for a long time, and many internet searches
still say to use them for reporting bugs.

Add stub versions which inform to use the gitlab url instead.

Merge branch 'stable'

Add more delimiters for gitlab url in the po files.

Some of them hid from me in my wgrep session because they were marked
as binary.

Merge branch 'stable'

Add delimiters around gitlab issues URL.

Without the delimiters, the period can be easily inadvertantly copied
as part of the URL.

Merge branch 'stable'

automatic post-release commit for mutt-1.9.3

Remove useless assignments in add_to_list functions.

The code was probably pulled from some of the header processing
functions, which were in a loop.  In those versions, "last" pointed to
a collection of new entries appended, but in these two functions,
updating last just obfuscates the code.

Add a blurb about $new_mail_command in the mailboxes section.

Merge branch 'stable'

Fix the ftp site url in the manual.

Also fix the urlview link to point to sigpipe's github account.

Merge branch 'stable'

Fix one more dev site reference in the manual.

Merge branch 'stable'

Fix a couple memory leaks in pattern.c

BUFFER.data was not being freed in a couple cases.

The pattern and simple search were not being freed if imap_search()
failed.

Merge branch 'stable'

Change bug reporting URL to gitlab issues.

Pull latest Polish translation from master.

Update manual to point to gitlab.com as development site.

Remove muttbug and flea.

We have pretty much transitioned over to gitlab.com.

Update po

Followup for untranslate Copyright list.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Split Copyright and Thanks in help output.

The Copyright string is changing often, and its content is obvious.
It does not need translation. The remaining string can be translated.
This change avoids a stale translation once one of the years change.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Update po

Preparation for following changes.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

add clang static analysis to builds

gitlab-ci: build against openssl and gnutls

Add pgp_default_key and smime_sign_as info to contrib rc files.

Explain $pgp_default_key vs $pgp_sign_as in gpg.rc.

Explain $smime_default_key vs $smime_sign_as in smime.rc.

Add pages to .gitlab-ci.yml

Use this to regenerate the manual and store it in the pages for the
project.

Update smime.rc: Typo fix, consistent headings

use mutt/ubuntu build image, build with everything on

skeleton .gitlab-ci.yml to compile commits

Fixed GPGME translations that weren’t shown but affected the keyboard

Update pl.po

Fix typo in de.po

Signed-off-by: Olaf Hering <olaf@aepfle.de>

examine_directory: set directory/symlink size to zero

The size of a directory or symlink in the folder browser is not meaningful.
For directories it means just how many blocks were allocated to hold all
entries. It does not mean that the entries are still present in the directory.
For symlinks its the size of the target.
Set both to zero to simplify the folder browser output.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

mutt_pretty_size: show real number for small files

If a file is smaller than a certain size it is unfriendly to print 0K or
0,1K as number of mails or as file size. Instead use the real number.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Merge branch 'stable'

Add missing setup calls when resuming encrypted drafts.

Calls to get the passphrase were missing for app/pgp and app/smime.
App/smime was also missing a call to crypt_smime_getkeys().

If a failure occurs, report it back, rather than just continuing.
Otherwise, postponed messages could be completely lost.

Create pgp and s/mime default and sign_as key vars. (see #3983)

The $postpone_encrypt and $(pgp/smime)_self_encrypt configuration
variables have created a somewhat messier situation for users.  Many
of them now have to specify their keys across multiple configuration
variables.

(Trac) Ticket #3983 had a reasonable request: "if my encrypt and
signing keys are the same, why can't I just specify my key once in my
.muttrc?"

The problem currently is that $smime_default_key and $pgp_sign_as are
both used to specify signing keys, and are set by the "sign (a)s"
security menu choice.  So we can't store encryption keys there because
some users have separate sign-only capability keys.

Create $pgp_default_key to store the default encryption key.  Change
signing to use $pgp_default_key, unless overridden by $pgp_sign_as.
The pgp "sign (a)s" will continue setting $pgp_sign_as.

Create $smime_sign_as.  Change signing to use $smime_default_key
unless overridden by $smime_sign_as.  Change s/mime "sign (a)s" menu
to set $smime_sign_as instead.

Change $postpone_encrypt and $(pgp/smime)_self_encrypt to use
$(pgp/smime)_default_key by default.

Mark $(pgp/smime)_self_encrypt_as deprecated.  They are now aliases
for the $(pgp/smime)_default_key config vars.

Change $(pgp/smime)_self_encrypt default to set.

The intent is that most users now need only set
$(pgp/smime)_default_key.  If they have a sign-only key, or have
separate signing and encryption keys, they can put that in
$(pgp/smime)_sign_as.  This also enables to default self_encrypt on
and solve a very common request.

Thanks to Michele Marcionelli and Vincent Lefèvre for gently pushing
me towards a solution.

Updated French translation.

Change the default of NewMailCmd to 0.

This is so the documentation is generated consistently for unset
strings.

NewMailCmd: optionally execute a command upon new mail arrival

This setting allows to run any external program to e.g. produce a
notification on the desktop when Mutt finds new mail.

Signed-off-by: Fabian Groffen <grobian@gentoo.org>

cmd_handle_fatal: make error message a bit more descriptive

When there are multiple IMAP connections available, "Mailbox closed"
doesn't give a hint as to which one.  Use account info to identify which
mailbox was closed.

Merge branch 'stable'

Fix imap status count range check.

The strtoul() call for parsing the STATUS count wasn't checking the
range properly, because it was assigning to an unsigned int.

Change to assign to a unsigned long, and also add the conversion check
from mutt_atoui().

Thanks to Charles (@chdiza) for quickly noticing the problem!

Merge branch 'stable'

Change imap literal counts to parse and store unsigned ints.

IMAP literals are of type number.  Change imap_get_literal_count() to
use mutt_atoui() instead of atoi().  Change the return type variables
used to store the count to type unsigned int.

It's doubtful this was a real issue, but as long as we're cleaning up
incorrect atoi() usage, we should fix this too.

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.

Updated French translation.

Disable message security if the backend is not available.

Gitlab issue #3 exposed an awkward corner case: if mutt is configured
without PGP or S/MIME, and with GPGME, but $crypt_use_gpgme is unset.
In this case, no backend will be available, but WithCrypto will be set
with both APPLICATION_PGP and APPLICATION_SMIME bits.

That will allow various config vars to enable encryption or signing,
even though there will be no backend available to perform them.  The
message security flag might then be set, but when the user hits send,
will end up back at the compose menu due to the error.

The pgp or smime menu might not even be available to clear the
security setting!

Add a check in send.c before the compose menu is invoked, and give a
warning message for the menu ops inside the compose menu.

I believe this should prevent the issue.  However this is a corner
case combined with user misconfiguration, so I don't believe is worth
a large effort to completely eradicate.

Converted po/pl.po from ISO-8859-2 to UTF-8

I’ve converted the Polish translation file to UTF-8 using iconv 2.25.

Mention self_encrypt options in the documentation. (closes #3983)

Add a paragraph to the "Sending Cryptographically Signed/Encrypted
Messages" section of the manual.

Merge branch 'stable'

Update wiki link in manual.

Merge branch 'stable'