If sudo_ldap_get_values_len() fails goto cleanup instead of oom.
This is not strictly necessary as there's not anything to cleanup
in this case but it is more consistent with the code that follows.

Fix handling of timeout values in sudoers.
When passing the timeout back to the front end, ignore the
user-specified timeout if it is not set (initialized to 0).
Otherwise, sudo would choose a zero user-specified timeout over
the sudoers-specified timeout (non-zero).

Fix cut & pastos in cvtsudoers_make_gritem()

Fix expected test output now that command_timeout is parsed correctly
in LDIF.

tv_nsec can never be negative after timespecsub.
Found by PVS Studio

Avoid potentially undefined behavior.
Found by PVS Studio.

sudo_ldap_parse_option() never returns '=' as the operator.
When parsing command_timeout, role, type, privs and limitprivs,
check that val is non-NULL instead.  Found by PVS Studio.

Fix up #line entries that reference lex.sudoers.c.

Fix workaround for broken sudo 1.8.7 timing files.

Fix memory leak when reusing the runas list.  We need to free the
member list itself as well as its contents.

Some DIAGNOSTICS updates:
Update error message for when the user's uid does not exist in passwd.
Remove "This error indicates" and some other cosmetic cleanups.

If the user's passwd entry cannot be resolved via the uid, use the
same error message as visudo.

Add a DIAGNOSTICS section with an explanation of the more non-trivial
error messages.

Replace sudo_fatal(NULL) with an "unable to allocate memory" message
that includes the function name.

Make EOF handling while reading the password prompt more like getpass(3).
We now return the password as long as at least one character has
been read.  Previously, EOF at the password prompt was treated as
if nothing was entered.

regen

Print a warning for password read issues.
Issues include: timeout at the password prompt, read error while
reading the password, and EOF reading the password.

Handle EOF on password input when pwfedback is enabled.

Fix remaining instances of "e.g." without a trailing ','.

Use mdoc macros for BSD systems.
All manuals now pass "make lint"

Use -Wstyle with -Tlint since sudo is not part of the base system.
This avoids "referenced manual not found" and "operating system
explicitly specified" warnings.

Document log_suspend() and fix the description of the change_winsize() return value.

Fix problems found by igor.  Bug #854

Sort DOCS and DEVDOCS and remove extra sudoers entry (it was listed twice).

Add igor target to run igor(1) on the manuals.

Add new -S option to sleep while the command was suspended.
The default behavior is now to not consider the time the command
was suspended as part of the normal inter-event delay.

Add a suspend event type to the I/O log to log suspend/resume of
the command so we can skip that delay during replay.

Initialize the pty rows/cols based on the values we stored in user_details.
This fixes a minor issue where we would send an extra window size
change event the first time the command was suspended.

Add support for OpenLDAP's TLS_REQCERT setting in ldap.conf.

Move definition of TIME_T_MAX to sudo_util.h

Changes in 1.8.26 (so far).

Treat LOGIN, LOGNAME and USER specially.  If one is preserved
or deleted we want to preserve or delete all of them.

Remove special handling of the USERNAME environment variable.  It
used to be set on old versions of Fedora but that hasn't been the
case for some time.  It's worth noting that ssh doesn't set USERNAME
either.

sudo 1.8.26

Remove unused system_maxgroups argument from fill_group_list().

Pass getgrouplist() NGROUPS_MAX+1, not NGROUPS_MAX so we have room
for the primary gid.

In print_member_json_int() eliminate the need_newline variable
and just move the non-alias expansion printing bits into the
else clause, including the newline and comma printing.

Add regress test for bug #853

When expanding an alias in print_member_json_int() avoid printing
an extra comma at the end of the entry.  Bug #853.

Add Kan Sasaki

sudo 1.8.25p1

Fix a crash in the event system's poll() backend introduced with
support for nanosecond timers.  Only affects systems without ppoll().
Bug #851

regen

Allow for some clock drift due to ntpd, etc.

If sudo_lock_file() fails for a reason other than the file already
being locked, give the user a chance to edit anyway.

