emo/shadow
1
0
Fork 0
Code Issues Pull Requests Packages Releases Activity

Do not crash in commonio_close if database FILE not opened.

Browse Source 
The db->fp can be NULL if commonio_unlock() is called when the
shadow file is opened but did not exist before.
This commit is contained in:
Tomas Mraz 2019-03-25 14:51:26 +01:00
parent d2e1e9ce85
commit d6276066dd
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/commonio.c
View File

@ -925,7 +925,6 @@ static int write_all (const struct commonio_db *db)
int commonio_close (struct commonio_db *db)
/*@requires notnull db->fp@*/
{
char buf[1024];
int errors = 0;
@ -938,8 +937,10 @@ int commonio_close (struct commonio_db *db)
db->isopen = false;
if (!db->changed || db->readonly) {
(void) fclose (db->fp);
db->fp = NULL;
if (NULL != db->fp) {
(void) fclose (db->fp);
db->fp = NULL;
}
goto success;
}