diff options
author | Reiner Herrmann <reiner@reiner-h.de> | 2022-07-27 21:12:58 +0200 |
---|---|---|
committer | Reiner Herrmann <reiner@reiner-h.de> | 2022-07-30 10:48:33 +0200 |
commit | eb20f52ef570196e12e715e2a8c002feb103e448 (patch) | |
tree | dda8169cc4a48c40265973034e31bcf43f461c0f /src | |
parent | zero-initialize two variables (diff) | |
download | firejail-eb20f52ef570196e12e715e2a8c002feb103e448.tar.gz firejail-eb20f52ef570196e12e715e2a8c002feb103e448.tar.zst firejail-eb20f52ef570196e12e715e2a8c002feb103e448.zip |
Make list of paths const to fix a false positive of gcc analyzer
Diffstat (limited to 'src')
-rw-r--r-- | src/firejail/fs_bin.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/firejail/fs_bin.c b/src/firejail/fs_bin.c index 2b0b3003e..6228e9740 100644 --- a/src/firejail/fs_bin.c +++ b/src/firejail/fs_bin.c | |||
@@ -27,7 +27,7 @@ | |||
27 | 27 | ||
28 | static int prog_cnt = 0; | 28 | static int prog_cnt = 0; |
29 | 29 | ||
30 | static char *paths[] = { | 30 | static const char * const paths[] = { |
31 | "/usr/local/bin", | 31 | "/usr/local/bin", |
32 | "/usr/bin", | 32 | "/usr/bin", |
33 | "/bin", | 33 | "/bin", |
@@ -40,7 +40,7 @@ static char *paths[] = { | |||
40 | }; | 40 | }; |
41 | 41 | ||
42 | // return 1 if found, 0 if not found | 42 | // return 1 if found, 0 if not found |
43 | static char *check_dir_or_file(const char *name) { | 43 | static const char *check_dir_or_file(const char *name) { |
44 | EUID_ASSERT(); | 44 | EUID_ASSERT(); |
45 | assert(name); | 45 | assert(name); |
46 | struct stat s; | 46 | struct stat s; |
@@ -160,7 +160,7 @@ static void duplicate(char *fname) { | |||
160 | else { | 160 | else { |
161 | // Find the standard directory (by looping through paths[]) | 161 | // Find the standard directory (by looping through paths[]) |
162 | // where the filename fname is located | 162 | // where the filename fname is located |
163 | char *path = check_dir_or_file(fname); | 163 | const char *path = check_dir_or_file(fname); |
164 | if (!path) | 164 | if (!path) |
165 | return; | 165 | return; |
166 | if (asprintf(&full_path, "%s/%s", path, fname) == -1) | 166 | if (asprintf(&full_path, "%s/%s", path, fname) == -1) |