c6d234
commit 9114625bad23441c89eac5a7dcf319a9714ca31f
c6d234
Author: Joseph Myers <joseph@codesourcery.com>
c6d234
Date:   Thu Nov 27 16:00:08 2014 +0000
c6d234
c6d234
    Fix dlfcn/failtestmod.c warning.
c6d234
    
c6d234
    This patch fixes a "set but not used" warning from
c6d234
    dlfcn/failtestmod.c.  A variable is used only to store the return
c6d234
    value from dlsym.  As I understand this test, the point is simply to
c6d234
    do a sequence of load / unload operations in a loop, and all that
c6d234
    matters here is that dlsym gets called and returns without crashing,
c6d234
    not what its return value is.  So this patch removes the assignment to
c6d234
    a variable.
c6d234
    
c6d234
    Tested for x86_64.
c6d234
    
c6d234
            * dlfcn/failtestmod.c (constr): Do not store result of dlsym in a
c6d234
            variable.
c6d234
c6d234
diff --git a/dlfcn/failtestmod.c b/dlfcn/failtestmod.c
c6d234
index a03f90b734132d8d..64dadd53ff8ef109 100644
c6d234
--- a/dlfcn/failtestmod.c
c6d234
+++ b/dlfcn/failtestmod.c
c6d234
@@ -8,7 +8,6 @@ __attribute__ ((__constructor__))
c6d234
 constr (void)
c6d234
 {
c6d234
   void *handle;
c6d234
-  void *m;
c6d234
 
c6d234
   /* Open the library.  */
c6d234
   handle = dlopen (NULL, RTLD_NOW);
c6d234
@@ -19,7 +18,7 @@ constr (void)
c6d234
     }
c6d234
 
c6d234
   /* Get a symbol.  */
c6d234
-  m = dlsym (handle, "main");
c6d234
+  dlsym (handle, "main");
c6d234
   puts ("called dlsym() to get main");
c6d234
 
c6d234
   dlclose (handle);