Make sure message didn't get truncated before sending command to FIFO

FIFO sending utilities must send NUL character at the end of commands

Without NUL character, calling utilities in succession has chances of
concatenating different commands in a single invalid one.

Address CVE-2018-17141 and fixes a few vulnerabilities in code supporting JPEG

These changes are adapted from Lee's fix for this vulnerability.

Luis Merino, Markus Vervier, and Eric Sesterhenn of X41 D-SEC GmbH
(Security Advisory: X41-2018-008) discovered an uninitialized pointer write
and also an out-of-bounds write in FaxModem::writeECMData() that could lead
to remote code execution with a specially-crafted fax sender.

These changes fix the coding errors and deliberately prevent malicious and
malfunctioning senders from inadvertently or deliberately setting JPEG and
MH/MR/MMR/JBIG formats in the same DCS signal.

Fix GCC-7 compiler error

Fix memory corruption when using libtiff v4.0 or newer

One tag was missed when adapting to
libtiff v4.0 uint64 data types.

Upstream libtiff documents the following tags as changed:
   TIFFTAG_FREEBYTECOUNTS
   TIFFTAG_FREEOFFSETS

   TIFFTAG_STRIPBYTECOUNTS
   TIFFTAG_STRIPOFFSETS

   TIFFTAG_TILEBYTECOUNTS
   TIFFTAG_TILEOFFSETS

Allow TSI to be specified in config file (and with X-FAX-TSI header)

serverconfig fixup: JobRetryOther + JobRequeueOther

Check faxanswer "how" parameter length...

The last line of a configuration file may not be terminated by a '\n'

Make sure the last character of a read line is '\n' before nuking it.

Set fax status on glare errors

faxsend: JobRetryOther/JobRequeueOther weren't actually being used

Update releasenotes and documentation re IPv6 support

Str: Add wchar_t support

Some uses of our HylaFAX Clients user wchar_t strings

Bug 949: notify man page does not mention TEMPLATE variable

The notify man page should mention that FaxNotify may set TEMPLATE variable in
order to change the language of the sent email.

This uses the same text as faxrcvd.1m.

Update release notes, prepare for moving to 6.1

Update releasenotes

Support libtiff 4.0

libtiff 4.0 changed a few things:
- no more TIFFDirEntry
- Moved to "Classic" vs "Big" tiffs, so renamed some things
- Moved to 64-bit types for bytecounts/offsets

hfaxd: Release old accept fd

By unlinking from the dispatcher early, we prevented it from being
closed in the child, leaving an open listening fd on the main hfaxd
port.

If the main (parent) hfaxd exits, the child will inheret the listen
but won't actually include it in it's selects, or handle it.  It just
prevents a new master hfaxd from binding to it.

hfaxd: Make source port for active connections be ctrl port - 1

Traditionally, active FTP data connections come from port-1 (4558 by default).
Even though firewalls now don't usually rely on that, but do active control connection
inspection to find ports being used, let's try and be as nice as we can to
non FTP-aware firewalls.

This was subtracting 1 before the bites swap, causing it to be 4303 on little
endian machines.

Noticed by Amin Mozafari.

man: Dialrules recursion

DialRules - recursively allow dialrules to reference others

This allows rules to reference other rules.  The names are looked up
at runtime, not parse time, so they can reference rules not yet
defined.  The expected usage would be to reference a "canonicalization" rule
(either CanonicalNumber directly, or through some indepenant rule) in
multiple places without needing repeats.

The replace format is of the form be \Name(...)

Example dialrules
    ! rules to canonoicalize numbers to +IDD format
    CanonicalNumber := [
....
    ]
    DialString := [
.* = \CanonicalNumber(&) ! Get the canonical
^[+]${Country}${Area} =  ! Local call
^[+]${Country} = ${LDPrefix} ! domestic long distance
^[+] = ${IDDPrefix} ! International
    ]

There is a recursion depth limit in this, just to avoid infinute rule recursion.

DialRules: Avoid abort on zero length matches

Pattern rules may contain 0 length matches, such as:
   (^[+])?([0-9]*)

