diff options
| author | Dave Reisner <dreisner@archlinux.org> | 2013-07-04 02:33:19 +0200 |
|---|---|---|
| committer | Allan McRae <allan@archlinux.org> | 2013-07-05 06:32:23 +0200 |
| commit | eb19d41d5f85f169cee7570f783821cb705ad37a (patch) | |
| tree | 245a12aa27e01a6f8ee74f5c1624d4e341551b81 /lib/libalpm/be_sync.c | |
| parent | 7b8f8753b15037bf4a88126ffde8195416432c72 (diff) | |
| download | pacman-eb19d41d5f85f169cee7570f783821cb705ad37a.tar.gz pacman-eb19d41d5f85f169cee7570f783821cb705ad37a.tar.xz | |
do not check error from close(2)
On operating systems we support, the behavior is always such that the
kernel will do the right thing as far as invalidating the file
descriptor, regardless of the eventual return value. Therefore,
potentially looping and calling close multiple times is wrong.
At best, we call close again on an invalid FD and throw a spurious EBADF
error. At worst, we might close an FD which doesn't belong to us when a
multi-threaded application opens its own file descriptor between
iterations of the loop.
Signed-off-by: Dave Reisner <dreisner@archlinux.org>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/be_sync.c')
| -rw-r--r-- | lib/libalpm/be_sync.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index f9fd5d1f..feda6f5d 100644 --- a/lib/libalpm/be_sync.c +++ b/lib/libalpm/be_sync.c @@ -474,7 +474,7 @@ static int sync_db_populate(alpm_db_t *db) cleanup: _alpm_archive_read_free(archive); if(fd >= 0) { - CLOSE(fd); + close(fd); } return count; } |