Add regress test for bug #623

Cope with a comment on the last line of the file with no newline.
Bug #623

Include arpa/inet.h for HP-UX; from Daniel Richard G.

Add missing $(mansrcdir) to visudo.mdoc and visudo.man.
From Daniel Richard G.

In v{warning,fatal}x?() make a new copy of ap for the debug functions.
It is not legal to use ap twice without reinitializing it.
Noticed by Daniel Richard G.

Remove errant warning_restore_locale() call.

Move va_copy compat macro to missing.h

Uniquify header dependencies so we don't end up with duplicates
when a header file includes other headers.  The header dependencies
are sorted so the generated order is stable.

Add getaddrinfo.lo to LTLIBOBJS for systems that need it.
From Daniel Richard G.

Fix pasto

Fix typo.

regen

Fix warnings from -Wold-style-definition

Add -Wold-style-definition to --enable-warnings

Extra debugging for ready fds.

When deleting an event, check ev->events to determine whether to
remove from readfds or writefds instead of blinding removing from
both.  Also fix highfd adjustment.

Only check an fd that is >= 0.  Timeout-only events may have a
negative fd.

Don't call sudo_ev_{add,del}_impl() for timeout-only events.  This
makes it possible to pass sudo_ev_alloc() an fd of -1 for events
only use SUDO_EV_TIMEOUT.

Make a copy of readfds/writefds before calling select() instead
of calculating it each time.  Keep track of high fd in the base.

Add Stephen Gelman

Fix sign comparison warning.

Fix potential NULL dereference in non-interactive mode.

Use MSG_WAITALL when receiving struct command_status over the Unix
domain socket since we no longer use datagrams.  This should avoid
the need to handle incomplete reads, though in theory it is still
possible.

SIGKILL is not catchable

Add sudo_ev_get_timeleft() to get the amount of time left before
an event times out and use it in sudoreplay.

If the user presses <return> or <enter> in sudoreplay, skip to the
next event.  Useful for skipping past long pauses in the data.

Fix sudo_ev_scan_impl() return value in event_poll.c.
Make sure we clear active flag from unprocessed events if
sudo_ev_loopbreak() or sudo_ev_loopcontinue() are used.
Remove bogus optimization when the timeout is zero or negative; it
could prevent an I/O event from being triggered.

Move session replay into its own function.

Get rid of cur and pending pointers in struct sudo_event_base.  We
now pop the first event off the active queue instead of using a
foreach loop with deferred removal of the event.
Add SUDO_EVQ_INSERTED and SUDO_EVQ_TIMEOUTS flags to indicate that
the event on the event queue and timeouts queue respectively.
No longer need to compare the timeout to {0,0} or compare the
event's base pointer to NULL to determine queue membership.

rename sudo_ev_loop_impl() -> sudo_ev_scan_impl()

Add support for libevent-style timed events.  Adding a timed event
is currently O(n).  The only consumer of timed events is sudoreplay
which only used a singled one so O(n) == O(1) for now.  This also
allows us to remove the nanosleep compat function as we now use a
timeout event instead.

Now that sudo_ev_base_free() removes all events before freeing we
don't need to do this by hand.

Add a list of active events in the base that the back end sets when
it calls poll or select.  This allows the front end to iterate over
the events instead of having that code in both back ends.  It will
also simplify support for timeout events.  Also make sure we can't
touch freed memory if a callback frees its own event.

Remove any existing events before freeing the event base.

mon_handler() should be static

If user specified start_tls and ldaps, display a warning and ignore
start_tls.  There's no reason to make this a fatal error.

Add missing else when the connection from the monitor to the parent
sudo process is broken (due to the parent dying).  Prevents a
spurious "unexpected reply type on backchannel" warning.

When flushing output we don't care whether we are the foreground
process or not, we still need to flush to /dev/tty.  If we are in
the background, it is OK to get SIGTTOU.

Should not attempt start_tls on an ldaps connection.

Fix sign compare warning.

Eliminate warning about circular dependency from GNU make.

More sign compare fixes.  On Solaris id_t is signed so use uid_t
in the set_perms.c ID macro instead.

Quiet sign comparision warnings.

Add -Wsign-compare to --enable-warnings

Ignore SIGPIPE when connecting to the LDAP server so we can get a
proper error message with the IBM LDAP libs.  Also return LDAP_SUCCESS
instead of 0 from most sudo_ldap_* functions that return an int.

Quiet compiler warnings.

sudo_ldap_parse_uri() should join multiple URIs in the string list
together but it was clearing the host entry each time through the
loop.  Fixes a bug with multiple URI entries in ldap.conf where
only the last one was being honored.

