summaryrefslogtreecommitdiffstats
path: root/sway/config.c
diff options
context:
space:
mode:
authorLibravatar Brian Ashworth <bosrsf04@gmail.com>2019-02-05 01:59:40 -0500
committerLibravatar Drew DeVault <sir@cmpwn.com>2019-02-11 10:15:00 -0500
commitdf3ea6a55f4a163ce5d1f241060a1308198ff27a (patch)
tree2d2e2affc4c11bd8a95c511345d065ccfa9e7592 /sway/config.c
parentseat_cmd_cursor: do not create non-existing seat (diff)
downloadsway-df3ea6a55f4a163ce5d1f241060a1308198ff27a.tar.gz
sway-df3ea6a55f4a163ce5d1f241060a1308198ff27a.tar.zst
sway-df3ea6a55f4a163ce5d1f241060a1308198ff27a.zip
load_include_configs: fix wordexp fail condition
This fixes the failure condition for the wordexp call in load_include_configs. The only success value is zero. Since the error codes are positive, having the check be less than zero was causing segfaults on failure when accessing the words.
Diffstat (limited to 'sway/config.c')
-rw-r--r--sway/config.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sway/config.c b/sway/config.c
index ee1c42df..0c23fad8 100644
--- a/sway/config.c
+++ b/sway/config.c
@@ -557,7 +557,7 @@ bool load_include_configs(const char *path, struct sway_config *config,
557 557
558 wordexp_t p; 558 wordexp_t p;
559 559
560 if (wordexp(path, &p, 0) < 0) { 560 if (wordexp(path, &p, 0) != 0) {
561 free(parent_path); 561 free(parent_path);
562 free(wd); 562 free(wd);
563 return false; 563 return false;