From 9114625bad23441c89eac5a7dcf319a9714ca31f Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Thu, 27 Nov 2014 16:00:08 +0000 Subject: [PATCH] Fix dlfcn/failtestmod.c warning. This patch fixes a "set but not used" warning from dlfcn/failtestmod.c. A variable is used only to store the return value from dlsym. As I understand this test, the point is simply to do a sequence of load / unload operations in a loop, and all that matters here is that dlsym gets called and returns without crashing, not what its return value is. So this patch removes the assignment to a variable. Tested for x86_64. * dlfcn/failtestmod.c (constr): Do not store result of dlsym in a variable. --- ChangeLog | 5 +++++ dlfcn/failtestmod.c | 3 +-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 92ffe16bd6..9090877c86 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2014-11-27 Joseph Myers + + * dlfcn/failtestmod.c (constr): Do not store result of dlsym in a + variable. + 2014-11-27 Stefan Liebler * nscd/connections.c: Include libc-internal.h because of macro diff --git a/dlfcn/failtestmod.c b/dlfcn/failtestmod.c index a03f90b734..64dadd53ff 100644 --- a/dlfcn/failtestmod.c +++ b/dlfcn/failtestmod.c @@ -8,7 +8,6 @@ __attribute__ ((__constructor__)) constr (void) { void *handle; - void *m; /* Open the library. */ handle = dlopen (NULL, RTLD_NOW); @@ -19,7 +18,7 @@ constr (void) } /* Get a symbol. */ - m = dlsym (handle, "main"); + dlsym (handle, "main"); puts ("called dlsym() to get main"); dlclose (handle);