diff options
| author | Richard W.M. Jones <rjones@redhat.com> | 2012-01-19 11:30:23 +0000 |
|---|---|---|
| committer | Richard W.M. Jones <rjones@redhat.com> | 2012-01-19 11:31:23 +0000 |
| commit | 61abcf71c05c07bfaca96666b16a0bc30a6edab4 (patch) | |
| tree | 877f4f9c475248b7425a2f890e40a10e5e66fdd1 /daemon/file.c | |
| parent | 08840bab44c38d0c69b5780c57b2bf370c96d58c (diff) | |
| download | libguestfs-61abcf71c05c07bfaca96666b16a0bc30a6edab4.tar.gz libguestfs-61abcf71c05c07bfaca96666b16a0bc30a6edab4.tar.xz libguestfs-61abcf71c05c07bfaca96666b16a0bc30a6edab4.zip | |
daemon: pwrite/pread: Don't double close on error path.
In Linux, close (fd) closes the file descriptor even if it returns an
error.
Diffstat (limited to 'daemon/file.c')
| -rw-r--r-- | daemon/file.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/daemon/file.c b/daemon/file.c index e0f87948..91746e0d 100644 --- a/daemon/file.c +++ b/daemon/file.c @@ -483,7 +483,6 @@ pread_fd (int fd, int count, int64_t offset, size_t *size_r, if (close (fd) == -1) { reply_with_perror ("close: %s", display_path); - close (fd); free (buf); return NULL; } @@ -539,7 +538,6 @@ pwrite_fd (int fd, const char *content, size_t size, int64_t offset, if (close (fd) == -1) { reply_with_perror ("close: %s", display_path); - close (fd); return -1; } |