diff options
| author |  Brian Ashworth <bosrsf04@gmail.com> | 2019-02-05 01:59:40 -0500 |
|---|---|---|
| committer |  Drew DeVault <sir@cmpwn.com> | 2019-02-11 10:15:00 -0500 |
| commit | df3ea6a55f4a163ce5d1f241060a1308198ff27a (patch) | |
| tree | 2d2e2affc4c11bd8a95c511345d065ccfa9e7592 | |
| parent | seat_cmd_cursor: do not create non-existing seat (diff) | |
| download | sway-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.
| -rw-r--r-- | sway/config.c | 2 |
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; |