Delete obsolete /* ARGSUSED */ lint comments.

ok miod@ millert@

When it's the possessive of 'it', it's spelled "its", without the
apostrophe.

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

change if ((a == b)) to if (a == b); silences noisy on-by-default
"equality comparison with extraneous parentheses
[-Wparentheses-equality]" clang warning.
OK deraadt, kettenis

Replace remaining calls to index(3) with strchr(3). OK jca@ krw@

Teach mopd and mopa.out about ELF files, and allow forthcoming VAX ELF boot
blocks to be converted to working mop binaries. From NetBSD.

Make life easier for parfait. I corporation with deraadt. -moj

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

Change the compile time option -DINFO into a runtime option to get ride
of some lint warnings. -moj

Remove 'ipc' in PLT message and replace it with current hostname.
Some more code cleanup and make lint happier. -moj

more cleanup

chroot and drop privileges to _mopd:_mopd after initialization.
testing and ok hshoexer@

~1000 lines worth of ansi, KNF, various cleaning, replacing a homebrown
daemon() equivalent and such things, theo ok

-Wall cleanup: trim unused vars, right format strings, constify rcsids,
braces where needed, add missing prototypes.

tested and ok maja@

remove clause 3 and 4. -moj

pedantic ansify. -moj

use snprintf

File of the week :-) If Tertiary Loader data link size might not be set.
Since Ethernet is the only data link supported assume datalink to be
1492. Found when ragge@ludd.luth.se tried to boot a VAX 6

File of the week :-) If Tertiary Loader data link size might not be set.
Since Ethernet is the only data link supported assume datalink to be
1492. Found when ragge@ludd.luth.se tried to boot a VAX 6000/400. -moj

show more ...

If the spec allows 128-character filenames, our buffer has to be at least 129
characters to hold the trailing NULL. This also fixes a one-byte overflow.

typo

Ignore filenames that are longer than spec allows (128 for MOP V4.0.0).
Matt Power <mhpower@mit.edu>. -moj

creating line ourselves, but just in case, use %s with syslog

string manipluation paranoia changes. Most probably not exploitable.

Replace $Id with $OpenBSD, 3:rd try. -moj

Initial import of mopd-2.5.3. -moj