Re: [PATCH] cgroup: initialize xattr before calling d_instantiate()

From: Tejun Heo
Date: Tue May 14 2013 - 11:38:20 EST


On Tue, May 14, 2013 at 07:44:20PM +0800, Li Zefan wrote:
> cgroup_create_file() calls d_instantiate(), which may decide to look
> at the xattrs on the file. Smack always does this and SELinux can be
> configured to do so.
>
> But cgroup_add_file() didn't initialize xattrs before calling
> cgroup_create_file(), which finally leads to dereferencing NULL
> dentry->d_fsdata.
>
> This bug has been there since cgroup xattr was introduced.
>
> Cc: <stable@xxxxxxxxxxxxxxx> # 3.8.x
> Reported-by: Ivan Bulatovic <combuster@xxxxxxxxxxxx>
> Reported-by: Casey Schaufler <casey@xxxxxxxxxxxxxxxx>
> Signed-off-by: Li Zefan <lizefan@xxxxxxxxxx>

Applied to cgroup/for-3.10-fixes.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/