Djalal Harouni <[email protected]> writes: > Do not give an fd on privileged /proc/*/pagemap files for free.
The same objections apply to this patch as it's predecessor so I won't repeat them. Eric > Restore the previous 0400 mode > > Signed-off-by: Djalal Harouni <[email protected]> > --- > fs/proc/base.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/proc/base.c b/fs/proc/base.c > index 6b162cd..93a1c89 100644 > --- a/fs/proc/base.c > +++ b/fs/proc/base.c > @@ -2605,7 +2605,7 @@ static const struct pid_entry tgid_base_stuff[] = { > #ifdef CONFIG_PROC_PAGE_MONITOR > REG("clear_refs", S_IWUSR, proc_clear_refs_operations), > REG("smaps", S_IRUGO, proc_pid_smaps_operations), > - REG("pagemap", S_IRUGO, proc_pagemap_operations), > + REG("pagemap", S_IRUSR, proc_pagemap_operations), > #endif > #ifdef CONFIG_SECURITY > DIR("attr", S_IRUGO|S_IXUGO, proc_attr_dir_inode_operations, > proc_attr_dir_operations), > @@ -2943,7 +2943,7 @@ static const struct pid_entry tid_base_stuff[] = { > #ifdef CONFIG_PROC_PAGE_MONITOR > REG("clear_refs", S_IWUSR, proc_clear_refs_operations), > REG("smaps", S_IRUGO, proc_tid_smaps_operations), > - REG("pagemap", S_IRUGO, proc_pagemap_operations), > + REG("pagemap", S_IRUSR, proc_pagemap_operations), > #endif > #ifdef CONFIG_SECURITY > DIR("attr", S_IRUGO|S_IXUGO, proc_attr_dir_inode_operations, > proc_attr_dir_operations), -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
