Remove dead codepath
authorHenning Rogge <henning.rogge@fkie.fraunhofer.de>
Mon, 29 Oct 2012 13:37:37 +0000 (14:37 +0100)
committerHenning Rogge <henning.rogge@fkie.fraunhofer.de>
Mon, 29 Oct 2012 13:37:37 +0000 (14:37 +0100)
CID 739641: Logically dead code (DEADCODE)
Execution cannot reach this expression "NULL" inside statement "err = olsr_new_netlink_rout...".

src/common/avl.c

index 55080f7..5941369 100644 (file)
@@ -508,10 +508,8 @@ avl_delete_worker(struct avl_tree *tree, struct avl_node *node)
       avl_rotate_right(tree, parent->right);
       avl_rotate_left(tree, parent);
       avl_post_delete(tree, parent->parent);
-      return;
     }
-
-    if (parent->right == node) {
+    else {
       parent->right = NULL;
       parent->balance--;
 
@@ -537,8 +535,8 @@ avl_delete_worker(struct avl_tree *tree, struct avl_node *node)
       avl_rotate_left(tree, parent->left);
       avl_rotate_right(tree, parent);
       avl_post_delete(tree, parent->parent);
-      return;
     }
+    return;
   }
 
   if (node->left == NULL) {