aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLibravatar Thomas Jarosch <thomas.jarosch@intra2net.com>2017-05-23 21:43:42 +0200
committerLibravatar Thomas Jarosch <thomas.jarosch@intra2net.com>2017-05-23 21:44:36 +0200
commit4ec84fa66436a6e64bf54edd98f80398e40765cb (patch)
treeb6a25cba5b3a2c12a9622725de20dc4955ec84cb
parentfix for #1301 (diff)
downloadfirejail-4ec84fa66436a6e64bf54edd98f80398e40765cb.tar.gz
firejail-4ec84fa66436a6e64bf54edd98f80398e40765cb.tar.zst
firejail-4ec84fa66436a6e64bf54edd98f80398e40765cb.zip
minor bugfix: Correctly dereference "subdirname" variable
cppcheck reported: [firejail/main.c:1335]: (warning) Char literal compared with pointer 'subdirname'. Did you intend to dereference it?
-rw-r--r--src/firejail/main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/firejail/main.c b/src/firejail/main.c
index 012b2f230..95c325f9f 100644
--- a/src/firejail/main.c
+++ b/src/firejail/main.c
@@ -1332,7 +1332,7 @@ int main(int argc, char **argv) {
1332 arg_overlay_reuse = 1; 1332 arg_overlay_reuse = 1;
1333 1333
1334 char *subdirname = argv[i] + 16; 1334 char *subdirname = argv[i] + 16;
1335 if (subdirname == '\0') { 1335 if (*subdirname == '\0') {
1336 fprintf(stderr, "Error: invalid overlay option\n"); 1336 fprintf(stderr, "Error: invalid overlay option\n");
1337 exit(1); 1337 exit(1);
1338 } 1338 }