History log of /openbsd-src/usr.bin/script/script.c (Results 1 – 25 of 35)
Revision Date Author Comments
# 3aaa63eb 28-Jun-2019 deraadt <deraadt@openbsd.org>

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.

show more ...


# 328f2935 21-Jan-2018 jasper <jasper@openbsd.org>

add new '-c' option which runs a command instead of an interactive shell

from Paul de Weerd
ok ian@ job@; manpage bits are ok jmc@


# ce164175 12-Apr-2017 deraadt <deraadt@openbsd.org>

bluhm discovered "script < /dev/null" attempts tty operations in violation
of pledge. We can wrap isatty() around those codepaths.
ok bluhm beck


# c6c21631 20-Oct-2015 semarie <semarie@openbsd.org>

add "proc" to pledge for script(1)

it needs to "forward" SIGWINCH to subprocesses.

ok deraadt@


# 0bd1216c 09-Oct-2015 deraadt <deraadt@openbsd.org>

Change all tame callers to namechange to pledge(2).


# 2c634e5c 07-Oct-2015 deraadt <deraadt@openbsd.org>

move from tame "ioctl" to tame "tty", which provides a better fit for
this program which uses tcgetattr(). the tcsetattr() calls are outside
the tame regions.


# 03d04e93 04-Oct-2015 deraadt <deraadt@openbsd.org>

Repair tame() error check to be == -1


# f77331b6 03-Oct-2015 deraadt <deraadt@openbsd.org>

script is two processes. the main io-loop process can be locked down with
tame "stdio" since all it does is move data back and forth, while the master
process needs "stdio ioctl" to use TCSAFLUSH at

script is two processes. the main io-loop process can be locked down with
tame "stdio" since all it does is move data back and forth, while the master
process needs "stdio ioctl" to use TCSAFLUSH at the very end. TCSAFLUSH is
included in the kernel's rather restrictive ioctl feature lists made
available with the "ioctl" ability.

show more ...


# 9e69f79d 19-Jul-2015 deraadt <deraadt@openbsd.org>

SIGCHLD handler was established before forking into two processes.
As a result it also ran in the child (incorrect).
ok miod


# f7055df5 15-Mar-2015 millert <millert@openbsd.org>

tzfile.h is an internal header that should never have been installed.
What's worse, the tzfile.h that gets installed is over 20 years old
and doesn't match the real tzfile.h in libc/time. This makes

tzfile.h is an internal header that should never have been installed.
What's worse, the tzfile.h that gets installed is over 20 years old
and doesn't match the real tzfile.h in libc/time. This makes the
tree safe for /usr/include/tzfile.h removal. The TM_YEAR_BASE
define has been moved to time.h temporarily until its usage is
replaced by 1900 in the tree. Actual removal of tzfile.h is pending
a ports build. Based on a diff from deraadt@

show more ...


# 043fbe51 27-Oct-2009 deraadt <deraadt@openbsd.org>

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(th

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

show more ...


# d090f823 12-Dec-2005 deraadt <deraadt@openbsd.org>

in the write codepaths, errors other than EAGAIN are fatal to the loop; ok otto


# 79052954 11-Apr-2005 deraadt <deraadt@openbsd.org>

use STDERR_FILENO; ok cloder


# c54d461e 19-Dec-2004 millert <millert@openbsd.org>

Block SIGALRM during write--we only want it to interrupt read().
OK deraadt@


# 259e51e4 10-Oct-2004 mickey <mickey@openbsd.org>

use err/warn and __progname


# 79602374 14-Sep-2004 deraadt <deraadt@openbsd.org>

ARGSUSED before signal handler with unused signo


# 1837a5ca 10-Jun-2003 deraadt <deraadt@openbsd.org>

mostly ansi cleanup; pval ok


# f75387cb 03-Jun-2003 millert <millert@openbsd.org>

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999. Proofed by myself and Theo.


# c72b5b24 16-Feb-2002 millert <millert@openbsd.org>

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be don

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

show more ...


# 486be880 18-Nov-2001 deraadt <deraadt@openbsd.org>

After a simplistic rewrite, signal flags were not being noticed until a
succesfull read operation. To avoid this, we must use SA_RESTART, and
rewrite most of the guts to handle interrupted read, wri

After a simplistic rewrite, signal flags were not being noticed until a
succesfull read operation. To avoid this, we must use SA_RESTART, and
rewrite most of the guts to handle interrupted read, write, etc, even being
careful to check what library routines are called.... input from millert,
markus, and espie

show more ...


# c96f6a27 09-Jul-2001 deraadt <deraadt@openbsd.org>

correct type on last arg to execl(); nordin@cse.ogi.edu


# 50db3a12 11-Jan-2001 deraadt <deraadt@openbsd.org>

comment on a real nasty race


# b638aa94 30-Jun-2000 millert <millert@openbsd.org>

warnx?/errx? paranoia (use "%s" not a bare string unless it is a
constant). These are not security holes but it is worth fixing
them anyway both for robustness and so folks looking for examples
in t

warnx?/errx? paranoia (use "%s" not a bare string unless it is a
constant). These are not security holes but it is worth fixing
them anyway both for robustness and so folks looking for examples
in the tree are not misled into doing something potentially dangerous.
Furthermore, it is a bad idea to assume that pathnames will not
include '%' in them and that error routines don't return strings
with '%' in them (especially in light of the possibility of locales).

show more ...


# 6f6caaa9 16-Apr-2000 espie <espie@openbsd.org>

Propagate window size changes to slave.


# e3989934 22-Mar-2000 ericj <ericj@openbsd.org>

Make script give correct return values; adapted from FreeBSD


12