aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/block/rbd.c
diff options
context:
space:
mode:
authorSage Weil <sage@newdream.net>2011-05-12 19:10:50 -0400
committerSage Weil <sage@newdream.net>2011-05-24 14:52:01 -0400
commit916d4d672779de8e42346fff338617c7b841e8e5 (patch)
treeaa323878aa43221442b03137614f5f0a4251171c /drivers/block/rbd.c
parent13143d2d1cffd243a6d778000b02ab4938ac751a (diff)
rbd: cleanup: make kfree match kmalloc
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'drivers/block/rbd.c')
-rw-r--r--drivers/block/rbd.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 78d0011a7556..0d80967e140c 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -1602,7 +1602,7 @@ static int rbd_header_add_snap(struct rbd_device *dev,
1602 int name_len = strlen(snap_name); 1602 int name_len = strlen(snap_name);
1603 u64 new_snapid; 1603 u64 new_snapid;
1604 int ret; 1604 int ret;
1605 void *data, *data_start, *data_end; 1605 void *data, *p, *e;
1606 u64 ver; 1606 u64 ver;
1607 1607
1608 /* we should create a snapshot only if we're pointing at the head */ 1608 /* we should create a snapshot only if we're pointing at the head */
@@ -1619,16 +1619,16 @@ static int rbd_header_add_snap(struct rbd_device *dev,
1619 if (!data) 1619 if (!data)
1620 return -ENOMEM; 1620 return -ENOMEM;
1621 1621
1622 data_start = data; 1622 p = data;
1623 data_end = data + name_len + 16; 1623 e = data + name_len + 16;
1624 1624
1625 ceph_encode_string_safe(&data, data_end, snap_name, name_len, bad); 1625 ceph_encode_string_safe(&p, e, snap_name, name_len, bad);
1626 ceph_encode_64_safe(&data, data_end, new_snapid, bad); 1626 ceph_encode_64_safe(&p, e, new_snapid, bad);
1627 1627
1628 ret = rbd_req_sync_exec(dev, dev->obj_md_name, "rbd", "snap_add", 1628 ret = rbd_req_sync_exec(dev, dev->obj_md_name, "rbd", "snap_add",
1629 data_start, data - data_start, &ver); 1629 data, p - data, &ver);
1630 1630
1631 kfree(data_start); 1631 kfree(data);
1632 1632
1633 if (ret < 0) 1633 if (ret < 0)
1634 return ret; 1634 return ret;