commit 76764ddcf53fa4840993c395fdf383a47ad61e8e
author Donald Sharp <sharpd@cumulusnetworks.com> Fri Nov 20 09:05:46 2015 -0500
committer Donald Sharp <sharpd@cumulusnetworks.com> Tue Dec 08 14:16:19 2015 -0500
tree f2c0a5b808f404071ce5c023eb26650671911de9
parent 642577340cfb7ad66d021ab2b86d72f7cdde03bc

isisd, lib: Fix some more compiler warnings

A couple compiler warnings snuck in from the last
round of work being looked at.  This cleans them up

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

isisd/isis_bpf.c

diff --git a/isisd/isis_bpf.c b/isisd/isis_bpf.c
index fd65608..889b4c3 100644
--- a/isisd/isis_bpf.c
+++ b/isisd/isis_bpf.c
@@ -348,6 +348,8 @@
         return ISIS_WARNING;
       return ISIS_ERROR;
     }
+
+  return ISIS_OK;
 }
 
 int

lib/log.c

diff --git a/lib/log.c b/lib/log.c
index e376205..0914bf8 100644
--- a/lib/log.c
+++ b/lib/log.c
@@ -1036,7 +1036,7 @@
               if (j >= len) /* end of block, not really printing */
                 s += sprintf(s, " ");
 
-              else if(isprint(((char*)mem)[j])) /* printable char */
+              else if(isprint((int)((char*)mem)[j])) /* printable char */
                 s += sprintf(s, "%c", 0xFF & ((char*)mem)[j]);
 
               else /* other char */