diff options
| author | Tejun Heo <tj@kernel.org> | 2013-05-14 13:52:30 -0700 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2013-05-14 13:52:30 -0700 |
| commit | 2423c9c3f0ffffa8f87cbdafe9781273c5d1b6a2 (patch) | |
| tree | 6ddb6e8f4cd577adab194c90eccec5ad50f2ab08 /block | |
| parent | f722406faae2d073cc1d01063d1123c35425939e (diff) | |
| download | linux-2423c9c3f0ffffa8f87cbdafe9781273c5d1b6a2.tar.bz2 | |
blkcg: fix error return path in blkg_create()
In blkg_create(), after lookup of parent fails, the control jumps to
error path with the error code encoded into @blkg. The error path
doesn't use @blkg for the return value. It returns ERR_PTR(ret).
Make lookup fail path set @ret instead of @blkg.
Note that the parent lookup is guaranteed to succeed at that point and
the condition check is purely for sanity and triggers WARN when fails.
As such, I don't think it's necessary to mark it for -stable.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Vivek Goyal <vgoyal@redhat.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/blk-cgroup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/blk-cgroup.c b/block/blk-cgroup.c index e8918ffaf96d..7fc35f683c4e 100644 --- a/block/blk-cgroup.c +++ b/block/blk-cgroup.c @@ -238,7 +238,7 @@ static struct blkcg_gq *blkg_create(struct blkcg *blkcg, if (blkcg_parent(blkcg)) { blkg->parent = __blkg_lookup(blkcg_parent(blkcg), q, false); if (WARN_ON_ONCE(!blkg->parent)) { - blkg = ERR_PTR(-EINVAL); + ret = -EINVAL; goto err_put_css; } blkg_get(blkg->parent); |