diff options
| author | Gert Wollny <[email protected]> | 2017-11-16 16:09:36 +0100 |
|---|---|---|
| committer | Brian Paul <[email protected]> | 2017-11-17 09:27:56 -0700 |
| commit | 373c263e2c3886c88e84d853b9df61ad03d1a8ea (patch) | |
| tree | c7cd3cbcb984b5df4a92d3ae1fd3118e4fc77c2f | |
| parent | 9b80c03870f97be1e5ed1ed5c7b82b87749760f2 (diff) | |
gallium/aux/util/u_debug_stack.c: Silence -Wunused-result warning
asprintf is decorated with the attrbute "warn_unused_result", and if the
function call fails, the pointer "temp" will be undefined, but since it is
used later it should contain some usable value.
Test return value of asprintf and assign some save value to "temp" if
the call failed.
Signed-off-by: Gert Wollny <[email protected]>
Reviewed-by: Brian Paul <[email protected]> (v1)
| -rw-r--r-- | src/gallium/auxiliary/util/u_debug_stack.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/gallium/auxiliary/util/u_debug_stack.c b/src/gallium/auxiliary/util/u_debug_stack.c index 7013807b6b2..6ddacdb362f 100644 --- a/src/gallium/auxiliary/util/u_debug_stack.c +++ b/src/gallium/auxiliary/util/u_debug_stack.c @@ -90,8 +90,8 @@ symbol_name_cached(unw_cursor_t *cursor, unw_proc_info_t *pip) procname[1] = 0; } - asprintf(&name, "%s%s", procname, ret == -UNW_ENOMEM ? "..." : ""); - + if (asprintf(&name, "%s%s", procname, ret == -UNW_ENOMEM ? "..." : "") == -1) + name = "??"; util_hash_table_set(symbols_hash, addr, (void*)name); } mtx_unlock(&symbols_mutex); |