diff options
| author |  Reiner Herrmann <reiner@reiner-h.de> | 2017-08-29 18:56:40 +0200 |
|---|---|---|
| committer | Reiner Herrmann <reiner@reiner-h.de> | 2017-08-29 18:56:40 +0200 |
| commit | 4af5bc09308e3fdd67c1f178082bc5de32d9884c (patch) | |
| tree | 86dc5651d8df3aa4fffeb2c97ae41d2bca051995 /src | |
| parent | Merge pull request #1500 from pizzadude/patch-3 (diff) | |
| download | firejail-4af5bc09308e3fdd67c1f178082bc5de32d9884c.tar.gz firejail-4af5bc09308e3fdd67c1f178082bc5de32d9884c.tar.zst firejail-4af5bc09308e3fdd67c1f178082bc5de32d9884c.zip | |
Fix pointer dereference
Found by compiler:
fs_bin.c:108:14: warning: comparison between pointer and zero character constant [-Wpointer-compare]
if (++fname == '\0')
^~
Diffstat (limited to 'src')
| -rw-r--r-- | src/firejail/fs_bin.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/firejail/fs_bin.c b/src/firejail/fs_bin.c index 110f027f7..d2c8fbbc8 100644 --- a/src/firejail/fs_bin.c +++ b/src/firejail/fs_bin.c | |||
| @@ -105,7 +105,7 @@ static int valid_full_path_file(const char *name) { | |||
| 105 | char *fname = strrchr(full_name, '/'); | 105 | char *fname = strrchr(full_name, '/'); |
| 106 | if (!fname) | 106 | if (!fname) |
| 107 | goto errexit; | 107 | goto errexit; |
| 108 | if (++fname == '\0') | 108 | if (*(++fname) == '\0') |
| 109 | goto errexit; | 109 | goto errexit; |
| 110 | 110 | ||
| 111 | int i = 0; | 111 | int i = 0; |