spelling fixes; from paul tagliamonte
amendments to his diff are noted on tech

For open/openat, if the flags parameter does not contain O_CREAT, the
3rd (variadic) mode_t parameter is irrelevant. Many developers in the past
have passed mode_t (0, 044, 0644, or such), which mig

For open/openat, if the flags parameter does not contain O_CREAT, the
3rd (variadic) mode_t parameter is irrelevant. Many developers in the past
have passed mode_t (0, 044, 0644, or such), which might lead future people
to copy this broken idiom, and perhaps even believe this parameter has some
meaning or implication or application. Delete them all.
This comes out of a conversation where tb@ noticed that a strange (but
intentional) pledge behaviour is to always knock-out high-bits from
mode_t on a number of system calls as a safety factor, and his bewilderment
that this appeared to be happening against valid modes (at least visually),
but no sorry, they are all irrelevant junk. They could all be 0xdeafbeef.
ok millert

show more ...

Delete useless call to setlocale(3):
Spell uses nothing locale-dependent, and it will never need support
for non-English characters because the basic algorithms used are
specific to the English langu

Delete useless call to setlocale(3):
Spell uses nothing locale-dependent, and it will never need support
for non-English characters because the basic algorithms used are
specific to the English language in the first place.
While here, make usage() static and return from main() rather than exit(3).
Minimially tweaked diff originally from Jan Stary.
OK tedu@

show more ...

Fix some warnings: p1() and p2() are only declared without arguments so
that nop() can be nop(void), so just change nop() to have the same
arguments as all the other functions and make p1() and p2()

Fix some warnings: p1() and p2() are only declared without arguments so
that nop() can be nop(void), so just change nop() to have the same
arguments as all the other functions and make p1() and p2() proper
prototypes; and do not use '\0' as a pointer constant.

ok florian deraadt

show more ...

pledge "stdio rpath wpath cpath"
ok doug

in getopt() blocks, stop incrementing flag variable which are supposed
to just be 0/1
ok miod florian

SIZE_MAX is standard, we should be using it in preference to the
obsolete SIZE_T_MAX. OK miod@ beck@

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

Use errc/warnc to simplify code.
Also, in 'ftp', always put the error message last, after the hostname/ipaddr.

ok jsing@ krw@ millert@

unsigned char for ctype
ok krw okan

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

use calloc() to avoid malloc(n * m) overflows; checked by djm canacar jsg

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

KNF [return(foo) -> return (foo)]. ok millert@

No more writable strings in -b mode.

spell command from 4.4BSD, now freed by Caldera.
I have made some significant changes:
o spell no longer uses hashed files. The algorithm for generating
the hashed files breaks down for large file

spell command from 4.4BSD, now freed by Caldera.
I have made some significant changes:
o spell no longer uses hashed files. The algorithm for generating
the hashed files breaks down for large files. Instead, I use the
same approach as look(1) and do binary searches of mmap(2)ed files
which has acceptable performance. A side effect of this is that
spell can now search multiple word lists.
o The spell binary has been renamed to "spellprog" so as to not conflict
with the spell shell script. This is consistent with other operating
systems.
o The putsuf() and putword() functions have been renamed to
trypref() and tryword() respectively for consistency with modern
versions and for general sanity.
o the spell front-end shell script has been completely rewritten
o The code has been ANSIfied and re-style(9)d

spell currently must be compiled with -fwriteable-strings to make
the -b option work. This should be fixed.

show more ...