* lib/commonio.c: Avoid implicit conversion of pointers to

booleans.
	* lib/commonio.c: Ignore the return value of close when a
	failure is reported.
This commit is contained in:
nekral-guest 2010-03-17 23:59:47 +00:00
parent a013f8519b
commit fe71812b1d
2 changed files with 10 additions and 3 deletions

View File

@ -1,3 +1,10 @@
2010-03-17 Nicolas François <nicolas.francois@centraliens.net>
* lib/commonio.c: Avoid implicit conversion of pointers to
booleans.
* lib/commonio.c: Ignore the return value of close when a
failure is reported.
2010-03-16 Nicolas François <nicolas.francois@centraliens.net>
* man/faillog.8.xml: Document the behavior in display mode of the
@ -878,7 +885,7 @@
2009-04-30 Nicolas François <nicolas.francois@centraliens.net>
* lib/commonio.c: Ignore teh return values of fclose() and
* lib/commonio.c: Ignore the return values of fclose() and
unlink() in case of failure of fopen_set_perms() or
create_backup().
* lib/commonio.c: Should the backup file be unlink'ed in case of

View File

@ -572,8 +572,8 @@ int commonio_open (struct commonio_db *db, int mode)
#endif /* WITH_TCB */
db->fp = fdopen (fd, db->readonly ? "r" : "r+");
saved_errno = errno;
if (!db->fp) {
close (fd);
if (NULL == db->fp) {
(void) close (fd);
}
}
errno = saved_errno;