From 76dbaecf4fdf31c95df43690798dd6de782b19af Mon Sep 17 00:00:00 2001
From: David Barksdale <amatus.amongus@gmail.com>
Date: Thu, 30 Sep 2010 19:55:00 +0000
Subject: [PATCH] The NULL test clearly needs to be before any dereferences.

---
 src/transport/transport_api.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/transport/transport_api.c b/src/transport/transport_api.c
index 093c00bef..1eea40bb7 100644
--- a/src/transport/transport_api.c
+++ b/src/transport/transport_api.c
@@ -1810,6 +1810,13 @@ GNUNET_TRANSPORT_notify_transmit_ready (struct GNUNET_TRANSPORT_Handle
       n = neighbour_add (handle, target);
 
     }
+  if (n == NULL)
+    {
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+		  "Could not create neighbour entry for peer `%s'\n",
+		  GNUNET_i2s (target));
+      return NULL;
+    }
 
   /**
    *  Send a request connect message if not connected,
@@ -1821,13 +1828,6 @@ GNUNET_TRANSPORT_notify_transmit_ready (struct GNUNET_TRANSPORT_Handle
       send_request_connect_message(handle, n);
     }
 
-  if (n == NULL)
-    {
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-		  "Could not create neighbour entry for peer `%s'\n",
-		  GNUNET_i2s (target));
-      return NULL;
-    }
   switch (n->transmit_stage)
     {
     case TS_NEW:
-- 
2.25.1