aboutsummaryrefslogtreecommitdiff
path: root/lib/flex_array.c
diff options
context:
space:
mode:
authorAlex Elder <elder@inktank.com>2012-10-25 23:34:40 -0500
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-01-17 08:46:49 -0800
commita35420155f46065b5fded0a7478fff003be4e872 (patch)
tree3de888a256f1d90120051525291f7b1d908a360a /lib/flex_array.c
parentfefc034ae0af85c0b18fe209a1b6803d51880556 (diff)
rbd: remove snapshots on error in rbd_add()
(cherry picked from commit 41f38c2b2f8b66b176a0e548ef06294343a7bfa2) If rbd_dev_snaps_update() has ever been called for an rbd device structure there could be snapshot structures on its snaps list. In rbd_add(), this function is called but a subsequent error path neglected to clean up any of these snapshots. Add a call to rbd_remove_all_snaps() in the appropriate spot to remedy this. Change a couple of error labels to be a little clearer while there. Drop the leading underscores from the function name; there's nothing special about that function that they might signify. As suggested in review, the leading underscores in __rbd_remove_snap_dev() have been removed as well. Signed-off-by: Alex Elder <elder@inktank.com> Reviewed-by: Josh Durgin <josh.durgin@inktank.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'lib/flex_array.c')
0 files changed, 0 insertions, 0 deletions