Add gpg signature name. (closes #3478)

Some mail clients display "noname" for the gpg signature
attachment.  This patch sets the mime name to "signature.asc".

The patch is based on
http://dev.mutt.org/trac/attachment/ticket/3478/gpg_signature_name.patch
with the changes:
  - removes the configuration options
  - removes mime description

merge stable

merge stable

Change bounce to use mutt_parse_adrlist() for address parsing.

When prompting for recipients, forwarding, replying, and mailing all use
mutt_parse_adrlist() to parse the addresses.  This allows for addresses
to be separated by spaces.

Bounce message currently uses rfc822_parse_adrlist(), which doesn't
allow the spaces.  This one-line patch simply changes bounce to behave
the same as mail, reply, and forwarding for the address prompt.

Thanks to Roger Cornelius for the patch!

Fix segfault of extract-keys (^K) under gpgme. (closes #3698)

This patch is based on the patch by Ben Price, which relocated the
safe_fclose (&in) after its use by keydata in pgp_gpgme_extract_keys.
Thank you for the patch!

In addition, this patch:
  * removes spurious (debug?) output when the extract keys is finished.
  * adds a gpgme_data_release() call to free the keydata.

Fix segfault of extract-keys (^K) under gpgme. (closes #3698)

This patch is based on the patch by Ben Price, which relocated the
safe_fclose (&in) after its use by keydata in pgp_gpgme_extract_keys.
Thank you for the patch!

In addition, this patch:
  * removes spurious (debug?) output when the extract keys is finished.
  * adds a gpgme_data_release() call to free the keydata.

Revert write_one_header() to skip space and tab.  (closes #3716)

This patch fixes CVE-2014-9116 in the stable branch.  It reverts
write_one_header() to the pre [f251d523ca5a] code for skipping
whitespace.

Thanks to Antonio Radici and Tomas Hoger for their analysis and patches
to mutt, which this patch is based off of.

Fix segv in pgp_getkeybystr().  (closes #3725)

When searching for keys, and the user supplies "" to match against,
pgp_getkeybystr will have values:
  p = "";
  l = 0;
After returning from pgp_select_key(), it will try to assign to
p[l-1].

(As a note, the function is chopping off and restoring the trailing
"!" character because of ticket #1928.)

Fix a memory leak in mutt_query_complete.

When a single result was returned, it was written to the buffer and
returned, but the query result was never freed.

This patch creates a free_query function and changes the code to use
that everywhere.

mutt_sasl: fix double negative in iptostring result check (fixes #3705)

mutt_sasl: whitespace only

Fix variable name in configure.ac.

$ac_cv_search_STRINGPREP_CHECK_VERSION should be
$ac_cv_search_stringprep_check_version, to match the first parameter of
the AC_SEARCH_LIBS([stringprep_check_version] above.

Running configure was giving a "test: =: unary operator expected" error.

Remove duplicate SmimeCryptAlg declaration in globals.h

Thank you Elias Diem for the patch.

Wrap pgp_uid_t->addr in NONULL(). (closes #3564)

The previous patch introduced the possibility for addr to be null.  Mutt
is surprisingly robust against null strings, but there are a few places
that should be wrapped in NONULL().

Fix parse_pub_line to allow an empty User-ID field for a pub record. (see #3564)

A key whose primary uid record has an empty User-ID will result in the
user being unable to use the key to encrypt an email in mutt.  This is
because the mutt functions for key selection iterate through the address
fields of a key for matching against and for displaying to the user.

This change allows a pgp_uid_t record to be created for a pub record
whose User-ID field is blank.  So the key will have one address record,
albeit with a null addr field.

Updated French translation.

Updated Catalan translation.

Updated Ukrainian translation.

Updated Russian translation.

fix typo in man page.

Updated Czech translation

mutt-1.5.23 signed

Added tag mutt-1-5-23-rel for changeset b8a63730c848

automatic post-release commit for mutt-1.5.23

build-release: make sure version is picked up by autoconf

mkchangelog: only show log entries on current branch

No more devel

add user visible changes since 1.5.22 release.

Fix buffer overrun caused by not updating a string length after address expansion.

Override draft headers with arguments. (closes #3580)

Currently, if the -H option is used, it is not possible to redefine the
recipient addresses with positional argument. This patch fixes it as
well as CC, Bcc, and Subject headers.

This patch is based on the patch by Petr Písař at
http://dev.mutt.org/trac/attachment/ticket/3580/override_draft_headers_with_arguments.patch

This version of the patch merges the To, Cc, and Bcc commandline
arguments into the template, and fixes a few small problems with the
previous patch.

This patch also moves the fin close outside the "if (tempfile)" block,
since it is opened outside that block.

Enable %a format string for all pgp command configs. (closes #3699)

Currently, the documentation states that, with the exception of
pgp_getkeys_command, all the PGP format commands should be available to
the pgp_*_command configuration variables.

However, the %a format is actually only enabled for
pgp_clearsign_command, pgp_encrypt_sign_command, and pgp_sign_command.

Philip Rinn discovered this problem when he was trying to add a
"--encrypt-to %a" to pgp_encrypt_only_command.  Rather than just fix
the one configuration he was having the problem with, this patch
enables %a usage for all of them. (With the exception of the documented
pgp_getkeys_command.)

Note correct usage of $edit_headers. (closes #3673)

Ticket 3673 shows that it isn't necessarily obvious a blank
line is needed between the headers and body of a message when using
$edit_headers.  This patch adds a small note to the documentation,
indicating the blank line should be preserved between the headers and
body.

Thank you Sinan Kaan Yerli for the patch.

Clarify PGPPATH usage in manpages.

Make more clear that, when used with the original PGP program, mutt and
pgpring rely on this being set.

Fix a memory leak in mutt_query_complete.

When a single result was returned, it was written to the buffer and
returned, but the query result was never freed.

This patch creates a free_query function and changes the code to use
that everywhere.

Remove unused variable t in start_debug().

t was set but never used.  This was generating a compiler warning.

Fix variable name in configure.ac.

$ac_cv_search_STRINGPREP_CHECK_VERSION should be
$ac_cv_search_stringprep_check_version, to match the first parameter of
the AC_SEARCH_LIBS([stringprep_check_version] above.

Running configure was giving a "test: =: unary operator expected" error.

Remove duplicate SmimeCryptAlg declaration in globals.h

Thank you Elias Diem for the patch.

Disable SSLv3 by default.

Since Oct. 14 2014 SSLv3 is no longer considered safe. See POODLE:
https://en.wikipedia.org/wiki/Transport_Layer_Security#POODLE_attack

hcache tcdb debug logging

add check-syntax target for flymake

ignore generated pgpewrap/pgpring man pages

ignore gnu global tags

merge stable

merge stable

Fix deprecated gnutls_protocol_set_priority.  (closes #3515)

This patch is based off of the patch at
http://dev.mutt.org/trac/attachment/ticket/3515/624058-gnutls-deprecated-set-priority.patch

The mutt source has changed since then, with more TLS options being
added.  This patch therefore uses a slightly different strategy for generating the
priority string.

The patch also adds autoconf support, as requested by brendan at
http://dev.mutt.org/trac/ticket/3515#comment:1

Updated Dutch translation

Merge stable into default

remove trailing period from documentation for $pgp_list_secring_command

remove trailing period from documentation for $pgp_list_pubring_command

merge changesets from stable branch

Merge stable into default

Move HEAD to default

create stable branch

add note about use of ! ~Q in send-hook when paired with reply-hook.

remove <stdbool.h> and use "1" instead of "true".  closes #3657

see #3642 for the change that caused this bug.

update UPDATING with 1.5.22 release date

mutt-1.5.22 signed

Added tag mutt-1-5-22-rel for changeset 8c5ea668e7e0

automatic post-release commit for mutt-1.5.22

Rework logic to avoid advancing list pointer inside the for loop.

closes #3531

updated Catalan translation

updated Ukranian translation

updated Japanese translation

add missing files that were supposed to be included with [76f8df0ac80396559aa80e7f145be6f235a6db51]

adds some missing options in doc and adds man pages for binaries pgpewrap and pgpring.

closes #3643

Problems with muttrc.5:

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Unknown or invalid macro.  That is, one that does not fit in the
macro set that the man page seems to be using.  This is a serious
error; it often means part of your text is being lost or rendered
incorrectly.

Use local definitions of .EX/.EE or .DS/.DE to avoid low-level troff
requests in the page body.  There are plans to add these to groff man;
in the interim, this patch adds a compatible definition to your page.

Problems with mutt.1:

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Spelling error or typo.

include <stdbool.h> for definition 'true'

closes #3642

Fix segfault when viewing text attachments in compose menu.  (closes #3644)

The segfault was introduced in changeset b9f9e3147eb4.  Since decoding
and charset conversion aren't needed for attachments when composing a
message, this patch reverts to just using mutt_save_attachment() to view
"raw data" for text attachments in the compose/send case.

This patch is based on Michael Elkins' patch at
http://dev.mutt.org/trac/attachment/ticket/3644/view_attach_compose_segfault
with just a missing return value check added.

Ensure that BUFFER variables are initialized to prevent later attempts
to traverse an uninitialized pointer.

Consistently use mutt_buffer_init function

Despite the existence of the mutt_buffer_init function, most
initializations were instead done with memset.  Use the specific
function instead to make it easier if later changes are made to how the
initialization should be done.

When recalling a postponed message that does not have a x-mutt-fcc header
field, generate a default FCC just as we do for new messages.  If x-mutt-fcc is
present but empty, we assume that means the user does not want a copy.

closes #3653

Ensure that at least one signature was generated to catch the case where $pgp_sign_as is unset and no default key is specified in ~/.gnupg/gpg.conf

closes #3658

use LOFF_T as the type for variables used in fseeko()

closes #3487

Display the KeyID when the pubkey is not found in the user's keyring during signature verification.

closes #3654

return 0 if import of cert failed to match other error conditions.

only terminate the write-half of the TLS connection to avoid hanging if the remote doesn't respond.

RFC5246 section 7.2.4 states that the initiator of the closure need not wait for the remote to response before closing the read-half.

closes #3633

fix more comment typos

fix typos in comments

Fix postpone/resume to not remove a Mail-Followup-To header  (closes #3070)

This solution changes mutt_prepare_template() to check whether the
message_id field is NULL to decide whether to wipe the message-id and
mail-followup-to headers when instantiating the message.

If we are resending a message, we don't want the previous message-id
and mail-followup-to headers.  If we are resuming a postponed message,
however, we want to keep the mail-followup-to header if any was set
before the postpone.
(grafted from d7d83298011a6e7aa31bde49e0b6b21209a85fd2)

version.sh: step back from $() to ``.  closes #3661
(grafted from 01cf5814dfe5e79b65b1d950aab1276d19a9f031)

Allow filename prompt when saving multiple attachments to a directory. (closes #3083)

Currently, if you specify a directory when saving multiple attachments
and choose (a)ll, the first attachment is saved without confirming the
filename.  Subsequent attachments prompt for the filename.

Fix postpone/resume to not remove a Mail-Followup-To header  (closes #3070)

This solution changes mutt_prepare_template() to check whether the
message_id field is NULL to decide whether to wipe the message-id and
mail-followup-to headers when instantiating the message.

If we are resending a message, we don't want the previous message-id
and mail-followup-to headers.  If we are resuming a postponed message,
however, we want to keep the mail-followup-to header if any was set
before the postpone.

UPDATING blurb for terminal status features

version.sh: step back from $() to ``.  closes #3661

Terminal status line support, based on the xterm title patch.

Christoph Berg took the xterm title patch from the mutt mailing list and
maintained it for Debian:
* Changes made:
  - 2007-01-27 myon: using %P caused a segfault, updated status.c to catch
    menu==NULL.
  - 2007-02-20 myon: make the note about the xterm_set_titles defaults a
    comment.
  - 2008-08-02 myon: move set_xterm_* prototypes into the proper header file
    (cleaner code, no functional change, evades conflict with sidebar patch)

This update reworks the Debian version as generic support for tsl/fsl
terminfo capability.  It no longer depends on a static list of supported
terminal types, although it will use default support for these terminal
types *if* they do not advertise tsl/fsl or XT in terminfo.

The generic terminal status line can be implemented in any fashion, and
on hardware terminals it can be a special out-of-screen region of the
display.  Most modern terminal emulators appear to emulate the status
line in the window title; thus the notion of the tsl escape as an xterm
window title escape.

Configuration variables have been renamed:
$ts_status_format
    format string (using $status_format codes) for the terminal status line
$ts_icon_format
    format string (using $status_format codes) for the xterm icon name
$ts_enabled
    toggle for whether to issue status updates

The old configuration names exist as synonyms.

Logic flow:
* If tsl (string) is set in terminfo, mutt will use that escape to switch
  to the status area, and will use fsl to switch back to the regular
  display area.
* Otherwise, if XT (boolean) is set in terminfo, mutt will use the
  standard xterm-compatible escapes.
* Otherwise, if $TERM matches a known terminal type, mutt will use the
  standard xterm-compatible escapes.
* Otherwise, this feature is disabled.

= NOTE =

The XTerm icon escape sequence is not formalized in terminfo, so the
egregious kludge of hardcoding the escape is necessary in this case.
However, some terminal emulators (e.g. MacOS X Terminal.app) will set
the status line (window title) when the icon sequence is received.
Since there is no way to detect this behavior, the only solution is to
write your configuration to take it into account.  Rather than add a
variable to control icon escapes separately, we just provide the advice:
if you use such a terminal, you may wish to set ts_icon_format to an
empty string.  This will cause it not to be used.

Don't exit pager if quit=ask is answered "no".

This patch addresses ticket 2788.

Currently if you quit ('Q') from inside the pager, with quit=ask-yes
or quit=ask-no, and you respond "no", mutt leaves you back in the index
menu.

This patch changes it so that mutt does not exit the pager.

backout c1371176ea45

Backed out changeset 1142ed8974fa

use gethostname() to determine the system host name

use getaddrinfo() to look up canonical DNS name, and fall back to hinting from /etc/resolv.conf

see #3298

fix various compiler warnings; most were due to unchecked return values from system calls.

use mkdtemp() to create temporary directory rather than mktemp() followed by mkdir()

closes #3637

use of sscanf() had undefined behavior, replace with simple parsing instead

see #3636

increase string size by one byte to accomodate the terminating null character in fscanf() since the maximum field width does not include it

closes #3636

hat tip: Will Fiveash

fix potential buffer overflow in off-by-one bounds checking error

closes #3635

explicitly call mutt_refresh() prior to endwin() to force a screen redraw.

[4903e80356d3] removed the call, but some environments (screen+xterm under
SuSE11/12) don't properly flush without it prior to the endwin() call

do not use mutt_message() to display prompt in mutt_yesorno().

properly calculate screen width of prompt string by using mutt_strwidth() rather than strlen()

use safe_asprintf() return the formatted yes/no string since specifying COLS+1 is wrong when dealing with multibyte characters

closes #3412
closes #3352

add support for $mail_check_recent to mh mailbox driver

closes #3629

do not explicitly reset the terminal attributes and move the cursor in mutt_endwin().  endwin() itself should handle this.

closes #3627

remove PKG_CHECK_MODULE() check and always use AC_CHECK_HEADERS()

only check for <idn/idna.h> when <idna.h> isn't found

closes #3626