Aren't we supposed to release the value on all possible execution paths ?

This commit was SVN r9757.

orte/mca/ras/base/ras_base_node.c

@@ -322,9 +322,9 @@ int orte_ras_base_node_query_alloc(opal_list_t* nodes, orte_jobid_t jobid)
         /* in case we get back more than we asked for */
         if(node->node_slots_inuse == 0) {
             OBJ_RELEASE(node);
-            continue;
+        } else {
+            opal_list_append(nodes, &node->super);
         }
-        opal_list_append(nodes, &node->super);
         OBJ_RELEASE(value);
     }