tidying: coverity issues

tidying: coverity issues

Utilites: fix exim_lock

Unaccountable fails to lock, seen in testsuite
Fix appeared when tidying a Coverity "unchecked returncode" fault

Fix build for Solaris: "all" must be the first make target.
Broken-by: dfe7d917154a Bug 1578

Docs: cuttthrough logging

Doc: Minor clarifications about the environment

tidying: coverity issues

Testsuite: fix output for 615, missing spool dir

Don't issue env warning if env is empty

keep_environment needs to be mentioned in the runtime config.
Setting add_environment isn't enough to suppress the warning.

Honour the -n for -bP config

Fix typos.

Thanks to Sander Bos for pointing out.

Doc: clarify -n

Extend configure.sample and the relevant spec part

String expansions: fix ${extract }, for the numeric/3-string case.  Bug 1807

Broken-by: 82dbd37
In "skipping" mode when parsing an expansion we want to avoid expanding
the arguments, as the data for expansion is not necessarily valid.  This
bit us previously for an extract within an "if inlist". But the number of
args for ${extract } depends on the expanded value of the first arg.

Retreat from strict parsing and just line up the outer braces,
accepting any number of args while skipping.  The separate
non-skipping execution will do the proper checking.

Test the return of getcwd()

Store the initial working directory, expand $initial_cwd. Bug 1805

Remove trailing \n from {keep,add}_environment config warning

Fix build for HP-UX and older Solaris: (un)setenv.  Bug 1578

Broken-by: bc3c7bb7d4ab

tidying: coverity issues

Docs: typo

tidying: coverity issues

tidying: coverity issues

Coverity: attempt to quieten null-deref whines about stringhandling

tidying: coverity issues

typoes

tidying: coverity issues

tidying: coverity issues

tidying

Cutthrough: Reflect 5xx recipient reject from target to originator

When connection not opened by verify and target hard-rejects a RCPT,
the reject was not being passed to the originating system (just the
cutthrough connection was being dropped).  Fix this.

Testsuite: testcases for rejected rcpt-callout-triggered cutthrough

Cutthrough: fix operation under -bhc to not actually deliver.  Bug 1800

Solaris /bin/sh does not understand 'export LC_ALL=C'

Delay chdir(/) until we opened the main config

Tidying: Issues detected by gcc --fsanitize=undefined

Merge branch 'master'

Use right type casts in string_compare_by_pointer

Remove confusing #ifndef environ

Testsuite: Adapt to portability fix in environment.c

Make qsort() in readconf.c more portable

Make environment.c more portable

Release: fix release script

- accept minor version number
- allow to skip the build-of-documentation step
- allow release of "any" version from anywhere
- avoid calling "old" reversion scripts, create version.sh

Removed doc references to relay-test.mail-abuse.org

As per github ticket #37 - https://github.com/Exim/exim/issues/37

relay-test.mail-abuse.org is no longer available so references to it have been removed from the documentation.

Fix CVE-2016-1531

Add keep_environment, add_environment.
Change the working directory to "/" during the early startup
phase.

(cherry picked from commit 2b92b67bfc33efe05e6ff2ea3852731ac2273832)
(cherry picked from commit 14b82c8b736c8ed24eda144f57703cb9feac6323)
(cherry picked from commit 9ca92d0c6e9c6f161bd8111366c6952d3a9315e2)
(cherry picked from commit 0020c6d9ecfd98ed7b2b337ed4f898fdc409784b)
(cherry picked from commit e8f96966360ea8867ad6a8b5affda6c37fa4958c)
(cherry picked from commit ef6fb807c1e1a665f444f644c60c77269f7c5209)

Docs: clarify unit of S= log line element

Compiler quietening

Malware: fix error logged on a scanner connect fail.  Bug 1796

TLS: support build with OpenSSL 1.1.0   Bug 1771

Doc: correct minor typo

DKIM: fix selection of header for signing/verification given several.  Bug 1792

DKIM: support oversigning.  Bugs 1309, 1310

Testsuite: Support running exim under valgrind

Fails mostly thanks to lack of suid handling, but -be tests usable
if you retry after de-suid of eximdir/exim

Remove empty lines from .mailmap, fix root

Add .mailmap for git

Testsuite: avoid retry db issue after deliberate fail subtests

Testsuite: avoid assuming 127.0.0.2 is a viable sending address

Testsuite: allow for different data arrival in SMTP synch check

Testsuite: log some stderr output on bad exit-code in --CONTINUE mode

SRS: fix crash in queryprogram router when compiled with EXPERIMENTAL_SRS

TLS: Whine to log on client config of SNI under too-old OpenSSL version

Testsuite: enforce different exim/testuser group numbers

Testsuite: sort logs in cases where we do parallel deliveries

Fix EXPERIMENTAL_DMARC build

Broken-by: df3def24

Testsuite: nuke retry db after deliberate-fail subtests in ocsp testcase

Add backward compatibility for EXIM_TMPDIR

Rename build-time option TMPDIR to EXIM_TMPDIR

Build-time option TMPDIR included in Makefile clashes with environment
variable of the same name. This breaks tools that make use of that
variable, such as distcc.

The following example demonstrates what's going on:

$ cat Makefile
TMPDIR="/tmp"
all:
env
$ export TMPDIR=test
$ make | grep ^TMPDIR
TMPDIR="/tmp"

distcc error (note the extra quotes):
ERROR: can't use TMPDIR ""/tmp"": No such file or directory

Testsuite: case should not depend on build-option

Testsuite: (more) complete usage message for server

Use GCRYPT if GNUTLS isn't good enough

Testsuite: output library versions during startup

Docs: add note on costs of cutthrough delivery in local-rejection cases

Testsuite: 3454: debian8 flavour

Testsuite: 3450: debian8 flavour

Docs: note DKIM signing options in smtp transport chapter

DKIM: replace SHA and RSA routines from gnutls, under earlier library
versions, using libgcrypt and libtasn1 directly.  Bug 1772

Keep options ordered alphabetical

GEF 20160130 Tiny corrections to Readme.pod

GEF 20160130 Changed date in header in test/README

Fix typo

Docs: clarify command string content in ${readsocket }

Fix typo on ChangeLog

Docs: clarify use of $dkim_selector variable

PDKIM: Fix use of private-keys having trailing '=' in the base-64.  Bug 1781

Cutthrough: Fix bug with dot-only line

Testuite: tidying

Copyright dates 2014, 2015

via:  vi $(git whatchanged --since=2014-12-31 --until=2016-01-01 | grep '^:100' | sed 's/^[^M]*M//' | grep -v 000000 | sort -u | fgrep -v test/)
(etc)

Change notes

Testsuite: bounce_return_linesize_limit.  Bug 1760

Tidying

Testsuite: changes for VFRY update, 4f6ae5c314e5

Restrict line lengths in bounces.  Bug 1760

VRFY: Permit an ACL to override the default 252 response, to support
verify-by-ACL instead of the more usual verify-by-routers.  Bug 1769

Tidying

Tidying

Expansions: avoid releasing memory used for $value in ${run }

Testsuite: ignore optional OCSP output from -bP testcase

Expansions: more detail in error messages

OpenSSL: Default the SINGLE_DH_USE option flag set

Docs: add note on HELO rejections, and add requirment on good HELO in
the example configuration

Expansions: Fix crash in crypteq: On OpenBSD a bad second-arg
results in an error-return from crypt().  Errorcheck that return.

Testsuite: split out conf for -bP test and lose dependency on Proxy/Socks