All of lore.kernel.org
 help / color / mirror / Atom feed
* [Patch 1/1] tabled: fix a crash when looking up non-existing NID
@ 2010-03-29  1:57 Pete Zaitcev
  2010-03-29 20:24 ` Jeff Garzik
  0 siblings, 1 reply; 2+ messages in thread
From: Pete Zaitcev @ 2010-03-29  1:57 UTC (permalink / raw)
  To: jeff; +Cc: Project Hail List

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>

---
 server/storage.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/server/storage.c b/server/storage.c
index 29e2435..913ef33 100644
--- a/server/storage.c
+++ b/server/storage.c
@@ -406,7 +406,8 @@ struct storage_node *stor_node_by_nid(uint32_t nid)
 
 	g_mutex_lock(tabled_srv.bigmutex);
 	sn = _stor_node_by_nid(nid);
-	stor_node_get(sn);
+	if (sn)
+		stor_node_get(sn);
 	g_mutex_unlock(tabled_srv.bigmutex);
 	return sn;
 }

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Patch 1/1] tabled: fix a crash when looking up non-existing NID
  2010-03-29  1:57 [Patch 1/1] tabled: fix a crash when looking up non-existing NID Pete Zaitcev
@ 2010-03-29 20:24 ` Jeff Garzik
  0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2010-03-29 20:24 UTC (permalink / raw)
  To: Pete Zaitcev; +Cc: Project Hail List

On 03/28/2010 09:57 PM, Pete Zaitcev wrote:
> Signed-off-by: Pete Zaitcev<zaitcev@redhat.com>
>
> ---
>   server/storage.c |    3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)

applied


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-03-29 20:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-29  1:57 [Patch 1/1] tabled: fix a crash when looking up non-existing NID Pete Zaitcev
2010-03-29 20:24 ` Jeff Garzik

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.