Avoid a double free introduced when plugging a memory leak in
safe_close().  A new ev_free_by_fd() function is used to remove and
free any events sharing the specified fd.  This can be used after
safe_close() to make sure we don't try to select() on a closed fd.

Quiet some llvm check false positives.  The common idiom of using
TAILQ_FIRST, TAILQ_REMOVE and free in a loop to free each entry in
a TAILQ confuses llvm.  Use TAILQ_FOREACH_SAFE instead (which is
probably faster anyway).

If pam_open_session() fails don't call pam_getenvlist() with a NULL
pam handle.

Fix newly introduced use after frees found by llvm checker.

Remove an errant list_next() call that should have been removed
in the TAILQ conversion.

Add "headless" tail queues and use them in place of the semi-circular
lists in sudoers.  Once the headless tail queue is built up it is
converted to a normal TAILQ.  This removes the last consumer of
list.c and list.h so those can now be removed.

Use SLIST and STAILQ macros instead of doing headless singly linked
lists manually.  As a bonus we now use a tail queue for ldap.c and
sudoreplay.c.

Convert sudo to use BSD TAILQ macros instead of home ground tail
queue functions.  This includes a private queue.h header derived
from FreeBSD.  It is simpler to just use our own header rather than
try to deal with macros that may or may not be present in various
queue.h incarnations.

Fix AND operator broken by changes to fix OR.

Fix OR operator.

Fix memory leak of I/O buffer events in safe_close().

Don't allow the debug subsystem to be initialized twice.
Otherwise we can exhuast our stack when built in static mode.

Make sure we do not try to usie index -1 in base->pfds[].

Bump version to 1.8.9

Convert the monitor process to the event subsystem.

Convert the main sudo event loop to use the event subsystem.
Read events for I/O buffers are added before the loop starts.
Write events are added on demand as the buffers are filled.

Simple event subsystem that uses poll() or select().  Basically a
simplied subset of libevent2.  Currently only fd events are supported
(since that's all we need).  The poll() backend is used by default,
except on Mac OS X where poll() is broken for devices (including
/dev/tty and ptys).

Use SOCK_STREAM for socketpair, not SOCK_DGRAM so we get consistent
semantics when the other end closes.  This should make the conversion
to poll() less problematic.

Fix removal of trailing newlines in a debug message.

When checking for unused Runas_Aliases, count those used as part
of a Runas Group too.  Fixes a false positive warning.

Include stddef.h for rsize_t and errno_t on systems that support
it natively.

Fix braino.

Rebuild message catalog files.

Rebuild message catalog files.

Czech translation for sudo from translationproject.org.

Sync with translationproject.org

Change "next" back to 2.  In the context of "next Friday" we really
do want the friday of the upcoming (not current) week.  Unfortunately,
this means that things like "next week" and "next year" will match
one more than we really want.  Fixing this will require some fairly
major changes to the grammar.

Mention that relative times don't always do what you might expect.

Add diacritical for Zdenek Behan.

Do not fail if ttyname() cannot determine the tty but sudo can.
Should fix problems with running "make check" under pbuilder.

Remove extraneous $$CWD; from Bdale Garbee

Make "this" and "next" qualifiers work a bit better.  There is still
room for improvement as "this week" will use the current time instead
of the beginning of the week.  That's a separate issue though.

Mark main() public to silence a warning on HP-UX.

Be specific that we are talking about the Unix epoch; bug #615

Do not use "setup" as a verb; bug #614

Fix logic goof when checking open() status.

Sync with translationproject.org

Work around a bug in sudo 1.8.7 timing files where the indexes are
off by two.

Repair writing of the I/O log file indices broken in sudo 1.8.7.

Try to improve the PAGERS noexec example a bit.

Document comment character in ldap.conf
Clarify what is and is not supported in TLS_KEYPW
Mention that gsk8capicmd can be used to create a stash file

New bugs fixed for 1.8.8.

Fix setting of quiet flag when -q / --quiet is specified.
Do not print "sudoers: parsed OK" in quiet mode.

Updated translations from translationproject.org

Don't allow root to change its SELinux role without a password.
Bug #611

Mention new Mac OS X symbol interposition.

Updated translations from translationproject.org

Add configure checks for the exec functions we will dummy out.  This
is only really needed on Mac OS X when symbol interposition is being
performed but won't hurt elsewhere.

Fix installation of sudo_noexec on Mac OS X.
Use library symbol interposition on Mac OS X 10.4 and higher so
we don't need to set DYLD_FORCE_FLAT_NAMESPACE=1.

Fix error display from ldap_ssl_client_init().  There are two error
codes.  The return value can be decoded via ldap_err2string() but the
ssl reason code cannot (you have to look it up in a table online).

Fix typo in tls_key example for Tivoli

Don't escape '$' when running "sudo -i command".  Bug #564

Fix typo in comment.