Quick sort is not a stable sort; use distinct sudoOrder values so
the output is predictable.

Fix warnings on OpenIndiana (Illumos)

Correct ldap_to_sudoers() return value.

Bug #849

The sssd backend used to take the first match, assuming that entries
were sorted in descending order by sudoOrder.  That allowed it to
avoid iterating over the entire list of rules.  Now that we convert
to a sudoers parse tree, we need to convert rules in ascending
order, not descending.  The simplest way to accomplish this is to
simply iterate over the rules from last to first, reversing the
sort order.  Bug #849

Add some more ldif -> sudoers tests to verify sudoOrder.

For conversion to a sudoers parse tree, ldap_entry_compare() now
needs to sort in ascending order, not descending.  Bug #849

No need to set input_file for stdin in parse_ldif(); noted by clang analyzer.

Use TIME_T_MAX when parsing the I/O log file timestamp and disallow
negative times.

When parsing an I/O log timing line, store the result in a timespec,
not a double.  The speed factor (for scaling the delay) in sudoreplay
is still a double but we only need to adjust the delay if the factor
is something other than 1.0.

Fix memory leak in test.

regen

Update conversion of DID_* to KEPT_* to match the new values of
DID_* and KEPT_*.

Set the LOGIN environment variable on AIX like we do LOGNAME.

Add a test for the 4-argument au_close() function found in Solaris
11 instead of assuming it is present if __sun is defined.  Fixes a
compilation error on OpenIndiana and older Solaris versions.

Add Miguel Sanders and Scott Cheloha

testsudoers changes

Add ldif support to testsudoers

Move ldif -> sudoers conversion code into parse_ldif.c

Move string list functions to their own file.

sync

Backward ABI compatibility for even functions that use a timeval.

Use a monotonic timer for the event subsystem.

Use struct timespec, not struct timeval in the event subsystem.
Use ppoll() or pselect() if avaialble which use timespec.

sync

Eliminate most use of parsed_sudoers in cvtsudoers

Make alias_apply() take 3 arguments, the first being a pointer to the
struct sudoers_parse_tree.

Handle systems where root's gid is not 0.

Add missing files from last commit.

Add regress test for I/O log plugin endpoints

We cannot reuse last_time for the I/O log info file now that it is
a monotonic timer.  Just call time(3) in write_info_log() directly.

Move the loop to free the monitor_messages list into free_exec_closure_pty()

regen

Fix typo in last commit.

Do not assume all Linux has linux/random.h.
Add missing sys/syscall.h include

Cast uid/gid to unsigned int before printing.

Only include stdarg.h if we need it.

Include stddef.h for offsetof() definition.

fix compiler warnings on Solaris 11

Fix setting of errno when gotdata() fails.

Bugs 846 and 847

We still need to include string.h for AIX (and possibly others)
when we are not using the system memset_r() function and rsize_t
is defined by the system headers.

Add --enable-package-build to give configure a hint that we are
building a package.  This can be used to avoid relying on libc
functions that may not be present in all libc versions for a
particular system.  For instance, AIX 7.1 may or may not have
memset_s() and getline() present.

AIX defines rsize_t in string.h, not stddef.h for use by the
memset_s() prototype.  We use our own memset_s() on AIX since it
is not available on all BOS levels which makes package building
problematic.

Fix printing of T_TIMESPEC values.

Remove unused struct script_buf

Document when the I/O log timing file entry bug was introduced.

sync

HP-UX doesn't suport CLOCK_MONOTONIC but we can use gethrtime() instead.

Close the pty slave in the parent so that when the command and
monitor exit, the pty gets recycled without our having to close
it directly.

Move updating of the window size to the monitor process.
This will allow us to close the slave in the main sudo process in
the future so only the command and monitor have it open.

sudo 1.8.25

Fix test output for bug #845

Fix pasto when converting sudoNotAfter; from Miguel Sanders
Bug #845

Use a monotonic timer that only runs while not suspended for the
iolog timing values and write nsec-precision entries.