[EGIT] [core/efl] master 01/02: gif loader - dup eina file as we keep our own copy of it in the loader

[Carsten Haitzler]

raster pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=c56791180d4860d710dfe89d2c568ab65bd3fef2

commit c56791180d4860d710dfe89d2c568ab65bd3fef2
Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
Date:   Mon Aug 5 18:47:20 2019 +0100

    gif loader - dup eina file as we keep our own copy of it in the loader
    
    we didnt dup the eina file handle since  we kee our own handle. we
    need ot do that. asan found this.
---
 src/modules/evas/image_loaders/gif/evas_image_load_gif.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/modules/evas/image_loaders/gif/evas_image_load_gif.c 
b/src/modules/evas/image_loaders/gif/evas_image_load_gif.c
index 308f909c86..02412a280d 100644
--- a/src/modules/evas/image_loaders/gif/evas_image_load_gif.c
+++ b/src/modules/evas/image_loaders/gif/evas_image_load_gif.c
@@ -881,7 +881,7 @@ evas_image_load_file_open_gif2(Eina_File *f,
         *error = EVAS_LOAD_ERROR_RESOURCE_ALLOCATION_FAILED;
         return NULL;
      }
-   loader->f = f;
+   loader->f = eina_file_dup(f);
    loader->opts = opts;
    loader->animated = animated;
    return loader;
@@ -899,6 +899,7 @@ evas_image_load_file_close_gif2(void *loader_data)
 #endif
    if ((loader->fi.map) && (loader->f))
      eina_file_map_free(loader->f, loader->fi.map);
+   if (loader->f) eina_file_close(loader->f);
    free(loader);
 }
 

--