On Sun, Apr 16, 2017 at 07:04:01PM +0000, Sebastian Schuberth wrote:
> Signed-off-by: Sebastian Schuberth <sschube...@gmail.com>
> ---
> sha1_file.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/sha1_file.c b/sha1_file.c
> index 7106389..9ecf71f 100644
> --- a/sha1_file.c
> +++ b/sha1_file.c
> @@ -3970,7 +3970,6 @@ int read_loose_object(const char *path,
> void **contents)
> {
> int ret = -1;
> - int fd = -1;
> void *map = NULL;
> unsigned long mapsize;
> git_zstream stream;
> @@ -4020,7 +4019,5 @@ int read_loose_object(const char *path,
> out:
> if (map)
> munmap(map, mapsize);
> - if (fd >= 0)
> - close(fd);
> return ret;
Thanks, this looks obviously correct. The mistake is mine; in an early
iteration of the series, the caller of map_sha1_file_1() was responsible
for opening the file itself, but I later switched it to take the path.
-Peff
