mir/bind
2
0
Fork 0
mirror of https://gitlab.isc.org/isc-projects/bind9 synced 2025-08-30 22:15:20 +00:00
Code Issues Releases Wiki Activity

Silence CID 329157: Dereference before null check in lib/dns/journal.c

Browse Source 
*** CID 329157:  Null pointer dereferences  (REVERSE_INULL)
    /lib/dns/journal.c: 754 in journal_open()
    748     			    j->header.index_size * sizeof(journal_rawpos_t));
    749     	}
    750     	if (j->index != NULL) {
    751     		isc_mem_put(j->mctx, j->index,
    752     			    j->header.index_size * sizeof(journal_pos_t));
    753     	}
       CID 329157:  Null pointer dereferences  (REVERSE_INULL)
       Null-checking "j->filename" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
    754     	if (j->filename != NULL) {
    755     		isc_mem_free(j->mctx, j->filename);
    756     	}
    757     	if (j->fp != NULL) {
    758     		(void)isc_stdio_close(j->fp);
    759     	}
This commit is contained in:
Mark Andrews
2021-03-08 07:06:57 +11:00
parent 27117f7b46
commit 4054405909
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/dns/journal.c
View File

@@ -751,9 +751,7 @@ failure:
isc_mem_put(j->mctx, j->index,
j->header.index_size * sizeof(journal_pos_t));
}
if (j->filename != NULL) {
isc_mem_free(j->mctx, j->filename);
}
isc_mem_free(j->mctx, j->filename);
if (j->fp != NULL) {
(void)isc_stdio_close(j->fp);
}