| #
359652 |
|
06-Apr-2020 |
hselasky |
MFC r333806:
Use NULL for SYSINIT's last arg, which is a pointer type
Sponsored by: The FreeBSD Foundation
|
| #
318293 |
|
15-May-2017 |
emaste |
MFC r315685: tighten buffer bounds in imgact_binmisc_populate_interp
We must ensure there's space for the terminating null in the temporary
buffer in imgact_binmisc_populate_interp().
Note that there's no buffer overflow here because xbe->xbe_interpreter's
length and null termination is checked in imgact_binmisc_add_entry()
before imgact_binmisc_populate_interp() is called. However, the latter
should correctly enforce its own bounds.
Sponsored by: The FreeBSD Foundation
|
| #
302408 |
|
07-Jul-2016 |
gjb |
Copy head@r302406 to stable/11 as part of the 11.0-RELEASE cycle.
Prune svn:mergeinfo from the new branch, as nothing has been merged
here.
Additional commits post-branch will follow.
Approved by: re (implicit)
Sponsored by: The FreeBSD Foundation |
| #
298819 |
|
29-Apr-2016 |
pfg |
sys/kern: spelling fixes in comments.
No functional change.
|
| #
297488 |
|
01-Apr-2016 |
sbruno |
Repair a overflow condition where a user could submit a string that was
not getting a proper bounds check.
Thanks to CTurt for pointing at this with a big red blinking neon sign.
PR: 206761
Submitted by: sson
Reviewed by: cturt@hardenedbsd.org
MFC after: 3 days
|
| #
284766 |
|
24-Jun-2015 |
sbruno |
At the suggestion of jhb, replace atomic_set/clear calls with use of
exclusive locks in the enable/disable interpreter path.
Tested with WITNESS/INVARIANTS on and off.
Reviewed by: sson davide
|
| #
284701 |
|
22-Jun-2015 |
sbruno |
Make imgact_binmisc_exec() static.
Submitted by: kib
Reviewed by: sson
|
| #
284606 |
|
19-Jun-2015 |
sbruno |
Remove uneeded NULL check since malloc the malloc is now M_WAITOK
Submitted by: mjg
|
| #
284605 |
|
19-Jun-2015 |
sbruno |
Must have one of either M_WAITOK or M_NOWAIT, read the man page bruno.
Submitted by: mjg
|
| #
284602 |
|
19-Jun-2015 |
sbruno |
Feedback from commit r284535
davide: imgact_binmisc_clear_entry() needs to use atomic ops to remove
the enable bit.
kib: M_NOWAIT is not warranted and comment is invalid.
|
| #
284535 |
|
18-Jun-2015 |
sbruno |
This change replaces the mutex with a sx lock for the interpreter list to
avoid the problem of holding a non-sleep lock during a page fault as
reported by witness. It also uses atomics where possible to avoid having
to acquire the exclusive lock. In addition, it consistently uses
memset()/memcpy() instead of bzero()/bcopy().
Differential Revision: https://reviews.freebsd.org/D1971
Submitted by: sson
Reviewed by: jhb
|
| #
284043 |
|
05-Jun-2015 |
sbruno |
Revert 284029, update imgact_binmisctl.c change mtx to reader count, at the
request of the submitter.
Will attempt to use an sx_lock for this fix to WITNESS crashes in a later
revision.
Submitted by: sson
|
| #
284029 |
|
05-Jun-2015 |
sbruno |
This change uses a reader count instead of holding the mutex for the
interpreter list to avoid the problem of holding a non-sleep lock during
a page fault as reported by witness. In addition, it consistently uses
memset()/memcpy() instead of bzero()/bcopy() except in the case where
bcopy() is required (i.e. overlapping copy).
Differential Revision: https://reviews.freebsd.org/D2123
Submitted by: sson
MFC after: 2 weeks
Relnotes: Yes
|
| #
271141 |
|
04-Sep-2014 |
sbruno |
Allow multiple image activators to run on the same execution by changing
imgp->interpreted to a bitmask instead of, functionally, a bool. Each
imgactivator now requires its own flag in interpreted to indicate whether
or not it has already examined argv[0].
Change imgp->interpreted to an unsigned char to add one extra bit for
future use.
With this change, one can execute a shell script from a 64bit host native
make and still get the binmisc image activator to fire for the script
interpreter. Prior to this, execution would fail.
Phabric: https://reviews.freebsd.org/D696
Reviewed by: jhb@
MFC after: 4 weeks
|
| #
264282 |
|
08-Apr-2014 |
sbruno |
sys/kern/imgact_binmisc.c -- free the right pointer mask vs magic
sys/sys/imagact_binmisc.h -- cleanup white space tabs vs spaces
-- remove stray " in comment
Submitted by: jmallett@
|
| #
264269 |
|
08-Apr-2014 |
sbruno |
Add Stacey Son's binary activation patches that allow remapping of
execution to a emumation program via parsing of ELF header information.
With this kernel module and userland tool, poudriere is able to build
ports packages via the QEMU userland tools (or another emulator program)
in a different architecture chroot, e.g. TARGET=mips TARGET_ARCH=mips
I'm not connecting this to GENERIC for obvious reasons, but this should
allow the kernel module to be built by default and enable the building
of the userland tool (which automatically loads the kernel module).
Submitted by: sson@
Reviewed by: jhb@
|