Do for most running out of memory err() what was done for most running out of memory log_warn(). i.e. ("%s", __func__) instead of manual function names and redundant verbiage about which wrapper detected the out of memory condition.

ok henning@

pam: add timestamp support

timestamp: rename and simplify

libopenbsd: minor cleanup

doas: remove unnecessary configure checks, move shadow to its own file

doas: remove v flag, not neccessary, upstream doesn't have it and __DATE__ is bad for reproducible builds

libopenbsd/closefrom: correctly handle snprintf truncation

libopenbsd/readpassphrase: update to latest version from openssh-portable

adjust yyerror() to precede with "progname: " the error message string
OK tedu@ phessler@

doas.c: put login_style in ifdef to compile on Linux

Closes: #23 [via git-merge-pr]

lowercase doas ee cummings style

not necessarily the same name, but the indicated name

pam: check watch child pid

persist_timestamp: add start time and document implementation details

persist_timestamp: move timespec macros to libopenbsd

persist_timestamp: create timestamp file with O_NOFOLLOW and don't leak the name

persist_timestamp: remove goto from persist_open

persist_timestamp: persist_check was only used internally, make it static

persist_timestamp: use open directory fd to check and work with timestamp files

persist_timestamp: add session id to timestamps

persist_timestamp: make tmpfs requirement optional and only available on linux

persist_timestamp: use CLOCK_MONOTONIC_RAW

persist_timestamp: don't allow og+rwx permission for timestamp directory

persist_timestamp: cleanup

persist_timestamp: use /proc/self/stat to get tty_nr

add initial timestamp file support, disabled by default and only with shadow auth

configure: update version

configure: fix usage

man pages with pseudo synopses which list filenames end up creating very ugly output in man -k; after some discussion with ingo, we feel the simplest fix is to remove such SYNOPSIS sections: the info is hardly helpful at page top, is contained already in FILES, and there are sufficiently few that just zapping them is simple;

ok schwarze, who also helpfully ran things through a build to check
output;

no need to generate y.tab.h if nothing uses it, set YFLAGS to nothing instead of CLEANFILES += y.tab.h

okay millert@

for password failure, print Authorization failed instead of EPERM. will make things less confusing with commands rejected by config file.

a little const here and there to prevent rules from changing

prepenv can take a const rule

simplify example. list of ports variables was non-exahustive, which means what exactly? there should be a better place for such lists.

add a geteuid check to make sure we're root before plowing into setauth. spare some debugging effort in case doas is not installed setuid.

envlist and arglist are both string lists; simplify ok benno

it has been six months and two days... remove keepenv { obsolete } syntax

Be more explicit about the "args" syntax. In part from a patch from Anton dot Lindqvist at gmail dot com. OK tedu@

missing semicolon at end of rule. yacc doesn't seem to mind, though. from Edakawa

Add back the call to yyparse() that was accidentally dropped in the previous commit. Fortunately, doas fails closed...

ok tedu

move yyparse decl next to yyfp

as a result of the env rework, arraylen() is only used in parse.y. move it there and make it static.

use static in the right places to seperate modules better ok tedu

-L means no command

don't allow combining nopass and persist in a single rule

the sudo timeout was 5 minutes i believe, so we'll match that.

clarify that -L will exit without running a command.

add support for the verified auth ioctls using 'persist' rules. ok deraadt henning

unconst these parameters; i won't be changing bsd auth today.

move the authentication code to a function

bump version to v6.0

Add closefrom(2) from openssh-portable

Print -a flag in usage() only if HAVE_BSD_AUTH_H

Closes: #11 [via git-merge-pr]

minor configure tweaks

configure: error out if no authentication found and fix default CC

The string with path to shell could be taken directly from struct passwd. At some point later the data it points to is overridden by getpwuid() call, resulting in garbage. The problem could be easily demonstreated by double doas call:

  $ doas doas -su _sndio
  doas: mpty: command not found

The fix is easy: just strdup() the pw_shell field value.

okay tedu@, tweaks from & okay natano@

add "recvfd" to doas(1) for use with skey.

ok tb@ deraadt@

use posix correct optstring

minor tweaks

minor tweaks; ok tedu

somehow nopass snuck onto the :wheel example. i think it's better without.

revise environment handling. Add a setenv keyword for manipulating the environment. keepenv now means only retain everything. (for one release, the old use of keepenv will still work.) Allow setting variables to new or existing values, and also removing vars when keepenv is used. ok djm martijn tb

move a space to the correct spot

Merge pull request #8 from frgm/master

configure: fix usage() formatting & fix make install

fix make install

configure: fix usage() formatting

bump to version v0.3.2

fix --with(out)-pam configure option

fix pamcleanup

fix sys/tree.h test

bump version 0.3.1

remove pam_timestamp from pam config

remove unnecessary warning output

bump version 0.3

add --without-pam configure option to allow passwd/shadow auth

fix err messages

some more cleanup and refactoring of pam code

rename doas_pam.c to pam.c

import sys-tree.h from openssh-portable

Move the RB_ code from doas.h to env.c, and limit the environment interface to a simple prepenv function.

OK tedu@

the environment handling code was showing its age. just because environ is a char** array doesn't mean we must exclusively operate on such. convert to a red-black tree, manipulate as desired, then flatten to array. potentially overkill for the current operations, but reading the tea leaves i see that more manipulations are desired. ok tb (and some thought provoking disagreement from martijn)

don't use specified twice in a sentence, noticed by jmc

clarify some wording

specify that default is deny if no rule matches

remove pledge seccomp shim

This will never work, seccomp can't filter for paths (pointer) and all
rules are inherited by child processes.
pledge does not limit processes executed by execve.

open pam sessions with right user and remove setusercontext shim

before this change the sessions were opened as the user running doas.
Now it sets its uid to root and then opens a pam session for the target
user.
The setusercontext shim was removed, because pam handles all this and
its easier to just call setresuid and setresgid instead.

Revert "sync with upstream (setenv)"

This reverts commit 7f11114f0f07c653e0ea3d4ae093d7dcdda4a4ef.

bump version to 0.2

add more restrictive permissions and root:root as owner for binary

fix ld and cflags

sync with upstream (setenv)

add a doas.conf setenv directive that allows setting environment
variables explicitly and by copying existing environment variables
of a different name. E.g.

permit nopass setenv { PS1=$SUDO_PS1 FOO=bar } keepenv :wheel

ok tedu@ benno@

remove version.h and define VERSION in configure script

check return value of setresuid

remove nonstandard sys/cdefs.h

Merge pull request #7 from frgm/master

fix configure and add a new target - "uninstall"

fix configure script

 * "+=" is not working in shell
 * fix a typo (OS_FLAGS => OS_CFLAGS)

bsd.prog.mk: add "uninstall" target

Add doas style prompt for pam authentication

The doas style prompt "doas ($USER@$HOST) password:" is used if pam
gives back a prompt that matches "Password:[ ]" in other cases the
prompt provided by pam is used.

Make pam session handling more failsafe

More configure and make cleanup