From: Pekka Pessi <[email protected]>
Do not check for null pointers after accessing data.
---
src/element.c | 12 ++++--------
1 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/element.c b/src/element.c
index 3309ae8..264b5a7 100644
--- a/src/element.c
+++ b/src/element.c
@@ -1138,8 +1138,7 @@ static gboolean remove_element(GNode *node, gpointer
user_data)
if (element == root)
return FALSE;
- if (node != NULL)
- g_node_unlink(node);
+ g_node_unlink(node);
if (element->driver) {
if (element->driver->remove)
@@ -1148,8 +1147,7 @@ static gboolean remove_element(GNode *node, gpointer
user_data)
element->driver = NULL;
}
- if (node != NULL)
- g_node_destroy(node);
+ g_node_destroy(node);
connman_element_unref(element);
@@ -1176,8 +1174,7 @@ static gboolean remove_element_type(GNode *node, gpointer
user_data)
if(element->type != type)
return FALSE;
- if (node != NULL)
- g_node_unlink(node);
+ g_node_unlink(node);
if (element->driver) {
if (element->driver->remove)
@@ -1186,8 +1183,7 @@ static gboolean remove_element_type(GNode *node, gpointer
user_data)
element->driver = NULL;
}
- if (node != NULL)
- g_node_destroy(node);
+ g_node_destroy(node);
connman_element_unref(element);
--
1.7.0.4
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
