evas: Fix directly dereferencing pointer which may be NULL
authorChris Michael <cp.michael@samsung.com>
Thu, 13 Oct 2016 16:50:12 +0000 (12:50 -0400)
committerChris Michael <cp.michael@samsung.com>
Thu, 13 Oct 2016 16:50:12 +0000 (12:50 -0400)
Coverity reports this as a dereference before null check which implies
that 'cur' May be null here, so let's not use it before we check it.

Fixes CID1363765

Signed-off-by: Chris Michael <cp.michael@samsung.com>
src/lib/evas/canvas/evas_object_textblock.c

index 2b01c0a..29d8cb9 100644 (file)
@@ -12185,10 +12185,12 @@ EAPI Eina_List *
 evas_textblock_cursor_range_geometry_get(const Efl_Canvas_Text_Cursor *cur1_obj, const Evas_Textblock_Cursor *cur2_obj)
 {
    Efl_Canvas_Text_Cursor_Data *cur = efl_data_scope_get(cur1_obj, EFL_CANVAS_TEXT_CURSOR_CLASS);
-   Efl_Canvas_Text_Data *o = efl_data_scope_get(cur->obj, MY_CLASS);
+   Efl_Canvas_Text_Data *o;
 
    if (!cur) return NULL;
 
+   o = efl_data_scope_get(cur->obj, MY_CLASS);
+
    return _efl_canvas_text_range_geometry_list_get(cur->obj, o, cur1_obj, cur2_obj);
 }