]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
Remove __ASSUME_AT_RANDOM.
authorJoseph Myers <joseph@codesourcery.com>
Fri, 20 Jun 2014 20:02:34 +0000 (20:02 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Fri, 20 Jun 2014 20:02:34 +0000 (20:02 +0000)
This patch removes __ASSUME_AT_RANDOM now it can be assumed to be true
unconditionally.

Tested x86_64 that the disassembly of installed shared libraries is
unchanged by this patch.

* sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_AT_RANDOM):
Remove macro.
* sysdeps/unix/sysv/linux/dl-osinfo.h (_dl_setup_stack_chk_guard)
[!__ASSUME_AT_RANDOM]: Remove conditional code.
(_dl_setup_pointer_guard) [!__ASSUME_AT_RANDOM]: Likewise.

ChangeLog
sysdeps/unix/sysv/linux/dl-osinfo.h
sysdeps/unix/sysv/linux/kernel-features.h

index a87a2d0d016d5e1dc9f09a6bd0cd5bf2a4db43fa..beea12ca42320a3a3328fd11f63bbb7a1fd1bb0e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2014-06-20  Joseph Myers  <joseph@codesourcery.com>
 
+       * sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_AT_RANDOM):
+       Remove macro.
+       * sysdeps/unix/sysv/linux/dl-osinfo.h (_dl_setup_stack_chk_guard)
+       [!__ASSUME_AT_RANDOM]: Remove conditional code.
+       (_dl_setup_pointer_guard) [!__ASSUME_AT_RANDOM]: Likewise.
+
        * sysdeps/unix/sysv/linux/kernel-features.h
        (__ASSUME_ADJ_OFFSET_SS_READ): Remove macro.
        * sysdeps/unix/sysv/linux/adjtime.c (ADJTIME)
index af1343822fbb0bd9ef0a5a1685d1d96f8111adb9..fd82287f0a3b134c9581932173196093ffee6d3b 100644 (file)
@@ -57,39 +57,17 @@ _dl_setup_stack_chk_guard (void *dl_random)
     unsigned char bytes[sizeof (uintptr_t)];
   } ret;
 
-#ifndef __ASSUME_AT_RANDOM
-  if (__glibc_unlikely (dl_random == NULL))
-    {
-      const size_t filllen = sizeof (ret.bytes) - 1;
-      ret.num = 0;
-# ifdef ENABLE_STACKGUARD_RANDOMIZE
-      int fd = open_not_cancel_2 ("/dev/urandom", O_RDONLY);
-      if (fd >= 0)
-       {
-         ssize_t reslen = read_not_cancel (fd, ret.bytes + 1, filllen);
-         close_not_cancel_no_status (fd);
-         if (reslen == (ssize_t) filllen)
-           return ret.num;
-       }
-# endif
-      ret.bytes[filllen] = 255;
-      ret.bytes[filllen - 1] = '\n';
-    }
-  else
-#endif
-    {
-      /* We need in the moment only 8 bytes on 32-bit platforms and 16
-        bytes on 64-bit platforms.  Therefore we can use the data
-        directly and not use the kernel-provided data to seed a PRNG.  */
-      memcpy (ret.bytes, dl_random, sizeof (ret));
+  /* We need in the moment only 8 bytes on 32-bit platforms and 16
+     bytes on 64-bit platforms.  Therefore we can use the data
+     directly and not use the kernel-provided data to seed a PRNG.  */
+  memcpy (ret.bytes, dl_random, sizeof (ret));
 #if BYTE_ORDER == LITTLE_ENDIAN
-      ret.num &= ~(uintptr_t) 0xff;
+  ret.num &= ~(uintptr_t) 0xff;
 #elif BYTE_ORDER == BIG_ENDIAN
-      ret.num &= ~((uintptr_t) 0xff << (8 * (sizeof (ret) - 1)));
+  ret.num &= ~((uintptr_t) 0xff << (8 * (sizeof (ret) - 1)));
 #else
 # error "BYTE_ORDER unknown"
 #endif
-    }
   return ret.num;
 }
 
@@ -97,18 +75,6 @@ static inline uintptr_t __attribute__ ((always_inline))
 _dl_setup_pointer_guard (void *dl_random, uintptr_t stack_chk_guard)
 {
   uintptr_t ret;
-#ifndef __ASSUME_AT_RANDOM
-  if (dl_random == NULL)
-    {
-      ret = stack_chk_guard;
-# ifndef HP_TIMING_NONAVAIL
-      hp_timing_t now;
-      HP_TIMING_NOW (now);
-      ret ^= now;
-# endif
-    }
-  else
-#endif
-    memcpy (&ret, (char *) dl_random + sizeof (ret), sizeof (ret));
+  memcpy (&ret, (char *) dl_random + sizeof (ret), sizeof (ret));
   return ret;
 }
index d0673211ad09a96a80a973f5a483a0c8c5f4befe..80655e0162043b3fec22cde44f272a7247a16c20 100644 (file)
 /* Support for the FUTEX_CLOCK_REALTIME flag was added in 2.6.29.  */
 #define __ASSUME_FUTEX_CLOCK_REALTIME  1
 
-/* Support for the AT_RANDOM auxiliary vector entry was added in 2.6.29.  */
-#define __ASSUME_AT_RANDOM     1
-
 /* Support for preadv and pwritev was added in 2.6.30.  */
 #define __ASSUME_PREADV        1
 #define __ASSUME_PWRITEV       1