eliminate nonsensical compare
authorSchanzenbach, Martin <mschanzenbach@posteo.de>
Thu, 1 Aug 2019 08:31:44 +0000 (10:31 +0200)
committerSchanzenbach, Martin <mschanzenbach@posteo.de>
Thu, 1 Aug 2019 08:31:44 +0000 (10:31 +0200)
src/cadet/gnunet-service-cadet_peer.c

index c25f46de58eac88c5ce94321f411794bcfe58ae8..d6cc6f914d0995ef3fa1204a368afb3169aeaf0b 100644 (file)
@@ -11,7 +11,7 @@
      WITHOUT ANY WARRANTY; without even the implied warranty of
      MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
      Affero General Public License for more details.
-    
+
      You should have received a copy of the GNU Affero General Public License
      along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
@@ -245,17 +245,14 @@ GCP_2s (const struct CadetPeer *cp)
   static char buf[5];
   char *ret;
 
-  if ((NULL == cp) || 
-      (NULL == &cp->pid.public_key))
+  if ((NULL == cp) ||
+      (0 == GNUNET_is_zero (&cp->pid.public_key)))
     return "NULL";
-  
-      
+
   ret = GNUNET_CRYPTO_eddsa_public_key_to_string (&cp->pid.public_key);
-  
   if (NULL == ret)
     return "NULL";
-  
-  
+
   GNUNET_strlcpy (buf,
                   ret,
                   sizeof (buf));
@@ -1218,7 +1215,7 @@ GCP_iterate_paths (struct CadetPeer *cp,
        (NULL == cp->core_mq) ? "" : " including direct link");
   if (NULL != cp->core_mq)
   {
-    /* FIXME: this branch seems to duplicate the 
+    /* FIXME: this branch seems to duplicate the
        i=0 case below (direct link). Leave out!??? -CG */
     struct CadetPeerPath *path;