From 5effc3ae744807d03c6779040d752c74ed4abfbf Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Fri, 4 Dec 2015 17:44:29 -0700 Subject: [PATCH] gallium/util: check callback pointers for non-null in pipe_debug_message() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit So the callers don't have to do it. v2: also check cb!=NULL in the macro Reviewed-by: Ilia Mirkin Reviewed-by: José Fonseca --- src/gallium/auxiliary/util/u_debug.h | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/gallium/auxiliary/util/u_debug.h b/src/gallium/auxiliary/util/u_debug.h index 99007039a7c..5307072fa3a 100644 --- a/src/gallium/auxiliary/util/u_debug.h +++ b/src/gallium/auxiliary/util/u_debug.h @@ -268,9 +268,11 @@ void _debug_assert_fail(const char *expr, */ #define pipe_debug_message(cb, type, fmt, ...) do { \ static unsigned id = 0; \ - _pipe_debug_message(cb, &id, \ - PIPE_DEBUG_TYPE_ ## type, \ - fmt, ##__VA_ARGS__); \ + if ((cb) && (cb)->debug_message) { \ + _pipe_debug_message(cb, &id, \ + PIPE_DEBUG_TYPE_ ## type, \ + fmt, ##__VA_ARGS__); \ + } \ } while (0) struct pipe_debug_callback; -- 2.30.2