diff options
| author | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2019-08-04 01:18:27 +0200 |
|---|---|---|
| committer | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2019-08-04 01:18:27 +0200 |
| commit | bf1dd4f1ab8ef70f96704c4e2bd95968e1615b37 (patch) | |
| tree | 807ca034531c8470c5128642be118352dbbf8775 /lib/fstree | |
| parent | 85e2e1629ba43ea2b2a11b1f81e3538e3cea64da (diff) | |
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 <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'lib/fstree')
| -rw-r--r-- | lib/fstree/fstree_from_dir.c | 4 |
1 files changed, 3 insertions, 1 deletions
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) |