diff options
author | Chris Michael <cp.michael@samsung.com> | 2017-11-06 10:13:08 -0500 |
---|---|---|
committer | Chris Michael <cp.michael@samsung.com> | 2017-11-06 10:13:08 -0500 |
commit | 6c76fdb8245fb6dc4226cb7b0707aa93f1d9422a (patch) | |
tree | 8960f373523412d606c0fb2fc9e4b83b4f243d1e /src/lib/evas/canvas/evas_object_main.c | |
parent | 04d1fe7299f73376902ec3d28e5568ed52e9b374 (diff) |
evas-object-main: Fix null dereference
Coverity reports that EVAS_OBJECT_DATA_SAFE_GET returns NULL here (184
out of 188 times). The _all_animation_objects_cancel function
directly dereferences the obj parameter passed into it, so lets null check it
before calling that function.
Fixes Coverity CID1381711
@fix
Signed-off-by: Chris Michael <cp.michael@samsung.com>
Diffstat (limited to '')
-rw-r--r-- | src/lib/evas/canvas/evas_object_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/evas/canvas/evas_object_main.c b/src/lib/evas/canvas/evas_object_main.c index 4ed5888cd5..dce4954e08 100644 --- a/src/lib/evas/canvas/evas_object_main.c +++ b/src/lib/evas/canvas/evas_object_main.c | |||
@@ -2840,7 +2840,7 @@ _efl_canvas_object_event_animation_cancel(Eo *eo_obj) | |||
2840 | { | 2840 | { |
2841 | Evas_Object_Protected_Data *obj = EVAS_OBJECT_DATA_SAFE_GET(eo_obj); | 2841 | Evas_Object_Protected_Data *obj = EVAS_OBJECT_DATA_SAFE_GET(eo_obj); |
2842 | 2842 | ||
2843 | _all_animation_objects_cancel(obj); | 2843 | if (obj) _all_animation_objects_cancel(obj); |
2844 | } | 2844 | } |
2845 | 2845 | ||
2846 | 2846 | ||