Date: Tue, 02 Sep 2014 15:59:17 -0700 From: Sean Bruno <sbruno@ignoranthack.me> To: freebsd-current <freebsd-current@FreeBSD.org> Subject: [PATCH]Modify do_exec() handler to deal with multiple imgact handlers Message-ID: <1409698757.55485.8.camel@bruno>
next in thread | raw e-mail | index | archive | help
https://reviews.freebsd.org/D696 I found that the binmisc handler was not executing if the shell handler fired. Both were using the same intepreted flag to determine if they should run. This change modifies struct image_params.interpreted to be a bitfield instead of a bool flag and assigns one bit to each image activator. Comments? sean Index: sys/kern/imgact_binmisc.c =================================================================== --- sys/kern/imgact_binmisc.c +++ sys/kern/imgact_binmisc.c @@ -600,12 +600,12 @@ } /* No interpreter nesting allowed. */ - if (imgp->interpreted) { + if (imgp->interpreted & IMGACT_BINMISC) { mtx_unlock(&interp_list_mtx); return (ENOEXEC); } - imgp->interpreted = 1; + imgp->interpreted |= IMGACT_BINMISC; if (imgp->args->fname != NULL) { fname = imgp->args->fname; Index: sys/kern/imgact_shell.c =================================================================== --- sys/kern/imgact_shell.c +++ sys/kern/imgact_shell.c @@ -115,10 +115,10 @@ * Don't allow a shell script to be the shell for a shell * script. :-) */ - if (imgp->interpreted) + if (imgp->interpreted & IMGACT_SHELL) return (ENOEXEC); - imgp->interpreted = 1; + imgp->interpreted |= IMGACT_SHELL; /* * At this point we have the first page of the file mapped. Index: sys/sys/imgact.h =================================================================== --- sys/sys/imgact.h +++ sys/sys/imgact.h @@ -61,7 +61,9 @@ unsigned long entry_addr; /* entry address of target executable */ unsigned long reloc_base; /* load address of image */ char vmspace_destroyed; /* flag - we've blown away original vm space */ - char interpreted; /* flag - this executable is interpreted */ +#define IMGACT_SHELL 0x1 +#define IMGACT_BINMISC 0x2 + unsigned char interpreted; /* mask of interpretes that have run */ char opened; /* flag - we have opened executable vnode */ char *interpreter_name; /* name of the interpreter */ void *auxargs; /* ELF Auxinfo structure pointer */
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1409698757.55485.8.camel>