diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2021-06-24 17:39:57 +0200 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2021-06-24 17:39:57 +0200 |
| commit | 4a68697521392f461c8627841531548d4bd6841a (patch) | |
| tree | 5dbdd042a179fbd5a87e52c959db174c3f28fc8c /libbb/remove_file.c | |
| parent | 6481070eb949125b750aae6e0ec08368a15ac371 (diff) | |
| download | busybox-4a68697521392f461c8627841531548d4bd6841a.tar.gz | |
libbb: do not check closedir() in remove_file()
If it's needed, there has to be a comment why. There isn't.
function old new delta
.rodata 103686 103669 -17
remove_file 598 571 -27
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-44) Total: -44 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'libbb/remove_file.c')
| -rw-r--r-- | libbb/remove_file.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/libbb/remove_file.c b/libbb/remove_file.c index cea5d47e6..1505e6218 100644 --- a/libbb/remove_file.c +++ b/libbb/remove_file.c @@ -60,11 +60,7 @@ int FAST_FUNC remove_file(const char *path, int flags) status = -1; free(new_path); } - - if (closedir(dp) < 0) { - bb_perror_msg("can't close '%s'", path); - return -1; - } + closedir(dp); if (flags & FILEUTILS_INTERACTIVE) { fprintf(stderr, "%s: remove directory '%s'? ", |