Messages in this thread | | | Subject | Re: [PATCH v2] clear file privilege bits when mmap writing | From | yalin wang <> | Date | Thu, 3 Dec 2015 17:45:06 -0800 |
| |
> On Dec 2, 2015, at 16:03, Kees Cook <keescook@chromium.org> wrote: > > Normally, when a user can modify a file that has setuid or setgid bits, > those bits are cleared when they are not the file owner or a member > of the group. This is enforced when using write and truncate but not > when writing to a shared mmap on the file. This could allow the file > writer to gain privileges by changing a binary without losing the > setuid/setgid/caps bits. > > Changing the bits requires holding inode->i_mutex, so it cannot be done > during the page fault (due to mmap_sem being held during the fault). > Instead, clear the bits if PROT_WRITE is being used at mmap time. > > Signed-off-by: Kees Cook <keescook@chromium.org> > Cc: stable@vger.kernel.org > —
is this means mprotect() sys call also need add this check? mprotect() can change to PROT_WRITE, then it can write to a read only map again , also a secure hole here .
Thanks
| |