Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Sigh, more of the same (missed two) | 2009-07-04 | 1 | -3/+3 | ||
| | ||||||
* | __attribute__((__noreturn__)) -> __dead | 2009-07-04 | 1 | -5/+5 | ||
| | ||||||
* | ANSIfy function declarations. | 2009-07-04 | 16 | -297/+152 | ||
| | ||||||
* | Remove two more bogus casts. These *do* create object diffs on amd64, | 2009-07-04 | 1 | -4/+4 | ||
| | | | for no clear reason, but it looks like the diffs are harmless. | |||||
* | Remove bogus casts. Two object file diffs inspected and found to be harmless. | 2009-07-04 | 3 | -20/+19 | ||
| | ||||||
* | Remove a pile of bogus casts. One object file diff, crosschecked and | 2009-07-04 | 8 | -43/+43 | ||
| | | | found harmless. | |||||
* | Fix two serious string-handling bugs (one exploitable, one probably | 2009-06-29 | 7 | -33/+49 | ||
| | | | | exploitable) and also add proper checking/paranoia in several other places. | |||||
* | sprintf -> snprintf | 2009-06-28 | 7 | -43/+58 | ||
| | ||||||
* | Fix possible remote DoS of a running hunt game, and prevent a possible | 2009-06-28 | 1 | -3/+9 | ||
| | | | theoretical attack involving >= 1 billion ammo. | |||||
* | a bit more const | 2009-06-07 | 2 | -5/+5 | ||
| | ||||||
* | sprinkle some more const | 2009-06-07 | 3 | -9/+9 | ||
| | ||||||
* | remove unnecessary casts | 2009-06-07 | 2 | -18/+18 | ||
| | ||||||
* | sprintf -> snprintf, plus some use of strlcpy/strlcat where appropriate | 2009-06-07 | 14 | -181/+274 | ||
| | | | XXX: there's still one sprintf left which will take some hacking to expunge. | |||||
* | ANSIfy function declarations. All object file diffs inspected. | 2009-06-07 | 53 | -1060/+687 | ||
| | ||||||
* | Rectify non-compiling code that appears when DEBUG is defined. | 2009-06-04 | 2 | -25/+29 | ||
| | ||||||
* | Set WARNS=4 here. This got missed on my previous WARNS sweep in games. | 2009-06-04 | 1 | -2/+2 | ||
| | | | Everything appears to compile ok with WARNS=4 anyway though. | |||||
* | Remove global scratch string buffer. Don't zoom off the end while reading | 2009-06-04 | 2 | -11/+10 | ||
| | | | user input, either. | |||||
* | Remove remaining references to sprintf. | 2009-06-04 | 2 | -36/+47 | ||
| | ||||||
* | Make a couple of the logging/printing functions printf-alikes. This removes | 2009-06-04 | 3 | -82/+78 | ||
| | | | most of the calls to sprintf. | |||||
* | attribute noreturn -> __dead | 2009-06-04 | 1 | -4/+4 | ||
| | ||||||
* | sprintf -> snprintf | 2009-06-04 | 1 | -3/+3 | ||
| | ||||||
* | Increase spending on vowels. No object file diffs. | 2009-06-04 | 5 | -69/+69 | ||
| | ||||||
* | ANSIfy function declarations.x | 2009-06-04 | 6 | -106/+53 | ||
| | ||||||
* | sprintf -> snprintf | 2009-06-04 | 3 | -11/+13 | ||
| | ||||||
* | Don't use a variable-size array here. There's not really that much point. | 2009-06-01 | 1 | -2/+2 | ||
| | ||||||
* | Rearrange where the alternate definition of __dead goes. | 2009-05-27 | 2 | -6/+6 | ||
| | | | | | It can't be before "include.h" as that includes a bunch of system headers, but it can't be after either as it also includes a bunch of local headers. Therefore, it needs to be *in* include.h. | |||||
* | Edit some. | 2009-05-26 | 1 | -36/+34 | ||
| | ||||||
* | Assorted fixes (mostly grammar, some markup) from OpenBSD. | 2009-05-26 | 1 | -50/+68 | ||
| | ||||||
* | Paranoia about lengths of login names. From pjanzen of OpenBSD. | 2009-05-26 | 1 | -2/+2 | ||
| | ||||||
* | Avoid SIGSEGV on users not in password file. From pjanzen of OpenBSD. | 2009-05-26 | 1 | -3/+10 | ||
| | ||||||
* | sprintf -> snprintf | 2009-05-26 | 3 | -18/+26 | ||
| | ||||||
* | attribute noreturn -> __dead | 2009-05-25 | 1 | -3/+3 | ||
| | ||||||
* | We don't need a private #define "unsgn" for "unsigned", especially since | 2009-05-25 | 2 | -5/+4 | ||
| | | | it was used exactly once. | |||||
* | Use the time to call srandom(), not getpid(). | 2009-05-25 | 1 | -3/+4 | ||
| | ||||||
* | KNF/whitespace: format preprocessor directives correctly. | 2009-05-25 | 11 | -147/+147 | ||
| | ||||||
* | ANSIfy function declarations. Object file diffs checked. | 2009-05-25 | 12 | -88/+64 | ||
| | ||||||
* | sprintf -> snprintf | 2009-05-25 | 5 | -16/+24 | ||
| | ||||||
* | ANSIfy function declarations. Object file diffs cross-checked. | 2009-05-25 | 7 | -128/+79 | ||
| | ||||||
* | attribute noreturn -> __dead | 2009-05-25 | 1 | -2/+2 | ||
| | ||||||
* | Clean up scorefile handling. | 2009-05-25 | 2 | -36/+556 | ||
| | | | | | | | | | | | | | | | | | | | | | | This was writing time_t to disk. Worse, the time_t member was at an unaligned offset in the structure in question, so after the time_t change the structure layout depended on compiler-inserted structure padding. This gives three legacy formats: one with 32-bit time, one with 64-bit time, and one with 64-bit time and 4 bytes of structure padding. And of course the file didn't have a header or version coding or anything. The new code writes a structure of well-defined size that should not receive unexpected padding, and gives the file a header and version number. It reads that format and any of the three legacy formats, figuring out which one it's dealing with by inspecting the file contents. For good measure, it also now handles opposite-endian files, doesn't bail out unceremoniously unless necessary, and won't croak if the file is corrupt and e.g. contains unterminated strings. (Was it worth going to this length? Maybe not. But it didn't seem right to just leave it, and it's not clear where to stop halfway.) | |||||
* | ANSIfy function declarations. | 2009-05-25 | 6 | -58/+34 | ||
| | | | | | Some object file diffs, but they are harmless. (Mostly they seem to come from internal counters in gcc... and in one case the order of two instructions was harmlessly swapped, which is odd and annoying.) | |||||
* | Remove description of -a option and optional log file name, since | 2009-05-25 | 1 | -8/+10 | ||
| | | | | | | | neither has done anything in a long long time. Add previously undocumented -s/-f (slow vs. fast) options, although I question the utility thereof and suspect they should just be removed. Bump date (first time since 1993) | |||||
* | sprintf -> snprintf | 2009-05-25 | 1 | -2/+2 | ||
| | ||||||
* | sprintf -> snprintf. | 2009-05-25 | 2 | -7/+9 | ||
| | ||||||
* | "abbreviation" has two Bs. | 2009-05-25 | 5 | -20/+20 | ||
| | ||||||
* | Use random() instead of rand(), so we get something like random | 2009-05-25 | 3 | -20/+10 | ||
| | | | | | | | | | numbers out. This changes the "tournament codes"; that is, the same code will give you a different game now from what it used to. (This is because the codes are basically random seeds.) I really really doubt anyone cares about this, especially since the tournament feature appears to be undocumented. | |||||
* | Remove obviously botched test for "fast mode" based on terminal speed. | 2009-05-25 | 1 | -8/+5 | ||
| | | | | | Default to "fast mode" as ~nobody has a 300 baud terminal any more. ("Fast mode" apparently controls whether short-range scans are printed by default at certain times.) | |||||
* | Use getopt to handle the remaining (all currently undocumented) options. | 2009-05-25 | 1 | -15/+14 | ||
| | ||||||
* | Remove last traces of undocumented -p option (apparently related to | 2009-05-25 | 1 | -19/+2 | ||
| | | | | process priority) and documented -a option, neither of which have done anything since CSRG days. | |||||
* | Don't give special privileges to uid 13107. | 2009-05-25 | 1 | -9/+2 | ||
| |