diff options
author | gad <gad@FreeBSD.org> | 2000-12-02 00:07:56 +0000 |
---|---|---|
committer | gad <gad@FreeBSD.org> | 2000-12-02 00:07:56 +0000 |
commit | 2b0e48c014b3752cb18b4108f2857f52c32094a7 (patch) | |
tree | 23ccd94621527744b665cb9474ed4efeb7d5e79c | |
parent | 6a64e36210e572b4242fa2a9b9db3599d7e8e30d (diff) | |
download | FreeBSD-src-2b0e48c014b3752cb18b4108f2857f52c32094a7.zip FreeBSD-src-2b0e48c014b3752cb18b4108f2857f52c32094a7.tar.gz |
Fix some error-handling logic so that ferror is called before fclose,
instead of immediately after the fclose. The previous logic did work
on freebsd, but is somewhat risky practice (and causes trouble when
porting to other OS's).
PR: bin/22965
Reviewed by: Garrett Wollman
-rw-r--r-- | lib/libc/gen/getcap.c | 28 |
1 files changed, 21 insertions, 7 deletions
diff --git a/lib/libc/gen/getcap.c b/lib/libc/gen/getcap.c index b3261ae..edb828b 100644 --- a/lib/libc/gen/getcap.c +++ b/lib/libc/gen/getcap.c @@ -647,7 +647,7 @@ cgetnext(bp, db_array) char **db_array; { size_t len; - int status, i, done; + int done, hadreaderr, i, savederrno, status; char *cp, *line, *rp, *np, buf[BSIZE], nbuf[BSIZE]; u_int dummy; @@ -665,9 +665,14 @@ cgetnext(bp, db_array) } else { line = fgetln(pfp, &len); if (line == NULL && pfp) { - (void)fclose(pfp); - if (ferror(pfp)) { - (void)cgetclose(); + hadreaderr = ferror(pfp); + if (hadreaderr) + savederrno = errno; + fclose(pfp); + pfp = NULL; + if (hadreaderr) { + cgetclose(); + errno = savederrno; return (-1); } else { if (*++dbp == NULL) { @@ -724,9 +729,18 @@ cgetnext(bp, db_array) } else { /* name field extends beyond the line */ line = fgetln(pfp, &len); if (line == NULL && pfp) { - (void)fclose(pfp); - if (ferror(pfp)) { - (void)cgetclose(); + /* Name extends beyond the EOF! */ + hadreaderr = ferror(pfp); + if (hadreaderr) + savederrno = errno; + fclose(pfp); + pfp = NULL; + if (hadreaderr) { + cgetclose(); + errno = savederrno; + return (-1); + } else { + cgetclose(); return (-1); } } else |