Address post-commit patch comments.

2014-03-24  Paul Pluzhnikov  <ppluzhnikov@google.com>

	* elf/tst-dlopen-aout.c: Adjust comments. Use test-skeleton.c

diff --git a/ChangeLog b/ChangeLog
index a21154d7a1b764e86287677ed3ee3ac195dfc976..69622e485157b66656e4ca6fba28ac08db482f53 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-03-24  Paul Pluzhnikov  <ppluzhnikov@google.com>
+
+       * elf/tst-dlopen-aout.c: Adjust comments. Use test-skeleton.c
+
 2014-03-24  Joseph Myers  <joseph@codesourcery.com>
 
        * sysdeps/ieee754/dbl-64/e_exp.c: Include <math.h>.
@@ -7,7 +11,6 @@
 2014-03-24  Paul Pluzhnikov  <ppluzhnikov@google.com>
 
        [BZ #16634]
-
        * elf/dl-load.c (open_verify): Add mode parameter.
         Error early when ET_EXEC and mode does not have __RTLD_OPENEXEC.
         (open_path): Change from boolean 'secure' to complete flag 'mode'

diff --git a/elf/tst-dlopen-aout.c b/elf/tst-dlopen-aout.c
index d3534250e0c886efa4213ec50a9207a33f404add..2462a9ff5511b00c57f07e6f5c0042043ba8300d 100644 (file)
--- a/elf/tst-dlopen-aout.c
+++ b/elf/tst-dlopen-aout.c
@@ -1,5 +1,8 @@
 /* Test case for BZ #16634.
 
+   Verify that incorrectly dlopen()ing an executable without
+   __RTLD_OPENEXEC does not cause assertion in ld.so.
+
    Copyright (C) 2014 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
@@ -33,8 +36,8 @@ fn (void *p)
   return p;
 }
 
-int
-main (int argc, char *argv[])
+static int
+do_test (int argc, char *argv[])
 {
   int j;
 
@@ -59,3 +62,6 @@ main (int argc, char *argv[])
 
   return 0;
 }
+
+#define TEST_FUNCTION do_test (argc, argv)
+#include "../test-skeleton.c"