diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2017-09-11 22:43:23 +0300 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2017-09-11 23:16:44 +0300 |
| commit | 0b045bd1c1c2819b33f4522e3efa4666d1ecf1a4 (patch) | |
| tree | 33fb862c96efa57895b6a67980f8e27c0c24363b /mm | |
| parent | 09c2c359be546df45be0b158ea1d3cc8ea83c876 (diff) | |
| download | linux-0b045bd1c1c2819b33f4522e3efa4666d1ecf1a4.tar.xz | |
mm/backing-dev.c: fix an error handling path in 'cgwb_create()'
If the 'kmalloc' fails, we must go through the existing error handling
path.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Fixes: 52ebea749aae ("writeback: make backing_dev_info host cgroup-specific bdi_writebacks")
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'mm')
| -rw-r--r-- | mm/backing-dev.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/mm/backing-dev.c b/mm/backing-dev.c index f028a9a472fd..e19606bb41a0 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c @@ -569,8 +569,10 @@ static int cgwb_create(struct backing_dev_info *bdi, /* need to create a new one */ wb = kmalloc(sizeof(*wb), gfp); - if (!wb) - return -ENOMEM; + if (!wb) { + ret = -ENOMEM; + goto out_put; + } ret = wb_init(wb, bdi, blkcg_css->id, gfp); if (ret) |