Andrew's git
/
gitweb.git
/ diff
summary
|
log
|
commit
| diff |
tree
commit
grep
author
committer
pickaxe
?
re
Merge branch 'rs/maint-diff-fd-leak' into maint
author
Junio C Hamano
<gitster@pobox.com>
Sun, 26 Dec 2010 19:18:39 +0000
(11:18 -0800)
committer
Junio C Hamano
<gitster@pobox.com>
Sun, 26 Dec 2010 19:18:39 +0000
(11:18 -0800)
* rs/maint-diff-fd-leak:
close file on error in read_mmfile()
xdiff-interface.c
patch
|
blob
|
history
raw
(from parent 1:
c2ed29b
)
diff --git
a/xdiff-interface.c
b/xdiff-interface.c
index e1e054e4d982de30d8a9c8c4109c6d62448f62a9..164581f87f49935f0d1b1885420960a4d11dea56 100644
(file)
--- a/
xdiff-interface.c
+++ b/
xdiff-interface.c
@@
-212,8
+212,10
@@
int read_mmfile(mmfile_t *ptr, const char *filename)
return error("Could not open %s", filename);
sz = xsize_t(st.st_size);
ptr->ptr = xmalloc(sz ? sz : 1);
- if (sz && fread(ptr->ptr, sz, 1, f) != 1)
+ if (sz && fread(ptr->ptr, sz, 1, f) != 1) {
+ fclose(f);
return error("Could not read %s", filename);
+ }
fclose(f);
ptr->size = sz;
return 0;