diff options
author | Junio C Hamano <gitster@pobox.com> | 2017-11-09 14:31:26 +0900 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-11-09 14:31:26 +0900 |
commit | 4e9762ed4791a44f7b1c2ab66328b81bdc243173 (patch) | |
tree | 70cdfe250ef08f480f1dd266c9db952dc109fbc6 /diff.c | |
parent | Merge branch 'sb/blame-config-doc' (diff) | |
parent | diff: fix lstat() error handling in diff_populate_filespec() (diff) | |
download | tgif-4e9762ed4791a44f7b1c2ab66328b81bdc243173.tar.xz |
Merge branch 'ao/diff-populate-filespec-lstat-errorpath-fix'
After an error from lstat(), diff_populate_filespec() function
sometimes still went ahead and used invalid data in struct stat,
which has been fixed.
* ao/diff-populate-filespec-lstat-errorpath-fix:
diff: fix lstat() error handling in diff_populate_filespec()
Diffstat (limited to 'diff.c')
-rw-r--r-- | diff.c | 14 |
1 files changed, 6 insertions, 8 deletions
@@ -3545,14 +3545,12 @@ int diff_populate_filespec(struct diff_filespec *s, unsigned int flags) int fd; if (lstat(s->path, &st) < 0) { - if (errno == ENOENT) { - err_empty: - err = -1; - empty: - s->data = (char *)""; - s->size = 0; - return err; - } + err_empty: + err = -1; + empty: + s->data = (char *)""; + s->size = 0; + return err; } s->size = xsize_t(st.st_size); if (!s->size) |