We have nothing to replace it with (mlen = 0), so trying to extract
it from the string causes and abort.  If there is nothing to replace
it with, there is nothing to try extracting for insertion.

If the match reference was not matched in the string, StartOfMatch and
EndOfMatch both return -1, leaving mlen = 0, making it an ideal check
if there is something to extract and insert.

DialRules: Warn on invalid escape sequence

Instead of just ripping the \ out, make sure we know what the next
character is and warn if it's not a proper digit.

hfaxd: Eliminte extraneous debug logging

This code is well excersised, and all this logging in everybody's
syslog is just annoying.

Update releasenotes

Bug 934: We need to avoid a 0-index in playList

Combine all Fontmap files in memory, including new Fontmap.HylaFAX

Reads the Fontmap.HylaFAX created by faxsetup when existing Fontmap
is missing filename mappings. Combine all Fontmap files so that aliases
can now point to entries in other Fontmap files.

Use a private Fontmap.HylaFAX file of .pfb files

If the existing Fontmap files do not provide the information to
find the default font, creates a private Fontmap.HylaFAX file
that will list all pfb fonts found to their font names. No
aliases are generated by this and every Fontmap file is searched
for any alias.

Reworked how faxsetup looks for Fontmap

If a previous faxsetup had added Fontmap/Fontpath lines in
hyla.conf and the hardcoded Fontmap path is the same as the
installed Ghostscript search path, the previously added
Fontmap/Fontpath lines were left untouched even if they
were not valid anymore.

This patch now looks for a valid combination to set in hyla.conf
and will remove unneeded lines if necessary. It will even set
a default font (using a font filename) if the default "Courier"
font (or any font specified in hyla.conf) is not found using
the Fontmap files.

Do not warn about one of the Fontpath directories not existing

Fontpath usually contain multiple directories with one or more
not existing. Having faxsetup warn the user about this directory
not existing and having the possibility of causing problems just
confuse users. There is already a warning if no .afm file is
found in any of those directories. That should be enough.

typerules: adding missing comma to typeNames array

man: JobReqError/JobRetryError were missing in hylafax-config.4f

faxalter: Wire up the page range -Z options

Code for range was in place, but the Z: was left out of the option string.

Compiler warning cleanup

Class2: Fix hangup cause handling

This makes sur that errors on reception in Class2 aren't "missed", as seen in:
Date: Mon, 30 Aug 2010 16:20:51 +0200
From: Giuseppe Sacco <giuseppe@eppesuigoccas.homedns.org>
Subject: [hylafax-users] COMREC error (code 72), Normal and proper end of connection,
To: hylafax-users@hylafax.org
Message-ID: <1283178051.5043.21.camel@scarafaggio>

faxq: better capturing of failed conversion output

Fixes compilation on some platforms

At least on avr32 Linux with gcc, there was missing casts to a fxmin parameters.

Bug 928: issues with installClient target of Hylafax

make installClient needed:
    installDSO in libhylafax
    installClient in libhylafax
    a fix in the NLS syntax

pdf2fax/ps2fax: Use -dMaxStripSize=0

Ghostscript changed their default from 0.  Unfortunately, in 8.71, they used a
small 8K, which causes "blank images" when this is send in 2D-MMR right to the
client by faxsend assuming it's a single strip.

This can be overridden/changed using FaxModify.

manpages: Updates from debian

notify: FaxMaster notifications wildcard fixups

In the faxmaster notification case, * instead of "*" is needed to match everything.

From David Oxley @ D. E. Shaw & Co.

Fix calls to pdf2fax/ps2fax in ConvertFile

Jorn Dreyer noticed that the call to pdf2fax from ConvertFile
was incorrect in it's handling of the output filename.
The same problem affected the ps2fax call.

Correctly check for long lines when checking for the last match

Bodo Meissner found that textfmt would wrongfully return a line
too long error message in some circumstances and suggested a
fix for this.

Updated ftp site address in setup error message

Updates LibTIFF homepage URLs

Allow "any" as answer method for FIFO messages

"ANSWER modem" hfaxd command with no method sends "any" as the method.
faxanswer defaults to an empty method, but lists "any" as a valid method
in the man page.This method was not accepted by faxgetty. This adds the
"any" method as an alias to no method at all.

