History log of /openbsd-src/libexec/comsat/comsat.c (Results 1 – 25 of 51)
Revision Date Author Comments
# 5b133f3f 08-Mar-2023 guenther <guenther@openbsd.org>

Delete obsolete /* ARGSUSED */ lint comments.

ok miod@ millert@


# bc5a8259 12-Jul-2021 beck <beck@openbsd.org>

Change the error reporting pattern throughout the tree when unveil
fails to report the path that the failure occured on. Suggested by
deraadt@ after some tech discussion.

Work done and verified by A

Change the error reporting pattern throughout the tree when unveil
fails to report the path that the failure occured on. Suggested by
deraadt@ after some tech discussion.

Work done and verified by Ashton Fagg <ashton@fagg.id.au>

ok deraadt@ semarie@ claudio@

show more ...


# 51d464e8 24-Sep-2018 deraadt <deraadt@openbsd.org>

unveil maildir, utmp, /tmp, and /dev. For the vast number of people
using biff.


# db006180 03-Apr-2017 tedu <tedu@openbsd.org>

check return from pread, don't divide -1 for count


# b15aba87 03-Apr-2017 deraadt <deraadt@openbsd.org>

use recallocarray() because the array contains data that can be observed


# 9733295b 02-Apr-2017 guenther <guenther@openbsd.org>

Prefer pread() over lseek()+read()
open() only needs the mode argument if O_CREAT is present

ok beck@ deraadt@


# 4e1fb0a6 02-Apr-2016 millert <millert@openbsd.org>

Use open(tty, O_WRONLY) + fdopen() instead of fopen(tty, "w") to
keep stdio from opening with O_CREAT which would require pledge cpath.


# 5b15ea38 12-Oct-2015 uebayasi <uebayasi@openbsd.org>

Call pledge(2) after initial getsockname(2) to avoid "inet" addition.

From & OK deraadt@


# ecfe3494 10-Oct-2015 deraadt <deraadt@openbsd.org>

normalize a few more tame request orderings, to help review


# 9c96c22e 09-Oct-2015 deraadt <deraadt@openbsd.org>

With nfs spool (fork + seteuid/setuid balony) support gone, it becomes
possible to pledge "stdio rpath wpath tty proc"
Noone uses this code anymore. This is a demonstration...


# 3f241492 09-Oct-2015 deraadt <deraadt@openbsd.org>

remove NFS spool support; it stands in the way of pledge(2)


# fcc8672b 06-Jul-2015 millert <millert@openbsd.org>

Trim trailing whitespace from the comsat message before calling
strtonum() to parse the offset since mail.local writes a trailing
newline. Otherwise comsat just discards the message. OK deraadt@


# a47b6461 18-Apr-2015 deraadt <deraadt@openbsd.org>

Convert many atoi() calls to strtonum(), adding range checks and failure
handling along the way.
Reviews by Brendan MacDonell, Jeremy Devenport, florian, doug, millert


# b9fc9a72 16-Jan-2015 deraadt <deraadt@openbsd.org>

Replace <sys/param.h> with <limits.h> and other less dirty headers where
possible. Annotate <sys/param.h> lines with their current reasons. Switch
to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_N

Replace <sys/param.h> with <limits.h> and other less dirty headers where
possible. Annotate <sys/param.h> lines with their current reasons. Switch
to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_NAME_MAX, etc. Change
MIN() and MAX() to local definitions of MINIMUM() and MAXIMUM() where
sensible to avoid pulling in the pollution. These are the files confirmed
through binary verification.
ok guenther, millert, doug (helped with the verification protocol)

show more ...


# f4faee91 04-Dec-2012 deraadt <deraadt@openbsd.org>

remove some unnecessary sys/param.h inclusions


# 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 ...


# 4e233fd5 17-Aug-2006 ray <ray@openbsd.org>

recv(2) returns a ssize_t, so match cc with it.
st.st_mtime is a time_t, so match utmpmtime with it.
Check for -1 instead of <0 for some system calls.
Remove pointless casts.

OK dhill@ and millert@


# 47e832f1 15-Nov-2005 millert <millert@openbsd.org>

the third arg of read() is size_t, not int (cast fix)
use UT_NAMESIZE more consistently
from Andrey Matveev


# 458e279f 16-Sep-2004 otto <otto@openbsd.org>

more signal flag races; ok miod@


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

ARGSUSED before signal handler with unused signo


# a6069947 26-Sep-2003 deraadt <deraadt@openbsd.org>

free(NULL) allowed


# 7cdf2e73 25-Sep-2003 millert <millert@openbsd.org>

Fix typo in revision 1.29; deraadt@ OK


# 06d77a2e 24-Sep-2003 deraadt <deraadt@openbsd.org>

realloc fixes; ho ok


# 17aedf09 19-Jun-2003 deraadt <deraadt@openbsd.org>

use fseeko; millert ok


# e33d3bd3 02-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.


123