diff options
| author |  John Keeping <john@keeping.me.uk> | 2013-03-23 17:16:46 +0000 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2013-03-23 23:50:50 -0700 |
| commit | e4ca819abff48af8e4a10059c88b3b1533b9f994 (patch) | |
| tree | 8eac07ff308aa6c1bf882c9fa010084ab76ae0ab /builtin/shortlog.c | |
| parent | reflog: add for_each_reflog_ent_reverse() API (diff) | |
| download | tgif-e4ca819abff48af8e4a10059c88b3b1533b9f994.tar.xz | |
refs.c: fix fread error handling
fread returns the number of items read, with no special error return.
Commit 98f85ff (reflog: add for_each_reflog_ent_reverse() API -
2013-03-08) introduced a call to fread which checks for an error with
"nread < 0" which is tautological since nread is unsigned. The correct
check in this case (which tries to read a single item) is "nread != 1".
Signed-off-by: John Keeping <john@keeping.me.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/shortlog.c')
0 files changed, 0 insertions, 0 deletions