r.out.png: reset file pointer to NULL after fclose (#4220)

This patch continues the work from 917ba58. It's a good practice to immediately reset the file pointer once we do fclose on it, as it prevents using/closing descriptor allocated to another file in the future execution paths. Signed-off-by: Mohan Yelugoti <[email protected]>
OSGeo · Sep 7, 2024 · 98e3fe1 · 98e3fe1
1 parent 83ee146
commit 98e3fe1
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/raster/r.out.png/main.c b/raster/r.out.png/main.c
@@ -369,8 +369,10 @@ int main(int argc, char *argv[])
     /* G_free (info_ptr); */
     png_destroy_write_struct(&png_ptr, &info_ptr); /* al 11/2000 */
 
-    if (fp)
+    if (fp) {
         fclose(fp);
+        fp = NULL;
+    }
 
     if (wld_flag->answer) {
         if (do_stdout)