From d2b85dd6af06c8350f6d8ad471f368ebf2182e60 Mon Sep 17 00:00:00 2001 From: Yuxuan Shui Date: Sat, 8 Sep 2018 15:12:33 +0100 Subject: [PATCH] Fix some null pointer problems --- README.md | 2 ++ src/win.c | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index e5130c9..229e1cd 100644 --- a/README.md +++ b/README.md @@ -1,6 +1,8 @@ Compton ======= +**This is a development branch, bug to be expected** + This is forked from the original Compton because that seems to have become unmaintained. I'll merge pull requests as they appear upstream, as well as trying to fix bugs reported to upstream, or found by myself. New features are not likely to be added, since I expect compton to become irrelevant in near future. diff --git a/src/win.c b/src/win.c index 3d74a50..a92cebd 100644 --- a/src/win.c +++ b/src/win.c @@ -185,7 +185,7 @@ int win_get_name(session_t *ps, win *w) { } int ret = 0; - if (strcmp(w->name, strlst[0]) != 0) { + if (!w->name || strcmp(w->name, strlst[0]) != 0) { ret = 1; free(w->name); w->name = mstrcpy(strlst[0]); @@ -208,7 +208,7 @@ int win_get_role(session_t *ps, win *w) { return -1; int ret = 0; - if (strcmp(w->role, strlst[0]) != 0) { + if (!w->role || strcmp(w->role, strlst[0]) != 0) { ret = 1; free(w->role); w->role = mstrcpy(strlst[0]);