public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
* [patch 2/2] libceph: potential NULL dereference in ceph_osdc_handle_map()
@ 2013-08-15  5:52 Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2013-08-15  5:52 UTC (permalink / raw)
  To: Sage Weil; +Cc: David S. Miller, ceph-devel, netdev, kernel-janitors

There are two places where we read "nr_maps" if both of them are set to
zero then we would hit a NULL dereference here.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
I think this is a real bug, but please review my fix for it because I'm
not very familiar with this code.

diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c
index 559a832..56f1fe5 100644
--- a/net/ceph/osd_client.c
+++ b/net/ceph/osd_client.c
@@ -1786,6 +1786,8 @@ void ceph_osdc_handle_map(struct ceph_osd_client *osdc, struct ceph_msg *msg)
 		nr_maps--;
 	}
 
+	if (!osdc->osdmap)
+		goto bad;
 done:
 	downgrade_write(&osdc->map_sem);
 	ceph_monc_got_osdmap(&osdc->client->monc, osdc->osdmap->epoch);

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2013-08-15  5:53 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-15  5:52 [patch 2/2] libceph: potential NULL dereference in ceph_osdc_handle_map() Dan Carpenter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox