Merge branch 'master' of ssh://marmaro.de:443/var/git/mmh

Merge remote-tracking branch 'origin/master'

Bring back the LINUX-IO check from the last commit

We need them for m_getfld(), too.

This reverts parts of the commit 33516e5b74bb4bb5c626d4ee42bd3a7a9faaabef.

Drop register storage class

Remove discard() total

discard() isn't needed in mmh anymore.

Fix autoconf check for mailspool world write permission

This is a bugfix and improvement for commit b15b7cc (``Use
realpath to find the mailspool'').

We can even do this check much simpler, if we focus on the
problem that

        $lspath -dlL $mailspool

does not check the target of a symlink, although -L is given.
The contradicting -d seems to overrule it.

We can fix this more easily by appending a slash to the path:

        $ ls -ld /var/mail/
        drwxrwxr-t 2 root root 4096 Apr 29 06:36 /var/mail//

Without the slash it was:

        $ ls -ld /var/mail
        lrwxrwxrwx 1 root root 10 Nov 22  2011 /var/mail -> spool/mail/

This small change does it for one level of indirection.
This should be sufficient.

Ignore empty MIME parameter

If a message contains an empty parameter, we only produce a warning.

Ignore wrong encoding in multipart/*

If a multipart message has an encoding other then
7bit or 8bit, 7bit encoding is assumed.

remove dublicated code in uip/spost

To avoid bugs like in commit dc81afc84b4d53177a1c4

fix bug if multible mbox in From-field

Now the first address of the user is used, if multible addresses
are in the From-field. If non of the addresses is an address
of the user, the the behaviour has not changed.

bugfix for a memory-reuse bug spost and whom

spost and whom use add to initialize a temporary char*.
This pointer maybe points to internal data or already freed
memory. Now getcpy is used.

spost: Refactor braces, comments, etc.

gitignore: More specific ignore patterns

I frequently make temporary backups this way:
cp uip/foo.c uip/foo.c.1
These files were ignored, although this ignore filter was meant
for the created man pages in section 1 only.

Add "Default-From" config option

The "Default-From" config option is used, if the Draft
don't contain a From header. If the draft contains a From
header and it's not from the user it's used as the Sender
header.

Bug: If the From-header contains more then one addresse
and one is from the user, no Sender-header is attached.

Use realpath to find the mailspool

inc is istalled with setgid-flag if the mailspool
isn't world writeable and dot-locking is used. This
work now if the mailspool is a sysmlink.

Use sysexits.h for better exit-codes

The sysexits header describe some standard exit-codes, which should
be used. adios has now a status argument so sysexits.h can be used.

reinclude some headers

These includes are necessary to compile on FreeBSD.

Move #include from h/mh.h to source files

Included files should not include further files. Now, only the
necessary files are included and not all of them in every file.

A large bunch of this work is done, but there are some more
nested include files.

Remove h/nmh.h

Move its contents to h/mh.h.

Remove the NLENGTH macro on the way, be cause it is not used.

Minimize the include

It's not necessary to include <h/signals.h>, <signal.h> is enough.

Fix FTBFS bug introduced by 600379c

One #include of h/signals.h is still necessary.

Removed unnecessary #include of h/signals.h.

Fixed the error message in lockit() when it fails to create tmp file.

Removed m_chkids(), per suggestion of Marcin Cieslak.

Removed #include of signal.h because it's in h/signals.h.

Replaced setgid() calls with setegid() so that it works with dot
locking on FreeBSD.  setegid() should be supported on modern POSIX
systems.

remove declaration of child_id in slocal:usr_pipe

Now the global version is used, so the signal handler
works correct.

return -1 if fork(2) fails in execprog

If fork in execprog fails and return 1 there is now
way to different between a proces witch return 1.
Thanks Marcin Cieslak <saper@saper.info> for reporting.

remove longjmp from signal handler in mhl

To avoid undefined behavior set an interrupt flag
in the signal handler.

remove longjmp in signalhandler of slocal

To avoid undefined behavior kill the child process
in the signal handler and set an interrupt flag.

remove longjmp in signalhandler of prompter

to avoid undefined behavior use close(2) in the signal handler

remove longjmp from signalhandler in getans

To avoid undefined behavior use close in the signal handler.

nonstatic function for atexit, check if atexit fails

atexit needs nonstatic functions to work correct. The
error check is importent, because atexit can fail for
some reasons and the cleanup woulden't work.

Merge branch 'master' of ssh://marmaro.de:443/var/git/mmh

Conflicts:
uip/mhl.c
uip/mhlist.c
uip/mhmail.c
uip/pick.c
uip/send.c

Refactor: Use if for two-way branching; Compare against 0 explicitly

I think that the use of switch for the two-way branches could have
been a performance optimization. I think it's clearer to use if,
especially in the fall-through case.

As setjmp() is known to return with either 0 or something else, we
should compare against 0 and not against some define (for which I
again and again wonder if it stands for 0 or for 1).

Fix minor coding style issues (whitespace)

Remove unnecessary headers

distout and ready_msg return NOTOK on error

If distout or ready_msg fails, the cleanup of send is done.

mhmail: use execprog()

sbr/execprog.c: Fix the return value

Use pidXwait(), which invokes pidstatus(), here for error reporting.
This eases the use of execprog() because we don't have to call it
there everytime. However, it might be unwanted to put the error
printing stuff into execprog() because we might have an invokation
of execprog() that want's to deal with errors itself in a different
way. If this would be the case, then we likely should change
pidXwait() to pidwait() and call pidstatus(execprog()).

Minor clarifications in mhmail(1)

Update the man pages to the shortened reserved sequence names

These reserved sequence names were shortened to one character
long ago in the code. Now the docs are adjusted to that change.

Thanks to Marcin for spotting it.

Initialize status in uip/send.c

Thanks Marcin Cieslak <saper@saper.info> for reporting.

Merge ../mmh

Conflicts:
uip/rmm.c
uip/whatnow.c

remove done finaly

removed the used of done in sbr and the dafinition
in the header. Also the source file is removed.

remove last rests of done in uip

At some points done was used as callback funktion.

Remove done from mhl pick and send

Second part of removing done, to make the code
better to read. This tools needed some refactoring
before done could removed. Next step is removing
done from sbr.

Initialize vecp in send anno()

Access to an uninialized variable can cause
a a buffer overflow or underrun.
(Thanks to the compiler for reporting.)

Add tests for pick

Add two patches for pick. One to test the normale behavior,
and one to test if a ``0'' is print to stdout when no message is
find.

Bugfix in pick.c

Undo last last commit in uip/pick.c. pick shoud only print "0" if
it exit with a non-zero exit.

Replace done with exit at uip

Replace done with exit in most sourcefiles of uip. This makes the
code better to read. If the done was overwritten by an externel function
atexit() is used instand.

send.c and mhl.c are excluded, because need a bit rewrite first.

new: Fix a pointer problem

I should learn to get such stuff right without the need to draw
the memory layout with boxes and lines on paper ...

Thanks to the test suite for catching this one!

send: Fix uninitialied variable

whatnow: Use execprog() ... one more time

whatnow: No need to fork

Plus refactoring.

sbr/ext_hook.c: Use execprog(); Refactoring

rmm: Use execprog() instead of own fork-exec

Remove unused variable

Convert pidXwait() from macro to function

Plus some refactoring and syntactic sugar in pidstatus().

Don't save the context within execprog()

This context_save() call was copied there accidently.

send: Convert system() to execprog() ... second half

Now only uip/spost.c has one left.

send: Convert system() to execprog() ... partly

forw: convert system() to execprog(); fix buffer check

Cleanup of code layout only

Refactoring in the small

Out-sorting failure cases instead of deep-nesting the success case.

Remove verbosity for -noverbose; cleanup in code layout and message texts

We shouldn't print information messages if we run non-verbosely.
In this case, the message appeared not to be helpful because it
suggests to re-run the command with `-verbose' ... but sending the
message again is usually not possible, not wanted or would produce
different results.

This is a cleanup of Philipp's patch.

Use execprog() instead of system() ... partly transition

There are several more places where system() should be replaced.

As execprog() does not split strings into multiple arguments,
`listproc' could no longer contain ``show -file''. We need to
rework the concept of these `...proc' entries.

This commit fixes tests/whatnow/test-attach-detach.

Added execprog() and execprogl(), which are safer alternatives to system(3)

This is motivated by the currently failing test case
tests/whatnow/test-attach-detach.

mhsign: use the strongest, newest not-expired secret key

Is there no GnuPG option to exclude expired keys from the listing?

Fix manpages (nroff code), as reported by tests/manpages/test-manpages

Partly fix tests/whatnow/test-attach-detach: changes in detach and alist

Whatnow's detach and alist commands are different today:
- detach does only take number arguments
- alist prints full path names

Fix tests/mhbuild/test-forw: rfc934 mode is no longer supported

Check with witch exit-code sendmail/spost exit and give an better error
message. So the User sees better wat's wrong. At the moment only
exit-code 75 is handlel extra.

Adjusted tests/new/test-basic to mmh.

mhparam: Renamed some sequences
We should fix these names once and for all!

tests: Adjusted most of the tests to mmh.

tests: Supress progress meter if non-interactive.

Added more include directories to fix building in another directory.
I discovered these problems when I tried to build from a git clone
(which has no config.h in the source directory).

Fixed the test framework by adjusting it to mmh.
In the same go: Cleaned up its code a bit and added a summary to
test/runalltests.

Adjusting the individual tests to mmh still needs to be done. ... and
afterwards we can ;dd a lot more of them. :-) (But before we write our
own tests, we first incorporate all the ones they've added to nmh in
the meanwhile. ;-) )

gitignore: ignore test dir name file

uip/mmh.sh: minor cleanup

Fixed building in different build directory, as described in INSTALL.

Merge

mhsign: Ignore expired, revoked, invalid, and similar public keys.

mhparam: Added more items for direct query and -debug listing.
Most new values are not included in the -all listing because they
are not part of the profile or context but are mainly internal
configuration values.

The -debug listing has now section headings (pipe through
`grep -v ^#' to remove them).

The motivation for this change was querying the default inbox
name. If `Inbox' wasn't set in the profile, `mhparam inbox' had
failed. Now it prints `+inbox' (if the defaultfolder has not been
changed on your installation).

mhpgp: Bugfix for 2932d136debcf53a45c44891ee80e6d34e61f109
(I should do better testing ...)

mhpgp: remove unused sed label
OpenBSD complained about it.
Reported by Dominik Lang <phil@phildom.de>. Thanks.

pick: Clarified the man page with respect to -ddd.
-ddd is an argument to e.g. -after, not a separate switch.

mhsign: Bugfix: Don't ignore MMHPGPKEY and the Pgpkey profile entry.
Bugreport and patch by Philipp Takacs <philipp@bureaucracy.de>. Thanks.
Additionally: use `test -z'.

Merge branch 'master' of ssh://marmaro.de:443/var/git/mmh

spost: If we add a From header, we add a qualified email address.
Also clarified a comment and changed the description for blind
distribution lists to the one that is most common today.

TODO: removed the double-dash TODO item, because it will not be done
The GNU style double-dashes collide with MH's command line concepts.
It makes no sense to simply allow a second dash, because all the
rest is different. (Think of `pick --component'.) It's more important
to keep the concepts clean than trying to imitate different concepts.

rmm: Use `refile -src' instead of a folder changing hack.
This is a cleanup for a ``hackish'' solution.
Thanks to Philipp Takacs <philipp@bureaucracy.de> for discovering this
improvement possibility and for preparing a patch.

Fixed a possible uninitialized variable access.
(Thanks to the compiler for reporting.)

coding style: Added braces even for one-line blocks
This commit changes only uip/rmm.c. It would be tedious to
change it everywhere. We can do this bit by bit as we go ...

Converted msgstats macros & Co. to functions and added range checks
This change is a reaction to bug, fixed in changeset 5b093c3. Although
the bug was fixed on a higher level, it would have been detected much
earlier if we would have had the range checks that we now have.

This patch was prepared by Philipp Takacs <philipp@bureaucracy.de>.

send: Check the draft folder only if we send msgs, not when sneding files.
Send can send files (specified by Unix path names) or messages (specified
by MH message names/sequences. (NB: It cannot send msgs from other folders
yet.) In case we've only specified files (this is done by whatnow, for
instance) we don't need to care if there is a draft folder and if it
contains any msgs.
This commit probably fixes a problem discovered by <philipp@bureaucracy.de>.

stringdex(): Clarified what it does
`stringdex' stands for `string index'. (See Changlog for why the `g'
was added.) This function is different to strcasestr() ... in the
order of the arguments and in the return value.

uprf: We don't have to implement it ourselves but can use strncasecmp()
Also clarified what an ``unsigned prefix'' really is.
I haven't changed all uses of uprf() in the code to strncasecmp(). This
could be done. I'm not sure if it would enhance the clarity of the code.
We don't need a prototype for strncasecmp(), because it's declared in
strings.h.

Use trim() to strip whitespace from the end of header fields
The main motivation was to ease adding attachments. You'll easy happen
to have trailing whitespace there, whereas you'll probably never want
to send files that have a name with whitespace at the end ... at least
you should never want to have or even send such files.
I don't know if it is a good idea to strip the trailing whitespace
in `anno -list' as well. I couldn't come up with an example when I
would want to keep it, hence I decided to remove it. This can be
changed back, if needed.