Age | Commit message (Collapse) | Author |
|
When invoking fortune with the -l option (to get long dictums only),
the program gets stuck in an infinite loop because fortlen() doesn't
return the fortune length correctly.
OK mestre@, tb@, deraadt@, schwarze@
|
|
from tomr
|
|
|
|
|
|
|
|
|
|
|
|
also delete ancient todo file:
Frankly,
I am getting pretty sick of playing this game.
Hence,
the version which you get may have several bugs
|
|
uses a pointer equality check for save game validity. remove it.
discovered by the ever inquisitive mlarkin
|
|
internal obfuscation scheme the other way around, thereby avoiding a
signed underflow.
ok mestre
|
|
ld(1) error found by mestre a while ago.
fix suggested and ok naddy
|
|
OK deraadt@
|
|
avoid compiler silly warnings
sure deraadt@
|
|
OK deraadt
|
|
"warning:" rectification.
|
|
of pointer subtraction without a cast. So cast those expressions
to (int).
Switch one local variable to the same type as the parameter it is
compared to.
ok deraadt@ guenther@ beck@
|
|
ok guenther, krw
|
|
ok beck@
|
|
|
|
ok tom@
|
|
dereference.
Same change made in NetBSD in 1997.
ok tb@ millert@ tom@
|
|
|
|
M_PI is always defined, so we can drop those directives.
OK deraadt@, millert@
|
|
|
|
out of bound accesses of arrays, leading to sefaults or bus errors. Verify
that at most one boardspec with lower case letters is given, or error out.
ok millert
|
|
|
|
ok deraadt
|
|
|
|
their setgid bits got removed. Mark them as such in their manuals.
ok deraadt, help with wording jmc
|
|
random lines are selected on output and that's enough.
From daniel, ok tb
|
|
aren't recognized as such, for example 18446744073709551577 given
in the commit message of factor.c r1.7 from NetBSD. Move the
return type of usqrt() from u_int32_t to u_int64_t.
ok guenther, tom, otto
|
|
nonsensical "if and only iff" with "if and only if" and zap some
trailing whitespace.
|
|
not stanislaw lem (an easy blunder, i'd say); i've also inserted the "J."
into one of these credits (Stanislaw J. Lec) since all the others have them;
credit to antoni grzymala; diff pockled from netbsd
|
|
ok deraadt@
|
|
Pull in <time.h> for time()
ok deraadt@
|
|
ok deraadt@
|
|
and setitimer()
ok deraadt@
|
|
ok deraadt@
|
|
Fix a type mismatch in ftp's "page" command and could make transfers restart
at the wrong position.
ok and a ull->ll tweak from natano@, ok tedu@
|
|
of the Newton method from ping.c. Fixes a rounding issue that caused
failure to factor numbers close to 2^64, e.g. 18446744030759878681.
While there, fix an off by one error that caused 4295360521 to be
reported as a prime. Issues reported by Paul Stoeber and Michael Bozon.
ok tedu, deraadt
|
|
'i' shadows the local loop indexing variable of scr_update(), so
no need to rename it. No binary change on amd64.
ok millert
|
|
can become unusable during game play. Same fix was made in NetBSD's
tetris.c -r1.31 by Christos Zoulas almost exactly a year ago.
ok tedu
|
|
correctly - logically complete that now by removing MLINKS from base;
authors need only to ensure there is an entry in NAME for any function/
util being added. MLINKS will still work, and remain for perl to ease
upgrades;
ok nicm (curses) bcook (ssl)
ok schwarze, who provided a lot of feedback and assistance
ok tb natano jung
|
|
ok millert@
|
|
setting the O_NONBLOCK flag on it with fcntl(F_SETFL) afterwards,
just pass SOCK_NONBLOCK to socket() or accept4() and get it right
to begin with.
ok millert@ krw@ beck@ deraadt@ jca@
|
|
then additionally it calls clearerr(3) deliberately but we want to catch any
error that may happen and this way we couldn't catch it
OK tb@
|
|
At the time when this was commited me and tb@ discussed that it shouldn't be
changed, but still snucked in within a larger diff that we didn't notice.
OK tb@
|
|
do it
- Replace random() >> 5 by only random(): this was discussed a few months ago
and naddy@ said at the time "Those games were originally written with the
rand(3) function. The lower bits returned by rand(3) suffered from notoriously
poor randomness, so this idiom developed where people would use the higher,
more random bits."
OK tb@ after his remarks
|
|
This was prompted by an initial patch sent by Edgar Pettijohn
<edgar ! pettijohn-web.com> but the actual commit is different
|
|
These are wrappers for malloc(3) and free(3) with NULL checks. do_free()
is pointless since free() already checks for NULL. do_malloc() is used
only three times, once asprintf(3) seems more appropriate, and for just
two calls the benefit of a custom wrapper is minimal.
ok millert@
|