raster pushed a commit to branch master.

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

commit 769066f606e6459192d0b6ed5069e0807cc7ae81
Author: Vincent Torri <vincent.to...@gmail.com>
Date:   Wed Mar 31 10:29:50 2021 +0100

    Evas avif loader: fix segfault when the avif loader tries to load a non 
avif image
    
    Test Plan: entice is not crashing
    
    Reviewers: raster
    
    Subscribers: cedric, #reviewers, #committers
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D12254
---
 src/modules/evas/image_loaders/avif/evas_image_load_avif.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/modules/evas/image_loaders/avif/evas_image_load_avif.c 
b/src/modules/evas/image_loaders/avif/evas_image_load_avif.c
index dd564dbc8a..e641473c89 100644
--- a/src/modules/evas/image_loaders/avif/evas_image_load_avif.c
+++ b/src/modules/evas/image_loaders/avif/evas_image_load_avif.c
@@ -250,7 +250,12 @@ evas_image_load_file_close_avif(void *loader_data)
    Evas_Loader_Internal *loader;
 
    loader = loader_data;
-   avifDecoderDestroy(loader->decoder);
+   /*
+    * in case _head() fails (because the file is not an avif one),
+    * loader is not filled and loader->decoder is NULL
+    */
+   if (loader->decoder)
+     avifDecoderDestroy(loader->decoder);
    free(loader_data);
 }
 

-- 


Reply via email to