Commit ce2160ab authored by Chris Michael's avatar Chris Michael

elementary: Remove logically dead code

In all code paths leading to this goto statement being called,
'iter_array' variable will already be NULL, so checking it's validity
is just logically dead code.

Fixes Coverity CID1357392

@fix
Signed-off-by: default avatarChris Michael <cp.michael@samsung.com>
parent 4309050f
......@@ -1158,6 +1158,7 @@ _action_actions_get(const Eldbus_Service_Interface *iface, const Eldbus_Message
{
const char *descr;
char *key;
key = elm_interface_atspi_action_keybinding_get(obj, id);
descr = elm_interface_atspi_action_description_get(obj, id);
descr = descr ? descr : "";
......@@ -1169,7 +1170,6 @@ _action_actions_get(const Eldbus_Service_Interface *iface, const Eldbus_Message
return ret;
error:
if (iter_array) eldbus_message_iter_container_close(iter, iter_array);
if (ret) eldbus_message_unref(ret);
return NULL;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment