On 07/11/2012 02:12 PM, Josh Durgin wrote: >> /* >> @@ -1711,15 +1721,17 @@ static int __rbd_refresh_header(struct >> rbd_device *rbd_dev) >> if head moves */ >> follow_seq = 1; >> >> - kfree(rbd_dev->header.snapc); >> - kfree(rbd_dev->header.snap_names); >> + kfree(rbd_dev->header.object_prefix); > > Object prefix isn't going to change, so it doesn't need to be reset > here. Otherwise looks good. In that case I will verify that and warn if they do differ. -Alex -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html