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 <machine/md_var.h> | #include <machine/md_var.h> | ||||
#include <machine/cache.h> | #include <machine/cache.h> | ||||
#ifdef PAX_ASLR | |||||
#include <sys/pax.h> | |||||
#endif | |||||
imp: Is there any reason to not include this file when PAX_ASLR is not defined? Seems safe enough… | |||||
lattera-gmail.comAuthorUnsubmitted 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. | |||||
#ifdef __mips_n64 | #ifdef __mips_n64 | ||||
struct sysentvec elf64_freebsd_sysvec = { | struct sysentvec elf64_freebsd_sysvec = { | ||||
.sv_size = SYS_MAXSYSCALL, | .sv_size = SYS_MAXSYSCALL, | ||||
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, | |||||
#else | |||||
.sv_pax_aslr_init = NULL, | |||||
#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, | |||||
#else | |||||
.sv_pax_aslr_init = NULL, | |||||
#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.