diff options
author | Chris Michael <cp.michael@samsung.com> | 2017-11-06 10:17:29 -0500 |
---|---|---|
committer | Chris Michael <cp.michael@samsung.com> | 2017-11-06 10:17:29 -0500 |
commit | 12947e01e89efc2a1e5fb4e044e798955f59af13 (patch) | |
tree | df731caae15f583d886902f45cd02cf35ac5ed53 | |
parent | 6c76fdb8245fb6dc4226cb7b0707aa93f1d9422a (diff) |
evas-object-main: Fix potential null dereference
Coverity reports that EVAS_OBJECT_DATA_SAFE_GET returns NULL here (184
out of 188 times). As such, we should not be calling
_event_animation_object_get with a null object so add a null check here.
Fixes Coverity CID1381713
@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, 2 insertions, 0 deletions
diff --git a/src/lib/evas/canvas/evas_object_main.c b/src/lib/evas/canvas/evas_object_main.c index dce4954e08..57f8dd7bc7 100644 --- a/src/lib/evas/canvas/evas_object_main.c +++ b/src/lib/evas/canvas/evas_object_main.c | |||
@@ -2829,6 +2829,8 @@ _efl_canvas_object_event_animation_is_running(Eo *eo_obj, | |||
2829 | { | 2829 | { |
2830 | Evas_Object_Protected_Data *obj = EVAS_OBJECT_DATA_SAFE_GET(eo_obj); | 2830 | Evas_Object_Protected_Data *obj = EVAS_OBJECT_DATA_SAFE_GET(eo_obj); |
2831 | 2831 | ||
2832 | if (!obj) return EINA_FALSE; | ||
2833 | |||
2832 | if (_event_animation_object_get(obj, desc)) | 2834 | if (_event_animation_object_get(obj, desc)) |
2833 | return EINA_TRUE; | 2835 | return EINA_TRUE; |
2834 | 2836 | ||