diff options
| author | Dan Carpenter <error27@gmail.com> | 2011-03-29 05:25:59 +0200 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2011-03-29 17:59:54 +0200 |
| commit | 234af26ff123dfb2aa48772124721b1354c8e0a5 (patch) | |
| tree | 9dfaaeb25a5b1265269796ebc48cab93d64fc1f2 /net/ceph | |
| parent | ceph: fix possible NULL pointer dereference (diff) | |
| download | linux-234af26ff123dfb2aa48772124721b1354c8e0a5.tar.xz linux-234af26ff123dfb2aa48772124721b1354c8e0a5.zip | |
ceph: unlock on error in ceph_osdc_start_request()
There was a missing unlock on the error path if __map_request() failed.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'net/ceph')
| -rw-r--r-- | net/ceph/osd_client.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index b6776cb627cc..03740e8fc9d1 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c @@ -1672,7 +1672,7 @@ int ceph_osdc_start_request(struct ceph_osd_client *osdc, if (req->r_sent == 0) { rc = __map_request(osdc, req); if (rc < 0) - return rc; + goto out_unlock; if (req->r_osd == NULL) { dout("send_request %p no up osds in pg\n", req); ceph_monc_request_next_osdmap(&osdc->client->monc); @@ -1689,6 +1689,8 @@ int ceph_osdc_start_request(struct ceph_osd_client *osdc, } } } + +out_unlock: mutex_unlock(&osdc->request_mutex); up_read(&osdc->map_sem); return rc; |