[EGIT] [core/efl] efl-1.24 06/09: evas_vg_container: return NULL in evas_vg_container_add when the parent of container is NULL

[Taehyub Kim]

stefan pushed a commit to branch efl-1.24.

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

commit c3adf72aa8390bd497fa145215ebffcd44af3a41
Author: Taehyub Kim <taehyub....@samsung.com>
Date:   Fri May 22 12:23:46 2020 +0900

    evas_vg_container: return NULL in evas_vg_container_add when the parent of 
container is NULL
    
    Summary:
    evas_vg_container should return NULL when the parent is NULL
    because the function does not allot NULL parent
    @fix
    
    Reviewers: kimcinoo, jsuya, Hermet
    
    Reviewed By: kimcinoo
    
    Subscribers: cedric, #reviewers, #committers
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D11867
---
 src/lib/evas/canvas/efl_canvas_vg_container.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/lib/evas/canvas/efl_canvas_vg_container.c 
b/src/lib/evas/canvas/efl_canvas_vg_container.c
index 37b59405c3..61bd74e7c3 100644
--- a/src/lib/evas/canvas/efl_canvas_vg_container.c
+++ b/src/lib/evas/canvas/efl_canvas_vg_container.c
@@ -493,7 +493,11 @@ evas_vg_container_add(Evas_Object *parent)
 {
    /* Warn it because the usage has been changed.
       We can remove this message after v1.21. */
-   if (!parent) CRI("Efl_VG Container doesn't allow null parent!");
+   if (!parent)
+     {
+        ERR("Efl_VG Container doesn't allow null parent!");
+        return NULL;
+     }
    return efl_add(MY_CLASS, parent);
 }
 

--