From 9d3395a98ff79596444796ba694be0ed1c8a8248 Mon Sep 17 00:00:00 2001 From: Katharina Fey Date: Sun, 21 Aug 2016 16:56:43 +0200 Subject: Fixing an issue that could cause a SIGABRT --- lib/dyn_tree.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/dyn_tree.c b/lib/dyn_tree.c index d72d1d5..2cf100d 100644 --- a/lib/dyn_tree.c +++ b/lib/dyn_tree.c @@ -225,8 +225,8 @@ dt_err dtree_split_trees(dtree *data, dtree *sp) dt_err dtree_merge_trees(dtree *data, dtree *merge) { /* REALLY make sure the type is correct */ - if(data->type == UNSET || - !(data->type == RECURSIVE || data->type == PAIR)) + if(data->type == UNSET) return INVALID_PARAMS; + if(!(data->type == RECURSIVE || data->type == PAIR)) return INVALID_PAYLOAD; /* This means elements already exist */ @@ -362,7 +362,7 @@ dt_err dtree_free(dtree *data) } else if(data->type == RECURSIVE || data->type == PAIR) { int i; dt_err err; - for(i = 0; i < data->size; i++) { + for(i = 0; i < data->used; i++) { err = dtree_free(data->payload.recursive[i]); if(err) return err; } -- cgit v1.2.3