* Re: [PATCH] xfstests: test dump/restore of file capabilities
[not found] <50E61CC3.1000108@redhat.com>
@ 2013-01-29 22:44 ` Ben Myers
2013-01-30 18:18 ` Ben Myers
1 sibling, 0 replies; 2+ messages in thread
From: Ben Myers @ 2013-01-29 22:44 UTC (permalink / raw)
To: Eric Sandeen; +Cc: fugazzi®, xfs-oss
Eric,
On Thu, Jan 03, 2013 at 06:05:23PM -0600, Eric Sandeen wrote:
> xfsrestore isn't properly restoring file capabilities; it restores
> them, but then chowns the file, which removes the caps:
>
> Call Trace:
> [<ffffffffa028de68>] xfs_xattr_set+0x118/0x120 [xfs]
> [<ffffffff8119a8c0>] generic_removexattr+0x80/0x90
> [<ffffffff8120b408>] cap_inode_killpriv+0x28/0x30
> [<ffffffff8120c666>] security_inode_killpriv+0x16/0x20
> [<ffffffff81192edf>] notify_change+0x18f/0x330
> [<ffffffff81176b70>] chown_common+0x60/0xa0
> [<ffffffff81176c30>] sys_fchown+0x80/0xd0
> [<ffffffff81537c59>] system_call_fastpath+0x16/0x1b
>
> This test demonstrates the problem.
>
> Reported-by: fugazzi® <fugazzi99@gmail.com>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
This looks good to me.
Reviewed-by: Ben Myers <bpm@sgi.com>
-Ben
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] xfstests: test dump/restore of file capabilities
[not found] <50E61CC3.1000108@redhat.com>
2013-01-29 22:44 ` [PATCH] xfstests: test dump/restore of file capabilities Ben Myers
@ 2013-01-30 18:18 ` Ben Myers
1 sibling, 0 replies; 2+ messages in thread
From: Ben Myers @ 2013-01-30 18:18 UTC (permalink / raw)
To: Eric Sandeen; +Cc: fugazzi®, xfs-oss
On Thu, Jan 03, 2013 at 06:05:23PM -0600, Eric Sandeen wrote:
> xfsrestore isn't properly restoring file capabilities; it restores
> them, but then chowns the file, which removes the caps:
>
> Call Trace:
> [<ffffffffa028de68>] xfs_xattr_set+0x118/0x120 [xfs]
> [<ffffffff8119a8c0>] generic_removexattr+0x80/0x90
> [<ffffffff8120b408>] cap_inode_killpriv+0x28/0x30
> [<ffffffff8120c666>] security_inode_killpriv+0x16/0x20
> [<ffffffff81192edf>] notify_change+0x18f/0x330
> [<ffffffff81176b70>] chown_common+0x60/0xa0
> [<ffffffff81176c30>] sys_fchown+0x80/0xd0
> [<ffffffff81537c59>] system_call_fastpath+0x16/0x1b
>
> This test demonstrates the problem.
>
> Reported-by: fugazzi® <fugazzi99@gmail.com>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Applied.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-01-30 18:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <50E61CC3.1000108@redhat.com>
2013-01-29 22:44 ` [PATCH] xfstests: test dump/restore of file capabilities Ben Myers
2013-01-30 18:18 ` Ben Myers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox