[ISC-support #10104] possible use of uninitialized value in zone_touched()
Migrated from BUGS RT #42440
From code inspection, it looks like bind9/lib/dns/zone.c:zone_touched() could use an uninitialized value of 'modtime' here:
isc_time_t modtime;
[...]
result = isc_file_getmodtime(zone->masterfile, &modtime);
if (result != ISC_R_SUCCESS ||
isc_time_compare(&modtime, &zone->loadtime) > 0)
{
zone->loadtime = modtime;
return (ISC_TRUE);
}if isc_file_getmodtime() fails. I don't know the intent of the implementation in this case, but depending on the intent this would probably either have to be intact or set to 'now' instead. (I don't have a particular opinion. I'd leave how to fix it to you).