VFK: avoid potential nullptr dereference

rouault · rouault · commit d56524ede27c · 2025-08-30T15:05:07.000+02:00
should likely fix https://issues.oss-fuzz.com/issues/441564453
diff --git a/ogr/ogrsf_frmts/vfk/vfkreaderp.h b/ogr/ogrsf_frmts/vfk/vfkreaderp.h
@@ -80,7 +80,7 @@ class VFKReader : public IVFKReader
 
     bool IsValid() const override
     {
-        return true;
+        return m_poFD != nullptr;
     }
 
     bool HasFileField() const override
@@ -143,7 +143,7 @@ class VFKReaderSQLite : public VFKReader
 
     bool IsValid() const override
     {
-        return m_poDB != nullptr;
+        return VFKReader::IsValid() && m_poDB != nullptr;
     }
 
     int ReadDataBlocks(bool = false) override;

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ class VFKReader : public IVFKReader`
`80`	`80`
`81`	`81`	`bool IsValid() const override`
`82`	`82`	`{`
`83`		`- return true;`
	`83`	`+ return m_poFD != nullptr;`
`84`	`84`	`}`
`85`	`85`
`86`	`86`	`bool HasFileField() const override`
`@@ -143,7 +143,7 @@ class VFKReaderSQLite : public VFKReader`
`143`	`143`
`144`	`144`	`bool IsValid() const override`
`145`	`145`	`{`
`146`		`- return m_poDB != nullptr;`
	`146`	`+ return VFKReader::IsValid() && m_poDB != nullptr;`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`int ReadDataBlocks(bool = false) override;`