Commit 24b8ed2b by Patrick Steinhardt

attr_file: fix resource leak

parent e2625457
...@@ -123,7 +123,7 @@ int git_attr_file__load( ...@@ -123,7 +123,7 @@ int git_attr_file__load(
break; break;
} }
case GIT_ATTR_FILE__FROM_FILE: { case GIT_ATTR_FILE__FROM_FILE: {
int fd; int fd = -1;
/* For open or read errors, pretend that we got ENOTFOUND. */ /* For open or read errors, pretend that we got ENOTFOUND. */
/* TODO: issue warning when warning API is available */ /* TODO: issue warning when warning API is available */
...@@ -133,7 +133,8 @@ int git_attr_file__load( ...@@ -133,7 +133,8 @@ int git_attr_file__load(
(fd = git_futils_open_ro(entry->fullpath)) < 0 || (fd = git_futils_open_ro(entry->fullpath)) < 0 ||
(error = git_futils_readbuffer_fd(&content, fd, (size_t)st.st_size)) < 0) (error = git_futils_readbuffer_fd(&content, fd, (size_t)st.st_size)) < 0)
nonexistent = true; nonexistent = true;
else
if (fd >= 0)
p_close(fd); p_close(fd);
break; break;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment