Messages in this thread | | | Date | Thu, 8 Mar 2012 14:14:29 -0800 | Subject | Re: [RFC] c/r: prctl: Add ability to set new mm_struct::exe_file v3 | From | Kees Cook <> |
| |
On Thu, Mar 8, 2012 at 2:12 PM, Cyrill Gorcunov <gorcunov@openvz.org> wrote: > On Thu, Mar 08, 2012 at 02:03:13PM -0800, Kees Cook wrote: >> >> How about "file_is_exec" instead, since it doesn't (and likely >> shouldn't) include the inode_permission(..., EXEC)? I'd like other >> people's thoughts on this since maybe it's not needed and I instead >> have accidentally derailed this patch with useless bike shedding. >> > > Yup. Anyway, I'm leaving the former mm_struct::exe_file patch with > EXEC test opencoded so we can do everything on top then.
Sounds good to me. Thanks for putting up with my distraction! :)
-Kees
-- Kees Cook ChromeOS Security
| |