| Age | Commit message (Collapse) | Author |
|---|
|
- Sort includes alphabetically
- Sort function prototypes alphabetically
- Add missing whitespace after commas
- Remove extra whitespace after commas
- Add missing whitespace between binary operators
- Add missing whitespace between reserved words and parens, i.e. fix "if("
- Add missing __dead to usage() definition
|
|
when <unistd.h> is included;
patch from Jan Stary <hans at stare dot cz>;
OK millert
|
|
okay millert@, tb@
|
|
my screen.
i don;t have a copy of this book, and a brief search online shows conflicting
punctuation, but i've chosen to replace the comma splice with a full stop,
which is at least grammatically correct, and seems the most likely solution.
whether this quote needs to be in two files escapes me...
|
|
the existing text encouraged the misconception that punched card
encodings were somehow standardized, which they were not. Lots of
different encodings existed, and the one simulated here was a
proprietary one of one specific vendor (GE). Also correct the
history: the folks at Bell Labs already played with this.
HISTORY mistake noticed by Sevan Janiyan <venture37 at geeklan
dot co dot uk>, who also agrees with the final patch.
tweak and OK jmc@
|
|
and correct Xr.
ok jmc
|
|
|
|
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.
|
|
From Jake Champlin
|
|
ok brynet@, tedu@
|
|
hoping to save somebody's life from the Leather Goddesses of Phobos
|
|
|
|
|
|
ok benno mlarkin tb
|
|
from fabio scotoni
|
|
- Write savegames and scorefiles to the current directory instead of /var
- Save oc_name and oc_descr alongside oc_uname in all situations
- When a levitation potion times out, explicitly float down
These patches were contributed last year by "tonypony76"; thanks!
ok deraadt@, with added enthusiasm from tedu@
|
|
From Randy Hartman
|
|
|
|
commonly used and metrically and logically more pleasant word
order, and fix the wrong translation;
translation error reported by Alessandro de Laurenzis on tech@;
OK pascal@ jmc@
|
|
just let the bonus happen if it does, players will figure it out.
|
|
|
|
less distracting this way.
|
|
bug noticed by mlarkin
|
|
ok cheloha deraadt schwarze
|
|
analysis and patch from David Fifield
|
|
properties for the Wumpus. The bug has been present since 4.3BSD-Reno
and was introduced by Keith Bostic on February 14, 1990 when committing
the major rewrite from Dave Taylor.
Patch (accompanied by a detailed functional and historical analysis)
from David Fifield <david at bamsoftware dot com> on bugs@.
With all the bats in these caves, how could a bug possibly survive
for twenty-eight years?
|
|
First, fill a missing pixel in the glyph for 'U'.
Second, fix an off-by-one in the line output routine. This caused an
extra space character to be output at the end of every line. It might
have caused a buffer overflow if any glyph had used the entire possible
width of 132 pixels, but as it is the widest is only 124 pixels.
Third, output blank lines instead of lines full of spaces (the number of
spaces was equal to the width of the most recent non-blank line).
Finally, add bounds checks to protect against buffer overflows that could arise
in case the data table were ever modified to draw out of bounds (characters
wider or taller than 132 pixels, or a two-byte code where the second byte is
past the end of the table).
ok bcallah
|
|
data table are unused and can be eliminated by rendering al glyphs and
re-encoding them. A few more entries can be eliminated through more
efficient encoding.
Patch generated with a python program by David Fifield.
For details see https://marc.info/?l=openbsd-bugs&m=153955110702704&w=2
ok bcallah
|
|
ok sthen@
|
|
user's home folder and setgid was removed. Therefore it's not possible to have a
single score file with all the users anymore but we forgot to change a comment
that still implied that.
Suggested by tb@ to just delete the comment.
|
|
but with much reduced permissions ("stdio tty" if ncurses based and "stdio"
for the ones that only perform basic operations).
There's still a few games that we cannot yet remove their fs access, through
pledge(2), since they open files on demand and too late, this might get
revisited in the future.
OK tb@
|
|
scorefile nevertheless an fd is opened way in advance and therefore we can
disable any further access to the filesystem.
OK tb@
|
|
OK tb@
|
|
effectively disable all fs access by dropping "rpath wpath cpath" from
pledge(2) right at the beginning of the program.
"looks right" tb@
|
|
Discussed with and OK tb@
OK cheloha@ on previous version
|
|
from Martin Kopta <martin at kopta dot eu>
|
|
in for loops.
ok stsp, bcallah
|
|
the battlestar code by reformatting a few lines.
special request from mlarkin so he'll go back to fixing meltdown.
|
|
David Fifield; ok tb@
|
|
|
|
Makes reading the classification algorithm easier.
ok tb@
|
|
adj360() is a modulo operation: use fmod(3) to simplify things.
Adding 0.5 to the phase to steer implicit truncation is a hack:
use lround(3) to simplify things.
Product of a discussion with pjanzen@/jca@/tb@ regarding floating-point
rounding and the importance of correctly classifying a gibbous moon.
ok pjanzen@ jca@ tb@
|
|
ok tb@ jca@
|
|
instead of gettimeofday(2).
From Scott Cheloha, ok jca
|
|
from scott cheloha
|
|
trailing whitespace is no longer permitted when invoked from stdin.
From Scott Cheloha.
ok jca
|
|
ok jasper@, jca@, deraadt@
|
|
From Scott Cheloha
|
|
ok tb
|
|
|
