diff options
author | netblue30 <netblue30@yahoo.com> | 2020-09-28 08:25:56 -0400 |
---|---|---|
committer | netblue30 <netblue30@yahoo.com> | 2020-09-28 08:25:56 -0400 |
commit | c6166d79c25cfc22be340c272447cf60a2fa932a (patch) | |
tree | ca0a84181a4a9f925309c58225b4a9d4b1457d00 /src/firejail/ls.c | |
parent | compile support for GCC static analyzer (--enable-analyzer) (diff) | |
download | firejail-c6166d79c25cfc22be340c272447cf60a2fa932a.tar.gz firejail-c6166d79c25cfc22be340c272447cf60a2fa932a.tar.zst firejail-c6166d79c25cfc22be340c272447cf60a2fa932a.zip |
free some memory; get rid of false positive from gcc static analyzer
Diffstat (limited to 'src/firejail/ls.c')
-rw-r--r-- | src/firejail/ls.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/firejail/ls.c b/src/firejail/ls.c index 4d0a001b6..ed50934a8 100644 --- a/src/firejail/ls.c +++ b/src/firejail/ls.c | |||
@@ -48,6 +48,7 @@ static void print_file_or_dir(const char *path, const char *fname) { | |||
48 | return; | 48 | return; |
49 | } | 49 | } |
50 | } | 50 | } |
51 | free(name); | ||
51 | 52 | ||
52 | // permissions | 53 | // permissions |
53 | if (S_ISLNK(s.st_mode)) | 54 | if (S_ISLNK(s.st_mode)) |
@@ -171,10 +172,11 @@ static void print_directory(const char *path) { | |||
171 | if (n < 0) | 172 | if (n < 0) |
172 | errExit("scandir"); | 173 | errExit("scandir"); |
173 | else { | 174 | else { |
174 | for (i = 0; i < n; i++) { | 175 | for (i = 0; i < n; i++) |
175 | print_file_or_dir(path, namelist[i]->d_name); | 176 | print_file_or_dir(path, namelist[i]->d_name); |
177 | // get rid of false psitive reported by GCC -fanalyze | ||
178 | for (i = 0; i < n; i++) | ||
176 | free(namelist[i]); | 179 | free(namelist[i]); |
177 | } | ||
178 | } | 180 | } |
179 | free(namelist); | 181 | free(namelist); |
180 | } | 182 | } |