| Age | Commit message (Collapse) | Author |
|---|
|
noticed by joao <salvatti at gmail dot com> on tech@;
feedback and ok kettenis@ deraadt@ and reminded by jmc@
|
|
to 2GB of vm space.
|
|
Adapted from FreeBSD. OK deraadt@
|
|
ok tedu@ deraadt@ krw@
|
|
fixing a regression introduced in rev. 1.16 spotted by otto@;
ok millert@ otto@
|
|
|
|
|
|
wording tweaked by jmc@
|
|
trying to allocate large blocks from bss memory pool in this case.
problem reported by Maksymilian Arciemowicz. ok otto@, millert@
|
|
|
|
to u_int32_t to do integer math with (in a situation where that is legit)
ok otto millert
|
|
If code is used from exactly one place, don't jump back dozens of lines to
reach it, only to "goto" back where you came from.
Instead, simply put the code where it belongs.
Also fixes a regression that crept in in rev. 1.30: After clearing the
variable __ypmode, don't try to make decisions based on its former value.
As a bonus, garbage collect the grname variable and the _ypmode enum type.
ok millert@
|
|
|
|
along with vnode type-specific info to make it more useful for fstat(1).
OK deraadt@
|
|
OK deraadt@
|
|
Thus, garbage collect one variable, one strdup, one free, two ifs,
one else and a couple of assignments. No functional change.
ok millert@
|
|
invalid data on the list, inviting later NULL pointer access.
noticed by deraadt@, algorithm proposed by millert, implemented by me;
feedback and ok millert@
|
|
noticed by and OK deraadt@, ok millert@
|
|
|
|
|
|
|
|
|
|
occurred". Based on this, decide to either bail or use the record as is.
Prevents getgrnam(3) and friends from silently skipping YP records
on system errors, for example when out of memory.
Also, calling yp_get_default_domain once is enough.
The first two switch cases are unchanged, just reindented.
ok millert@
|
|
ok millert@
|
|
map ECONNREFUSED -> YPERR_YPBIND, ENOMEM -> YPERR_RESRC, else YPERR_YPERR
while here, malloc(3) failure should raise YPERR_RESRC, not YPERR_YPERR
ok millert@
|
|
check whether YP is active up front, not half-way through the parsing.
This fixes the bug that group names starting with a dash (yuck!)
were skipped when YP was not active, introduced in rev. 1.27.
Besides, the code becomes easier to understand.
The code inside the two switch cases is unchanged, just reindented.
"looks good" millert@
|
|
so treat it similarly to a failing call to getdomainname.
ok millert@
|
|
ok millert
|
|
ok otto@
|
|
now doing.
ok deraadt@
|
|
bounds information, ie. the zone of the disk that OpenBSD can use. Have each
pre-disklabel parser (MBR, DPME, or per-arch MD disklabel parsers) figure out
this area and pass it up to userland. Then, delete all the same disk parsing
code from disklabel(8) since the kernel passes it up. Lots and lots of - signs
in the disklabel(8) code.
Tested on as many platforms as possible, the fallout will be repaired as time
goes on. To test, use disklabel -d <drive> and validate that the bounds do not
overlap any boot blocks. This same information is used by disklabel -A...
OK for the concept from krw, miod, and drahn
|
|
|
|
Someone may have passed a read-only string to putenv() (I'm looking
at you cron!).
|
|
PF_UNSPEC queries are made. While there change the default from inet6
first then inet4 to inet4 first then inet6, this prevents the many
people with IPv4 only connectivity from constantly trying to contact
IPv6 addresses, and also unbreaks many ports who don't use getaddrinfo
right.
ok deraadt@, plenty of cheering in the room wrt the idea, not loud
enough complaining from the v6 crowd.
|
|
do not just skip the entry, but report the error condition;
ok millert@
|
|
Bail out and clean up after all errors, and report in libc/rpc style.
Among others, this fixes a NULL pointer dereference in clntraw_call.
Feedback and ok kjell@ and millert@.
|
|
"no objection" millert@
|
|
ok deraadt@
|
|
|
|
YP group(5) exclusion, i.e. support -groupname:*:: in /etc/group.
Such groups will be excluded from later +:*::,
in just the same was as it is already done for passwd(5).
I have been running this since the autumn of 2008.
Discussed with several (including deraadt@, millert@, jmc@).
|
|
standard explicitly disallows passing setenv a name with a '=' in
it but historic BSD behavior is to allow this but to ignore the '='
and anything after it.
|
|
arches. ok todd@ beck@
|
|
That way, memory exhaustion simply cannot be reported because it would
require to... allocate memory.
Instead, really use a statical buffer as it is advertised in the manual.
"looks good" millert@
|
|
OK millert otto
|
|
hint from claudio@, ok millert@
|
|
because that one will typically derefenced later without any checking.
Instead, flag the error condition and bail out cleanly.
useful feedback from kjell@ and ok millert@
|
|
ok millert@ jmc@
|
|
|
|
an error message, rather skip the error message than dereferencing
a NULL pointer.
debugging help and ok kjell@
|
|
will help people understand that endpwent() is not normally needed.
OK deraadt@ jmc@
|