Typo in CONFIG_UNIXTRANSPORT define checks

Update releasenotes in preparation for 6.0.4-rc2

Class1: ECM page and block count should be sent as modulus.

faxsend: Class1 1 ECM when skipping pages needs to be more careful

pageNumberOfCall is not a generic number - it's used explicitly for putting the
page number in the ECM frames... This is ECM specific to Class 1, and needs to
marge forwared 1 every page sent in a call, regardless of job, skipping, etc.

hfaxd: log what address(es) we're listing on when starting

Re-order library files in fchmod() configure check

Newer gcc -E stops it's output on the first missing include file so we must
make sure that a possibly missing file in the list of headers to check is at
the end.

faxmail: Fix broken to address/name parsing

Update release notes for back branch releases

Str::vformat()  Don't reuse va_args

Use of the va_args is "destructive", and you need to use a copy
if you intend to re-use it.

Updated RPM dependencies

Moved the RPM dependencies to the binary packages and added an RPM build
dependency on gettext-tools for SUSE

faxq: Don't assert on bad FIFO messages

Bad fifo messages shouldn't get in, but if they do, we'ld do better
to just ignore them than assert somewhere deaper (in idToDev)

Add support for libtiff 3.9

PageHandling:  Fix handling of large blank areas

If the blank data at the end of a page is more than 64K, then it doesn't fit
into a 'Z%04x' field.  In reality, the only way you get more than 64K of blank
data is if the tiff encodes the blanks pixel for pixel, and *not* encoding.
Fortunately, the only program I've seen that does that is Microsoft FAX
service.

Unfortunately, if it is > 64K, we need to limit it, otherwise we end up
something like:
Internal botch; unknown post-page handling string "e1Z131b0P"

nls: Add German translation

clients: Add -O option

This adds a -O <option> syntax to the HylaFAX clients.  This is
similar to the -c <option> on the server counterparts.  Unfortunately
 -c is already used by many clients (and -O is used for other things on
the server parts).

This gives you the ability to set any config option on a per-invocation
basis, like:
faxstat -O 'JobFmt: %j,%s' -O 'RecvFmt: %m' -r -d -s

hfaxd/faxq: Add -c <option> to hfaxd/faxq as done in faxsend/faxgetty

This means all the "server" components of HylaFAX take -c option
arguments on the command line.  faxq, faxsend and faxgetty have
already done this, this gives hfaxd that ability too.  It's useful
if you need to start hfaxd with some option that you don't want
going in the config file.

This also documents the -c option in the rest of the man pages.

hfaxd:  Cleanup uid initizliaztion and theoretical overflow

This fixes a "techincal" array overflow:
FAXUID_MAX == FAXUID_ANON == 60002 == array size.
array[FAXUID_ANON] is out of bounds

It happens to not be one because our "bit array" doesn't end exactly
on long bindaries, to the out-of-bound by 1 is still in the allocated
"long array".

This also tightens up checks on acceptable uid values before being used
or assigned.

Revert "Bug 608: fixes for compilation on AMD 64"

This reverts commit 2b711903d33ea38717a1238f2277e8a3bdecbe44.

That commit changed the recvData API callback from:
  bool (*f)(void*, const char*, int, fxStr&)
to;
  bool (*f)(int, const char*, int, fxStr&)

Which is the exact *wrong* thing to do, and did it under the guise of
AMD fixes...  That warning is properly fixed here as well as bringing the
API back to the (void*) argument.

pagesend: Fix broken switch statement

Cleanup debian changelog

IPv6: Rework initial bind

From the getaddrinfo man page:
                    AI_PASSIVE      If the AI_PASSIVE bit is set it indicates
                                    that the returned socket address structure
                                    is intended for use in a call to bind(2).
                                    In this case, if the hostname argument is
                                    the null pointer, then the IP address por-
                                    tion of the socket address structure will
                                    be set to INADDR_ANY for an IPv4 address
                                    or IN6ADDR_ANY_INIT for an IPv6 address.

                                    If the AI_PASSIVE bit is not set, the
                                    returned socket address structure will be
                                    ready for use in a call to connect(2) for
                                    a connection-oriented protocol or
                                    connect(2), sendto(2), or sendmsg(2) if a
                                    connectionless protocol was chosen.  The
                                    IP address portion of the socket address
                                    structure will be set to the loopback
                                    address if hostname is the null pointer
                                    and AI_PASSIVE is not set.

