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 /unpack | |
| 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 'unpack')
| -rw-r--r-- | unpack/restore_fstree.c | 5 | 
1 files changed, 3 insertions, 2 deletions
| diff --git a/unpack/restore_fstree.c b/unpack/restore_fstree.c index 7fe074e..5681e0a 100644 --- a/unpack/restore_fstree.c +++ b/unpack/restore_fstree.c @@ -9,8 +9,8 @@  static int create_node(tree_node_t *n, int flags)  {  	tree_node_t *c; +	int fd, ret;  	char *name; -	int fd;  	if (!(flags & UNPACK_QUIET)) {  		name = fstree_get_path(n); @@ -82,7 +82,8 @@ static int create_node(tree_node_t *n, int flags)  			return -1;  		} -		assert(canonicalize_name(n->data.file->input_file) == 0); +		ret = canonicalize_name(n->data.file->input_file); +		assert(ret == 0);  		break;  	default:  		break; | 
