Add Guillem Jover

recent changes

Use roff conditionals in the manuals instead of post-processing.
We still need to process the resulting .man.in files to add back
the conditionals but this should be easier to debug as the changes
are visible in the .in file.
Some minor postprocessing is still used to make the manuals HP-UX
friendly and to change "0 seconds" -> unlimited after substitution.

Sudo plugin manual updates and clarification from Guillem Jover:
- Add missing return information for show_version().
- Fix prototypes for several function pointers.
- Update SUDO_API_VERSION_MINOR.
- Add missing references to log_suspend() and change_winsize().
- Add missing "array.".
- Clarify that argc can be zero on sudo -V.
- Clarify size requirements for conversation array arguments.
- Clarify timeout zero value for struct sudo_conv_message.
- Clarify initial and final state of reply in struct sudo_conv_reply.

Revert changes to give arguments to the .Bx macro.
This is intended for things like .Bx 4.3 to generate "4.3BSD" so
the argument ends up before the BSD, not after.  Just go back to
using "BSD authentication" and "BSD login classes" so fixmdoc.sh
can operate correctly.  Bug #861

Update fixmdoc.sh to match the BSD -> .Bx changes in the manuals.
Bug #861

Add support for utmps as found in HP-UX.

Support st_nmtime in struct stat as found in HP-UX.

If fcntl fails, fall back to the /proc implementation.

Mention schema.olcSudo

Mention schema.olcSudo here too.

OpenLDAP schema file for Sudo in on-line configuration (OLC) format.
From Frederic Pasteleurs.

Updated translations from translationproject.org

Only use closefrom_fallback() if no better method exists.
The previous logic was too fragile.

Updated translations from translationproject.org

Portuguese translation for sudo and sudoers from translationproject.org.

Add sudo_gai_fatal, sudo_gai_vfatal, sudo_gai_vwarn, sudo_gai_warn
and gai_log_warning that use gai_strerror() instead of strerror().

Fix memory leak in runaslist_matches().

typo

regen

More updates for 1.8.26

Add support for negated sudoRunAsUser and sudoRunAsGroup entries.

Document that the target user's groups may be specified via the -g option.

Include getpwent() version of sudo_getgrouplist2_v1() from getgrouplist.c

Use a testsudoers group file with known contents instead of the system one.

Allow the group set by "sudo -g" to be any of the target user's groups.
Previously, this was only allowed if the group matched the target
user's primary group ID (from the passwd database entry).
The sudoers policy will now allow the group if it is one of the
target user's supplemental groups as well.

Skip sudo_getgrouplist2() check on systems with getgrouplist_2().
sudo_getgrouplist2() is just a wrapper on such systems and this
avoids a test failure on macOS where a user is automatically a
member of certain groups.

Add missing exported symbol sudo_term_eof

Add missing #ifdef LDAP_OPT_X_TLS_REQUIRE_CERT
Fixes problems building on older LDAP sdks.

add getgrouplist_test.c

Check the user's primary gid from the passwd file too.

ignore prologue

Convert PVS-Studio comment to ANSI C.

Fix some mangled text in the license block.

Add regress test for sudo_getgrouplist2().
This test assumes all the groups in root's group list can
be resolved by group ID.

More changes in 1.8.26

Add padding option to cvtsudoers.
Bug #856

Remove an errant grset++ in the AIX version of sudo_getgrouplist2().
Bug #857

Pass --sourcetree-root to pvs-studio and don't check sudo_noexec.c.
Since we don't auto-generate dependencies for sudo_noexec.c we
can't easily check it from outside the source tree.  This
is not a problem as it just contains stub functions.

Asturian translation for sudo from translationproject.org

Add support for CLOCK_MONOTONIC_RAW and CLOCK_UPTIME_RAW, present
on macOS.

Add --enable-pvs-studio configure option to create PVS-Studio.cfg.

Add pvs-studio target and associated production rules.

Add comments in .c files so PVS-Studio will check them.

Simplify range checks.
No need to check for ERANGE in the cases where we also check
that the value is <= INT_MAX.  Found by PVS-Studio.

Avoid some PVS-Studio false positives.

Remove some calls to sudo_fatalx(); just propagate the error return.

No need to check if fd_dst is -1 in sudoedit mode.
Failure to open the destination sudoedit file is fatal so there's
no need to check that fd_dst != -1 later on.  Found by PVS-Studio.

In timestamp_open() no need to free cookie on error, it is NULL.
Found by PVS-Studio.

Fix a memory leak on malloc() error in sudo_ldap_role_to_priv().
Coverity CID 188804

Move the allocation of role to be immediately before in_role is set.
This makes it clear that when in_role == true, role is non-NULL.
Also remove two dead stores.

Fix trimming of non-escaped trailing space in ldif_parse_attribute().
Found by PVS-Studio.

Simplify the logic surrounding sudoers_args in command_args_match().
We only need to check that sudoers_args is non-NULL once.
Found by PVS-Studio.

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)