We were doing this manually by memset-ing the sockaddr if bindaddress was
null, but this is the proper way to do it.

Don't duplicate any file in a release tarball

Update release notes

Page sent protocol trace fixes

If skipped pages are not counted, make sure to not trace skipped pages as
faxed and to use the same page number for sent pages as in the tagline.
(cherry picked from commit 9ffba03b5ba4f41999b6231c6a1850ccffcad8aa)

More errno cleanups

... in line with the previous fixes.
(cherry picked from commit 4904aeb538e9160fcdf279bb6bcf618a7d292c0b)

Signal handlers need to be careful with errno

Because other code relies on errno being set correctly from the
result of a previous syscall.
(cherry picked from commit baefdd9588a8169679d32be84c0a26f3523c3664)

More debug statements need to be removed
(cherry picked from commit 0b34b0a0932344e958bfbc0d0e36fa6d8ba6d13e)

Remove debugging statements
(cherry picked from commit 7808eecb32a4e2467e0358194eca9e892a403642)

Socket fixes for BSD 4.4 sockets

Giovanni Bechis reported things not working on OpenBSD, which uses the BSD
4.4 sockaddr struct which doesn't have sa_family as the first member.  So
we can't rely on a family being the first member of our union either.
(cherry picked from commit 9695174b15fa8839ec7203c4c6def1a1b87e5f5d)

FaxClient: Apparently <modem>@ is used and common

At least, we shouldn't segfault on it.
(cherry picked from commit ad4574469b885952423711080c77e7ac31c0c743)

Socket.h:  OpenBSD portability

Fix from Giovanni Bechis <bigionews@snb.it>
(cherry picked from commit effbbec24aadf058859990f3ac3114e9a8de6224)

templates: Update Polish translation
(cherry picked from commit e5e8c4b50fc895ba7111d1b4d0802b1865647d2d)

Adds RPM building right from tarball

Build an RPM with just:

  rpmbuild -ta path/hylafax-$VERSION.tar.gz

Based on the following commits:
 395986440c00af624cf729c199af9daa385e8792
 111a6c004d81fdd22de16e6072fde0f5a748dad3
 331c20be8a352fec904f841040dbc108010036d9
 90db29cc72feccea81db09ffd954efa0295c13f4

Cleanup distrules

Update list of files to distribute in the tarball and make sure
`make release.tar` will build a correctly named release tarball.
(cherry picked from commit 9d866b3ab4d325f890051fc8e7fa4cef83e33030)

Changed revision control name in version number

CVS HEAD is now 6.1-CVS

HylaFAX 6.0.0 is out.

Update releasenotes

configure/build: Cleanup LIBZ defaults

configure/build: Clean up REGEXINC

REGEXINC now matches other *INC variables

Remove unused DBLIB configure/build support

configure/build: Cleanup TIFFINC/LIBTIFF

TIFFINC now matches the other *INC variables.

configure: Beter INTL configuration

make distclean: clean up i18n po catalog stuff

[defs.in] Remove hard coded -L/usr/local/lib from GLDOPTS.

configure: Work on deprecated CXX Headers

C++ has had the <file> syntax for headers now for a long time... But,
some platforms are *still* using the <file.h> syntax.

[Makefile.in port/Makefile.in libhylafax/Makefile.in] port/version.c and
libhylafax/pagesizes are generated files that should be removed on a
"make distclean".

[pkg/postinstall.in] typo in cron example

[etc/faxaddmodem.sh.in etc/probemodem.sh.in] shell portability fix

notify: Don't send blocked notifications as "requeued"

Blocked jobs really aren't being requeued, they are just "in the queue",
like any other job limited by no devices, or modem groups, or delayed,
etc.

Fixup README

Update debian changelog

HylaFAX 6.0.0-rc1

Update releasenotes