| Commit message (Collapse) | Author | Files | Lines |
|
use after free of the scsi link structure during detach.
OK krw@
|
|
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@
|
|
program early
- #define SPAMD_USER "_spamd" and use it on getpwnam(3) call
- Set usage() as __dead void
- Remove lint-style comments
OK beck@
|
|
detect this and bump ifq_congestion forward rather than claim the
system is congested for a long period of time.
ok mpi@ henning@ jmatthew@
|
|
ok mpi@ and tweaks from sthen@
|
|
ok millert
|
|
|
|
set and get parameters. This is much simpler.
ok semarie, armani, tweaks from jmc
|
|
|
|
|
|
ever used.
ok jsg
|
|
a timer event leak. Diff from Yuuichi Someya.
|
|
I'm discussing with deraadt@ whether it's a good idea to convert some of
these to functions. The one changed by this commit probably isn't
eligible because it defines only a for loop's condition, but many others
in less(1) should probably be converted.
ok millert@
|
|
Started by diff from Mical Mazurek.
|
|
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@
|
|
|
|
says that an abbreviation cannot start with ':', and cannot contain
',', '-', '+', NUL, or a digit. POSIX from 2001 on changes this
rule to say that an abbreviation can contain only '-', '+', and
alphanumeric characters from the portable character set in the
current locale. To be portable to both sets of rules, an abbreviation
must therefore use only ASCII letters." Adapted from tzcode2015f.
OK deraadt@ mestre@
|
|
okay jmc@
|
|
This avoids wasting kernel memory if the user process does not make
use of the allocated memory.
Testing by sthen@ and tobiasu@, thanks!
ok deraadt@
|
|
This saves some memory compared to using malloc, because there's
no roundup to the next bucket size. And it reduces kmem pressure
at least for some architectures (e.g. amd64).
Testing by sthen@ and tobiasu@, thanks!
ok sthen@ deraadt@
|
|
|
|
ok deraadt@
|
|
definitions; ok mpi@
|
|
prepend chroot value *after* canonization and not before.
|
|
order ("a += b > c" is the same as "a += (b > c)" which is wrong here)
ok jca@
|
|
merge code from urtwn(4). So timeouts must move back to the bus-specific
part of the driver.
ok mpi@
|
|
|
|
|
|
XR17V354)
|
|
|
|
|
|
|
|
|
|
o inline an ugly and potentially risky macro of the form:
#define x if (a) b; else
o fix a >21 y.o. bug resulting from someone writing:
puts("[nothing to parse], stdout");
when they meant:
fputs("[nothing to parse]", stdout);
ok canacar@
|
|
All its callers got reviewed and converted to
use uiomove() properly.
ok deraadt@
|
|
o move some declarations out of hexdump.h
o mark some declarations as __dead or static
from Michal Mazurek, with input from and ok tb@
|
|
load the ldt register with the null selector (disabling use of it),
stop reloading it on every context switch, and blow away the table
itself, as well as the pcb and pmap bits that were used to track
it (making sure to keep pcb_savefpu correctly aligned).
testing naddy@
ok kettenis@ mpi@ mlarkin@
|
|
or two. Upstream wants to keep them for C++ support, and it isn't worth
maintaining a larger diff. No binary change.
Upstream commit:
https://github.com/fanf2/unifdef/commit/87dfd91
I'll send more merging diffs to tech@ soon.
|
|
ok beck@ deraadt@
|
|
|
|
|
|
still specified snprintf() and vsnprintf() based on earlier drafts.
Allows snprintf() and vsnprintf() to be used when _XOPEN_SOURCE is
defined as 500. OK guenther@
|
|
open the device only once with the correct flags, instead of twice (one
file handle for reading and one for writing). Committing this at once
because the changes are interrelated.
ok stefan
|
|
with the -F and -s flags.
ok stefan
|
|
Compile test and ok tobiasu@
|
|
ok @stefan armani@
|
|
when attempting pubkey auth with a certificate, if no separate
private key is found among the keys then try with the certificate
key itself.
bz#2550 reported by Peter Moody
|
|
After discussion with tb@, deraadt@ and millert@, this was also OK'ed by them
|
|
applications like zdump(8) because pledge(2) doesn't allow access(2) to
/usr/share/zoneinfo.
millert@ better described why this call can go away:
"This looks like an attempt to do access checks based on the real uid instead
of the effective uid. Basically for setuid programs we don't want to allow a
user to set TZ to a path they should not be able to otherwise access.
However, we already have a check for issetugid() above so I think the doaccess
bits can just be removed and we can rely on open()."
After discussion with tb@, deraadt@ and millert@, this was also OK'ed by them
|
|
|