diff options
author | Joe Perches <joe@perches.com> | 2013-06-04 18:34:35 (GMT) |
---|---|---|
committer | Geert Uytterhoeven <geert@linux-m68k.org> | 2013-06-24 17:44:19 (GMT) |
commit | 957d6bf665462eb1e94f7c23b4bdf20a83fea4b2 (patch) | |
tree | 28ecf7f0230285508ca061a6f0c565b1fc1e389d | |
parent | 378f7ca6aa2269519b825246e63f81f95f10a63b (diff) | |
download | linux-fsl-qoriq-957d6bf665462eb1e94f7c23b4bdf20a83fea4b2.tar.xz |
swim: Release memory region after incorrect return/goto
The code uses
return foo;
goto err_type;
when instead the form should have been
ret = foo;
goto err_type;
Here this causes a useful release_mem_region to be skipped.
Signed-off-by: Joe Perches <joe@perches.com>
Reviewed-by: Laurent Vivier <Laurent@Vivier.EU>
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
-rw-r--r-- | drivers/block/swim.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/swim.c b/drivers/block/swim.c index 2f445b7..8ed6ccb 100644 --- a/drivers/block/swim.c +++ b/drivers/block/swim.c @@ -893,7 +893,7 @@ static int swim_probe(struct platform_device *dev) swim_base = ioremap(res->start, resource_size(res)); if (!swim_base) { - return -ENOMEM; + ret = -ENOMEM; goto out_release_io; } |