diff options
author | Robin Hack <hack.robin@gmail.com> | 2016-04-26 13:02:01 +0200 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2016-04-26 22:48:21 +0200 |
commit | b9ffb9322b987409693e4faa277b5e3f46ad8b22 (patch) | |
tree | 0ee9bbd2fc3298e7e4702e07e00bbe1c854a17de /lib/talloc | |
parent | f9099d3c469efd4085ca0b0e81941c77f1b61c4f (diff) | |
download | samba-b9ffb9322b987409693e4faa277b5e3f46ad8b22.tar.gz |
talloc/testsuite: Fix CID 1291641 - Logically dead code
Add check for snprintf return code.
Signed-off-by: Robin Hack <hack.robin@gmail.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Diffstat (limited to 'lib/talloc')
-rw-r--r-- | lib/talloc/testsuite.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/talloc/testsuite.c b/lib/talloc/testsuite.c index 34410b81c3f..5eab839f51d 100644 --- a/lib/talloc/testsuite.c +++ b/lib/talloc/testsuite.c @@ -1795,11 +1795,11 @@ static bool test_pthread_talloc_passing(void) * They will use their own toplevel contexts. */ for (i = 0; i < NUM_THREADS; i++) { - (void)snprintf(str_array[i], - 20, - "thread:%d", - i); - if (str_array[i] == NULL) { + ret = snprintf(str_array[i], + 20, + "thread:%d", + i); + if (ret < 0) { printf("snprintf %d failed\n", i); return false; } |