aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLibravatar smitsohu <smitsohu@gmail.com>2018-10-01 13:09:03 +0200
committerLibravatar smitsohu <smitsohu@gmail.com>2018-10-01 13:09:03 +0200
commit244aac803179c63fd9d8e98fd8aa9dcdabc67406 (patch)
treed8032b33b8ea03096e9a2f17a10aba672162a995
parenttiny memleaks (diff)
downloadfirejail-244aac803179c63fd9d8e98fd8aa9dcdabc67406.tar.gz
firejail-244aac803179c63fd9d8e98fd8aa9dcdabc67406.tar.zst
firejail-244aac803179c63fd9d8e98fd8aa9dcdabc67406.zip
fs_whitelist: reduce number of loop iterations
-rw-r--r--src/firejail/fs_whitelist.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/src/firejail/fs_whitelist.c b/src/firejail/fs_whitelist.c
index c4ab837c7..86a901506 100644
--- a/src/firejail/fs_whitelist.c
+++ b/src/firejail/fs_whitelist.c
@@ -380,6 +380,7 @@ void fs_whitelist(void) {
380 fprintf(stderr, "***\n"); 380 fprintf(stderr, "***\n");
381 } 381 }
382 entry->data = EMPTY_STRING; 382 entry->data = EMPTY_STRING;
383 entry = entry->next;
383 free(new_name); 384 free(new_name);
384 continue; 385 continue;
385 } 386 }
@@ -452,6 +453,7 @@ void fs_whitelist(void) {
452 } 453 }
453 454
454 entry->data = EMPTY_STRING; 455 entry->data = EMPTY_STRING;
456 entry = entry->next;
455 free(new_name); 457 free(new_name);
456 continue; 458 continue;
457 } 459 }
@@ -471,6 +473,7 @@ void fs_whitelist(void) {
471 } 473 }
472 nowhitelist[nowhitelist_c++] = fname; 474 nowhitelist[nowhitelist_c++] = fname;
473 entry->data = EMPTY_STRING; 475 entry->data = EMPTY_STRING;
476 entry = entry->next;
474 free(new_name); 477 free(new_name);
475 continue; 478 continue;
476 } 479 }
@@ -483,6 +486,7 @@ void fs_whitelist(void) {
483 printf("\"%s\" disabled by --private\n", entry->data); 486 printf("\"%s\" disabled by --private\n", entry->data);
484 487
485 entry->data = EMPTY_STRING; 488 entry->data = EMPTY_STRING;
489 entry = entry->next;
486 free(fname); 490 free(fname);
487 free(new_name); 491 free(new_name);
488 continue; 492 continue;
@@ -642,6 +646,7 @@ void fs_whitelist(void) {
642 if (arg_debug || arg_debug_whitelists) 646 if (arg_debug || arg_debug_whitelists)
643 printf("Skip nowhitelisted path %s\n", fname); 647 printf("Skip nowhitelisted path %s\n", fname);
644 entry->data = EMPTY_STRING; 648 entry->data = EMPTY_STRING;
649 entry = entry->next;
645 free(fname); 650 free(fname);
646 free(new_name); 651 free(new_name);
647 continue; 652 continue;