Commit ec7cf783 authored by Jeff Dike's avatar Jeff Dike Committed by Linus Torvalds

[PATCH] uml: error path cleanup

This cleans up the error path in ubd_open, causing it now to call ubd_close
appropriately when something fails.
Signed-off-by: default avatarJeff Dike <jdike@addtoit.com>
Cc: Paolo Giarrusso <blaisorblade@yahoo.it>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 7efd08c8
...@@ -668,21 +668,22 @@ static int ubd_add(int n) ...@@ -668,21 +668,22 @@ static int ubd_add(int n)
struct ubd *dev = &ubd_dev[n]; struct ubd *dev = &ubd_dev[n];
int err; int err;
err = -ENODEV;
if(dev->file == NULL) if(dev->file == NULL)
return(-ENODEV); goto out;
if (ubd_open_dev(dev)) if (ubd_open_dev(dev))
return(-ENODEV); goto out;
err = ubd_file_size(dev, &dev->size); err = ubd_file_size(dev, &dev->size);
if(err < 0) if(err < 0)
return(err); goto out_close;
dev->size = ROUND_BLOCK(dev->size); dev->size = ROUND_BLOCK(dev->size);
err = ubd_new_disk(MAJOR_NR, dev->size, n, &ubd_gendisk[n]); err = ubd_new_disk(MAJOR_NR, dev->size, n, &ubd_gendisk[n]);
if(err) if(err)
return(err); goto out_close;
if(fake_major != MAJOR_NR) if(fake_major != MAJOR_NR)
ubd_new_disk(fake_major, dev->size, n, ubd_new_disk(fake_major, dev->size, n,
...@@ -693,8 +694,11 @@ static int ubd_add(int n) ...@@ -693,8 +694,11 @@ static int ubd_add(int n)
if (fake_ide) if (fake_ide)
make_ide_entries(ubd_gendisk[n]->disk_name); make_ide_entries(ubd_gendisk[n]->disk_name);
err = 0;
out_close:
ubd_close(dev); ubd_close(dev);
return 0; out:
return err;
} }
static int ubd_config(char *str) static int ubd_config(char *str)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment