On Wed, Apr 20, 2016 at 11:52:47PM -0700, John Johansen wrote: > Signed-off-by: John Johansen <[email protected]>
Acked-by: Seth Arnold <[email protected]> I think this patch should also set i_ctime to match i_mtime -- a quick check through fs/ showed dozens of examples of both i_mtime and i_ctime being set together but only a handful of cases where i_ctime wasn't set within three lines of i_mtime. Thanks > --- > security/apparmor/apparmorfs.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/security/apparmor/apparmorfs.c b/security/apparmor/apparmorfs.c > index ad4fa49..45a6199 100644 > --- a/security/apparmor/apparmorfs.c > +++ b/security/apparmor/apparmorfs.c > @@ -379,6 +379,8 @@ void __aa_fs_profile_migrate_dents(struct aa_profile *old, > > for (i = 0; i < AAFS_PROF_SIZEOF; i++) { > new->dents[i] = old->dents[i]; > + if (new->dents[i]) > + new->dents[i]->d_inode->i_mtime = CURRENT_TIME; > old->dents[i] = NULL; > } > }
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
