Changeset View
Changeset View
Standalone View
Standalone View
sys/mips/mips/elf_machdep.c
Context not available. | |||||
#include <sys/cdefs.h> | #include <sys/cdefs.h> | ||||
__FBSDID("$FreeBSD$"); | __FBSDID("$FreeBSD$"); | ||||
#include "opt_pax.h" | |||||
#include <sys/param.h> | #include <sys/param.h> | ||||
#include <sys/kernel.h> | #include <sys/kernel.h> | ||||
#include <sys/systm.h> | #include <sys/systm.h> | ||||
Context not available. | |||||
#include <sys/linker.h> | #include <sys/linker.h> | ||||
#include <sys/sysent.h> | #include <sys/sysent.h> | ||||
#include <sys/imgact_elf.h> | #include <sys/imgact_elf.h> | ||||
#include <sys/pax.h> | |||||
#include <sys/proc.h> | #include <sys/proc.h> | ||||
#include <sys/syscall.h> | #include <sys/syscall.h> | ||||
#include <sys/signalvar.h> | #include <sys/signalvar.h> | ||||
imp: Is there any reason to not include this file when PAX_ASLR is not defined? Seems safe enough… | |||||
Not Done Inline ActionsThat's possible. I'll make that change and see what comes of it. lattera-gmail.com: That's possible. I'll make that change and see what comes of it. | |||||
Context not available. | |||||
.sv_fetch_syscall_args = cpu_fetch_syscall_args, | .sv_fetch_syscall_args = cpu_fetch_syscall_args, | ||||
.sv_syscallnames = syscallnames, | .sv_syscallnames = syscallnames, | ||||
.sv_schedtail = NULL, | .sv_schedtail = NULL, | ||||
#ifdef PAX_ASLR | |||||
.sv_pax_aslr_init = _pax_aslr_init, | |||||
#endif | |||||
}; | }; | ||||
static Elf64_Brandinfo freebsd_brand_info = { | static Elf64_Brandinfo freebsd_brand_info = { | ||||
Context not available. | |||||
.sv_fetch_syscall_args = cpu_fetch_syscall_args, | .sv_fetch_syscall_args = cpu_fetch_syscall_args, | ||||
.sv_syscallnames = syscallnames, | .sv_syscallnames = syscallnames, | ||||
.sv_schedtail = NULL, | .sv_schedtail = NULL, | ||||
#ifdef PAX_ASLR | |||||
.sv_pax_aslr_init = _pax_aslr_init32, | |||||
#endif | |||||
}; | }; | ||||
static Elf32_Brandinfo freebsd_brand_info = { | static Elf32_Brandinfo freebsd_brand_info = { | ||||
Context not available. |
Is there any reason to not include this file when PAX_ASLR is not defined? Seems safe enough and would reduce the number of ifdefs this patch introduces significantly.