From 62734f7e9447b5dfca1e03a760ff6c99f11528db Mon Sep 17 00:00:00 2001 From: Antoine Busque Date: Mon, 9 May 2016 16:33:29 -0400 Subject: [PATCH] Fix: remove NULL check of nonnull parameter in dlclose The `handle` parameter of `dlclose`, as defined in `include/lttng/ust-dlfcn.h` is marked `__nonnull`. GCC starting with version 6.1 emits warnings for comparisons of nonnull arguments against NULL by default (see -Wnonnull-compare). Therefore, this removes a superfluous NULL check on `handle`, for which GCC emitted a warning. Signed-off-by: Antoine Busque Signed-off-by: Mathieu Desnoyers --- liblttng-ust-dl/lttng-ust-dl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/liblttng-ust-dl/lttng-ust-dl.c b/liblttng-ust-dl/lttng-ust-dl.c index 954d9dfc..ac3c951f 100644 --- a/liblttng-ust-dl/lttng-ust-dl.c +++ b/liblttng-ust-dl/lttng-ust-dl.c @@ -144,7 +144,7 @@ void *dlopen(const char *filename, int flag) int dlclose(void *handle) { - if (__tracepoint_ptrs_registered && handle) { + if (__tracepoint_ptrs_registered) { struct link_map *p = NULL; int ret; -- 2.34.1