Correct minor issues reported by the analyzer.

Found with the Clang Static Analyzer.

diff --git a/i3bar/src/ipc.c b/i3bar/src/ipc.c
index 206b1b72c0bad0e2e0bfdf6ecf24fe9d4ff6b048..dc2255eadd8d68499958e2a2b6d1e7cbe7a0386c 100644 (file)
--- a/i3bar/src/ipc.c
+++ b/i3bar/src/ipc.c
@@ -248,7 +248,8 @@ int i3_send_msg(uint32_t type, const char *payload) {
     memcpy(walk, &type, sizeof(uint32_t));
     walk += sizeof(uint32_t);
 
-    strncpy(walk, payload, len);
+    if (payload != NULL)
+        strncpy(walk, payload, len);
 
     uint32_t written = 0;
 

diff --git a/i3bar/src/xcb.c b/i3bar/src/xcb.c
index f33a69cd9910c57305bcaba479a8ab5bc6aa3656..8d4f60b5d4ee14d81bdd9708f85af609c9806a28 100644 (file)
--- a/i3bar/src/xcb.c
+++ b/i3bar/src/xcb.c
@@ -51,7 +51,7 @@ xcb_charinfo_t         *font_table;
 /* These are only relevant for XKB, which we only need for grabbing modifiers */
 Display          *xkb_dpy;
 int              xkb_event_base;
-int              mod_pressed;
+int              mod_pressed = 0;
 
 /* Because the statusline is the same on all outputs, we have
  * global buffer to render it on */
@@ -409,7 +409,7 @@ void xcb_io_cb(struct ev_loop *loop, ev_io *watcher, int revents) {
  */
 void xkb_io_cb(struct ev_loop *loop, ev_io *watcher, int revents) {
     XkbEvent ev;
-    int modstate;
+    int modstate = 0;
 
     DLOG("Got XKB-Event!\n");