From bf1dd4f1ab8ef70f96704c4e2bd95968e1615b37 Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Sun, 4 Aug 2019 01:18:27 +0200 Subject: Fix functions with side effect being used inside asserts If -DNDEBUG is set, the entire thing is omitted from the output. Signed-off-by: David Oberhollenzer --- lib/fstree/fstree_from_dir.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'lib/fstree') diff --git a/lib/fstree/fstree_from_dir.c b/lib/fstree/fstree_from_dir.c index 9066809..a216226 100644 --- a/lib/fstree/fstree_from_dir.c +++ b/lib/fstree/fstree_from_dir.c @@ -22,6 +22,7 @@ static char *get_file_path(tree_node_t *n, const char *name) { char *ptr, *new; + int ret; if (n->parent == NULL) { ptr = strdup(name); @@ -34,7 +35,8 @@ static char *get_file_path(tree_node_t *n, const char *name) if (ptr == NULL) goto fail; - assert(canonicalize_name(ptr) == 0); + ret = canonicalize_name(ptr); + assert(ret == 0); new = realloc(ptr, strlen(ptr) + strlen(name) + 2); if (new == NULL) -- cgit v1.2.3