elm_atspi_bridge: fix bugs reported by coverity 15/229115/2
authorBartlomiej Grzelewski <b.grzelewski@samsung.com>
Fri, 27 Mar 2020 11:33:06 +0000 (11:33 +0000)
committerShinwoo Kim <cinoo.kim@samsung.com>
Tue, 31 Mar 2020 01:19:47 +0000 (10:19 +0900)
Prevent usage of uninitilized values.

CID: 1421995, 1421997, 1422001

Signed-off-by: Bartlomiej Grzelewski <b.grzelewski@samsung.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11612

Change-Id: Iaf4b2df1826a31f6d2c022f00e76e2db01732c81

src/lib/elementary/elm_atspi_bridge.c

index ebd6f98..f09951a 100644 (file)
@@ -2136,6 +2136,8 @@ _text_string_at_offset_get(const Eldbus_Service_Interface *iface, const Eldbus_M
           }
 
         str = efl_text_cursor_range_text_get(sel1, sel2);
+        start = efl_text_cursor_position_get(sel1);
+        end = efl_text_cursor_position_get(sel2);
 
         efl_del(sel1);
         efl_del(sel2);
@@ -2324,8 +2326,9 @@ _text_attribute_value_get(const Eldbus_Service_Interface *iface, const Eldbus_Me
      {
         Efl_Text_Cursor *sel1 = efl_ui_textbox_cursor_create(obj);
         Efl_Text_Cursor *sel2 = efl_ui_textbox_cursor_create(obj);
+        end = start+1;
         efl_text_cursor_position_set(sel1, start);
-        efl_text_cursor_position_set(sel2, start+1);
+        efl_text_cursor_position_set(sel2, end);
         annotations = efl_text_formatter_range_attributes_get(sel1, sel2);
 
         if (annotations)
@@ -2396,8 +2399,10 @@ _text_attributes_get(const Eldbus_Service_Interface *iface, const Eldbus_Message
         Efl_Text_Attribute_Handle *an;
         Efl_Text_Cursor *sel1 = efl_ui_textbox_cursor_create(obj);
         Efl_Text_Cursor *sel2 = efl_ui_textbox_cursor_create(obj);
+
+        end = start+1;
         efl_text_cursor_position_set(sel1, start);
-        efl_text_cursor_position_set(sel2, start+1);
+        efl_text_cursor_position_set(sel2, end);
         annotations = efl_text_formatter_range_attributes_get(sel1, sel2);
 
         efl_del(sel1);