about summary refs log tree commit diff
path: root/dlfcn/failtestmod.c
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2014-11-27 16:00:08 +0000
committerJoseph Myers <joseph@codesourcery.com>2014-11-27 16:00:08 +0000
commit9114625bad23441c89eac5a7dcf319a9714ca31f (patch)
treedd4e86ab370d83138a52247f287eaabaf6b17811 /dlfcn/failtestmod.c
parente7300fef49147a73d5dc7e0a462cc60aabb58400 (diff)
downloadglibc-9114625bad23441c89eac5a7dcf319a9714ca31f.tar.gz
glibc-9114625bad23441c89eac5a7dcf319a9714ca31f.tar.xz
glibc-9114625bad23441c89eac5a7dcf319a9714ca31f.zip
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.
Diffstat (limited to 'dlfcn/failtestmod.c')
-rw-r--r--dlfcn/failtestmod.c3
1 files changed, 1 insertions, 2 deletions
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);