diff options
author | smitsohu <smitsohu@gmail.com> | 2018-10-01 13:09:03 +0200 |
---|---|---|
committer | smitsohu <smitsohu@gmail.com> | 2018-10-01 13:09:03 +0200 |
commit | 244aac803179c63fd9d8e98fd8aa9dcdabc67406 (patch) | |
tree | d8032b33b8ea03096e9a2f17a10aba672162a995 | |
parent | tiny memleaks (diff) | |
download | firejail-244aac803179c63fd9d8e98fd8aa9dcdabc67406.tar.gz firejail-244aac803179c63fd9d8e98fd8aa9dcdabc67406.tar.zst firejail-244aac803179c63fd9d8e98fd8aa9dcdabc67406.zip |
fs_whitelist: reduce number of loop iterations
-rw-r--r-- | src/firejail/fs_whitelist.c | 5 |
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; |