diff options
author | Alex Elder <elder@inktank.com> | 2012-07-13 21:35:11 -0400 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2012-07-30 21:21:46 -0400 |
commit | d67d4be56a3ec8d07b1f29aab6095b363085b028 (patch) | |
tree | 9765e2fd27836f111d12d3120963f1beb2318b53 /drivers/block | |
parent | 57cfc1060f35ac2345cb37ea474f9644ac5cfd75 (diff) |
rbd: pass null version pointer in add_snap()
rbd_header_add_snap() passes the address of a version variable to
rbd_req_sync_exec(), but it ignores the result. Just pass a null
pointer instead.
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/rbd.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 28670c0c68d5..e1fa12b2ae2e 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c | |||
@@ -1668,7 +1668,6 @@ static int rbd_header_add_snap(struct rbd_device *rbd_dev, | |||
1668 | u64 new_snapid; | 1668 | u64 new_snapid; |
1669 | int ret; | 1669 | int ret; |
1670 | void *data, *p, *e; | 1670 | void *data, *p, *e; |
1671 | u64 ver; | ||
1672 | struct ceph_mon_client *monc; | 1671 | struct ceph_mon_client *monc; |
1673 | 1672 | ||
1674 | /* we should create a snapshot only if we're pointing at the head */ | 1673 | /* we should create a snapshot only if we're pointing at the head */ |
@@ -1693,7 +1692,7 @@ static int rbd_header_add_snap(struct rbd_device *rbd_dev, | |||
1693 | 1692 | ||
1694 | ret = rbd_req_sync_exec(rbd_dev, rbd_dev->header_name, | 1693 | ret = rbd_req_sync_exec(rbd_dev, rbd_dev->header_name, |
1695 | "rbd", "snap_add", | 1694 | "rbd", "snap_add", |
1696 | data, p - data, &ver); | 1695 | data, p - data, NULL); |
1697 | 1696 | ||
1698 | kfree(data); | 1697 | kfree(data); |
1699 | 1698 | ||