Move code to disable ASR to nat/
authorSergio Durigan Junior <sergiodj@redhat.com>
Thu, 15 Jan 2015 20:10:49 +0000 (15:10 -0500)
committerSergio Durigan Junior <sergiodj@redhat.com>
Thu, 15 Jan 2015 20:10:49 +0000 (15:10 -0500)
This patch moves the shared code present on
gdb/linux-nat.c:linux_nat_create_inferior and
gdb/gdbserver/linux-low.c:linux_create_inferior to
nat/linux-personality.c.  This code is responsible for disabling
address space randomization based on user setting, and using
<sys/personality.h> to do that.  I decided to put the prototype of the
maybe_disable_address_space_randomization on nat/linux-osdata.h
because it seemed the best place to put it.

I regression-tested this patch on Fedora 20 x86_64, and found no
regressions.

gdb/ChangeLog
2015-01-15  Sergio Durigan Junior  <sergiodj@redhat.com>

* Makefile.in (HFILES_NO_SRCDIR): Add nat/linux-personality.h.
(linux-personality.o): New rule.
* common/common-defs.h: Include <stdint.h>.
* config/aarch64/linux.mh (NATDEPFILES): Include
linux-personality.o.
* config/alpha/alpha-linux.mh (NATDEPFILES): Likewise.
* config/arm/linux.mh (NATDEPFILES): Likewise.
* config/i386/linux64.mh (NATDEPFILES): Likewise.
* config/i386/linux.mh (NATDEPFILES): Likewise.
* config/ia64/linux.mh (NATDEPFILES): Likewise.
* config/m32r/linux.mh (NATDEPFILES): Likewise.
* config/m68k/linux.mh (NATDEPFILES): Likewise.
* config/mips/linux.mh (NATDEPFILES): Likewise.
* config/pa/linux.mh (NATDEPFILES): Likewise.
* config/powerpc/linux.mh (NATDEPFILES): Likewise.
* config/powerpc/ppc64-linux.mh (NATDEPFILES): Likewise.
* config/powerpc/spu-linux.mh (NATDEPFILES): Likewise.
* config/s390/linux.mh (NATDEPFILES): Likewise.
* config/sparc/linux64.mh (NATDEPFILES): Likewise.
* config/sparc/linux.mh (NATDEPFILES): Likewise.
* config/tilegx/linux.mh (NATDEPFILES): Likewise.
* config/xtensa/linux.mh (NATDEPFILES): Likewise.
* defs.h: Remove #include <stdint.h> (moved to
common/common-defs.h).
* linux-nat.c: Include nat/linux-personality.h.  Remove #include
<sys/personality.h>; do not define ADDR_NO_RANDOMIZE (moved to
nat/linux-personality.c).
(linux_nat_create_inferior): Remove code to disable address space
randomization (moved to nat/linux-personality.c).  Create cleanup
to disable address space randomization.
* nat/linux-personality.c: New file.
* nat/linux-personality.h: Likewise.

gdb/gdbserver/ChangeLog
2015-01-15  Sergio Durigan Junior  <sergiodj@redhat.com>

* Makefile.in (SFILES): Add linux-personality.c.
(linux-personality.o): New rule.
* configure.srv (srv_linux_obj): Add linux-personality.o to the
list of objects to be built.
* linux-low.c: Include nat/linux-personality.h.
(linux_create_inferior): Remove code to disable address space
randomization (moved to ../nat/linux-personality.c).  Create
cleanup to disable address space randomization.

29 files changed:
gdb/ChangeLog
gdb/Makefile.in
gdb/common/common-defs.h
gdb/config/aarch64/linux.mh
gdb/config/alpha/alpha-linux.mh
gdb/config/arm/linux.mh
gdb/config/i386/linux.mh
gdb/config/i386/linux64.mh
gdb/config/ia64/linux.mh
gdb/config/m32r/linux.mh
gdb/config/m68k/linux.mh
gdb/config/mips/linux.mh
gdb/config/pa/linux.mh
gdb/config/powerpc/linux.mh
gdb/config/powerpc/ppc64-linux.mh
gdb/config/powerpc/spu-linux.mh
gdb/config/s390/linux.mh
gdb/config/sparc/linux.mh
gdb/config/sparc/linux64.mh
gdb/config/tilegx/linux.mh
gdb/config/xtensa/linux.mh
gdb/defs.h
gdb/gdbserver/ChangeLog
gdb/gdbserver/Makefile.in
gdb/gdbserver/configure.srv
gdb/gdbserver/linux-low.c
gdb/linux-nat.c
gdb/nat/linux-personality.c [new file with mode: 0644]
gdb/nat/linux-personality.h [new file with mode: 0644]

index 985097aec44b0142abbeb44780f11816fc2282ae..37955d4b398e72ee726d6b3f0dd6f07ee7cd22b8 100644 (file)
@@ -1,3 +1,38 @@
+2015-01-15  Sergio Durigan Junior  <sergiodj@redhat.com>
+
+       * Makefile.in (HFILES_NO_SRCDIR): Add nat/linux-personality.h.
+       (linux-personality.o): New rule.
+       * common/common-defs.h: Include <stdint.h>.
+       * config/aarch64/linux.mh (NATDEPFILES): Include
+       linux-personality.o.
+       * config/alpha/alpha-linux.mh (NATDEPFILES): Likewise.
+       * config/arm/linux.mh (NATDEPFILES): Likewise.
+       * config/i386/linux64.mh (NATDEPFILES): Likewise.
+       * config/i386/linux.mh (NATDEPFILES): Likewise.
+       * config/ia64/linux.mh (NATDEPFILES): Likewise.
+       * config/m32r/linux.mh (NATDEPFILES): Likewise.
+       * config/m68k/linux.mh (NATDEPFILES): Likewise.
+       * config/mips/linux.mh (NATDEPFILES): Likewise.
+       * config/pa/linux.mh (NATDEPFILES): Likewise.
+       * config/powerpc/linux.mh (NATDEPFILES): Likewise.
+       * config/powerpc/ppc64-linux.mh (NATDEPFILES): Likewise.
+       * config/powerpc/spu-linux.mh (NATDEPFILES): Likewise.
+       * config/s390/linux.mh (NATDEPFILES): Likewise.
+       * config/sparc/linux64.mh (NATDEPFILES): Likewise.
+       * config/sparc/linux.mh (NATDEPFILES): Likewise.
+       * config/tilegx/linux.mh (NATDEPFILES): Likewise.
+       * config/xtensa/linux.mh (NATDEPFILES): Likewise.
+       * defs.h: Remove #include <stdint.h> (moved to
+       common/common-defs.h).
+       * linux-nat.c: Include nat/linux-personality.h.  Remove #include
+       <sys/personality.h>; do not define ADDR_NO_RANDOMIZE (moved to
+       nat/linux-personality.c).
+       (linux_nat_create_inferior): Remove code to disable address space
+       randomization (moved to nat/linux-personality.c).  Create cleanup
+       to disable address space randomization.
+       * nat/linux-personality.c: New file.
+       * nat/linux-personality.h: Likewise.
+
 2015-01-15  Sergio Durigan Junior  <sergiodj@redhat.com>
 
        * Makefile.in (ALLDEPFILES): Including common/mingw-strerror.c and
index 57ed361f3ebcbdc055a493c87225d7cd6eaa0384..6b270ebf2804c9057436118b2099308a4b453157 100644 (file)
@@ -961,7 +961,7 @@ common/print-utils.h common/rsp-low.h nat/x86-dregs.h x86-linux-nat.h \
 i386-linux-nat.h common/common-defs.h common/errors.h common/common-types.h \
 common/common-debug.h common/cleanups.h common/gdb_setjmp.h \
 common/common-exceptions.h target/target.h common/symbol.h \
-common/common-regcache.h fbsd-tdep.h
+common/common-regcache.h fbsd-tdep.h nat/linux-personality.h
 
 # Header files that already have srcdir in them, or which are in objdir.
 
@@ -2282,6 +2282,10 @@ ppc-linux.o: ${srcdir}/nat/ppc-linux.c
        $(COMPILE) $(srcdir)/nat/ppc-linux.c
        $(POSTCOMPILE)
 
+linux-personality.o: ${srcdir}/nat/linux-personality.c
+       $(COMPILE) $(srcdir)/nat/linux-personality.c
+       $(POSTCOMPILE)
+
 #
 # gdb/tui/ dependencies
 #
index 1873211d572549d756edba2b7b0d18b920e3f3dc..e80d3327052fdc514e66df2c55444d962022b74f 100644 (file)
@@ -31,6 +31,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <stddef.h>
+#include <stdint.h>
 #include <string.h>
 #include <errno.h>
 #include <alloca.h>
index af04f46730c2854065737ba75d914ff5e2596214..7f96e4db6e20ee90092f31e4ae55c3e53fbb2fa9 100644 (file)
@@ -21,7 +21,8 @@
 NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o aarch64-linux-nat.o \
        proc-service.o linux-thread-db.o linux-nat.o linux-fork.o \
-       linux-procfs.o linux-ptrace.o linux-osdata.o linux-waitpid.o
+       linux-procfs.o linux-ptrace.o linux-osdata.o linux-waitpid.o \
+       linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 LOADLIBES= -ldl $(RDYNAMIC)
index 81819a16bb02759e130dba0bacd5fae9d7ea569c..2ea02a101986cd22b0e1899872274e1e80b2dffc 100644 (file)
@@ -3,7 +3,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o alpha-linux-nat.o \
        fork-child.o proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o
+       linux-waitpid.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 # The dynamically loaded libthread_db needs access to symbols in the
index 63c3eae45cd93f18031d3533797b370d37c16d07..549bf4259435f4b299960e457293c021b36cf403 100644 (file)
@@ -4,7 +4,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o arm-linux-nat.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o
+       linux-waitpid.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 LOADLIBES= -ldl $(RDYNAMIC)
index a4e4842c93c0f580823918fbff0a290afc81d434..33fb2812c64fdeb1c8c2fbf720c365bee1c1f109 100644 (file)
@@ -5,7 +5,7 @@ NATDEPFILES= inf-ptrace.o fork-child.o \
        x86-nat.o x86-dregs.o i386-linux-nat.o x86-linux-nat.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-btrace.o linux-waitpid.o
+       linux-btrace.o linux-waitpid.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 # The dynamically loaded libthread_db needs access to symbols in the
index d557202994a34cb18718df10d6920934dbb777e5..d52eb5269b4574d954d751a5eb814469f3a0253f 100644 (file)
@@ -5,7 +5,7 @@ NATDEPFILES= inf-ptrace.o fork-child.o \
        linux-nat.o linux-osdata.o \
        proc-service.o linux-thread-db.o linux-fork.o \
        linux-procfs.o linux-ptrace.o linux-btrace.o \
-       linux-waitpid.o
+       linux-waitpid.o linux-personality.o
 NAT_FILE= config/nm-linux.h
 NAT_CDEPS = $(srcdir)/proc-service.list
 
index baa8709d043f0c12ce644d99c8f5b72ec2027862..9dce22b55886c4f2f3651d503918d03533eec456 100644 (file)
@@ -5,6 +5,7 @@ NATDEPFILES= inf-ptrace.o fork-child.o \
        ia64-linux-nat.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o \
+       linux-personality.o \
        linux-procfs.o linux-ptrace.o linux-waitpid.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
index e9bb82cbc5a89cdb6776a30e66a5b0dbb86e0010..6b810e6ae7da94076cbe2a25fd3732886f6f8312 100644 (file)
@@ -4,7 +4,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o                         \
        m32r-linux-nat.o proc-service.o linux-thread-db.o       \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o
+       linux-waitpid.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 LOADLIBES= -ldl $(RDYNAMIC)
index bcec2958a14ff2089669ac2185236fd57945cc97..f3b3baae2a4f0f0e29393ddde9377e6eabd0bc29 100644 (file)
@@ -5,6 +5,7 @@ NATDEPFILES= inf-ptrace.o fork-child.o \
        m68klinux-nat.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
+       linux-personality.o \
        linux-waitpid.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
index 021838e0af7d38cb554f9d125cb13dc61659b149..d6a802fcb1a61dcd14aef601c66d0218d06ea1a8 100644 (file)
@@ -4,6 +4,7 @@ NATDEPFILES= inf-ptrace.o fork-child.o mips-linux-nat.o \
        linux-thread-db.o proc-service.o \
        linux-nat.o linux-osdata.o linux-fork.o \
        linux-procfs.o linux-ptrace.o linux-waitpid.o \
+       linux-personality.o \
        mips-linux-watch.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
index f1c0c32783b0eb1fdd3fa41bf4a737fc3ecdb1cc..9539b6476147f1ea80b881d6f02851ec310b1255 100644 (file)
@@ -3,7 +3,8 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o \
        hppa-linux-nat.o proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o \
-       linux-procfs.o linux-ptrace.o linux-waitpid.o
+       linux-procfs.o linux-ptrace.o linux-waitpid.o \
+       linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 LOADLIBES = -ldl $(RDYNAMIC)
index a807d3fac063a86c60f45ded9392a58f325e3a3e..76e62c0b9648310cd378206428dedd7610a103d9 100644 (file)
@@ -6,7 +6,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o \
        ppc-linux-nat.o proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o
+       linux-waitpid.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 LOADLIBES = -ldl $(RDYNAMIC)
index 1b3fbc76c00e331b40aee46d6baed2f0b7139284..7eb6507d9423b1119533387da2806b8f4001e0ec 100644 (file)
@@ -6,7 +6,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o \
        ppc-linux-nat.o proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o ppc-linux.o
+       linux-waitpid.o ppc-linux.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 # The PowerPC has severe limitations on TOC size, and uses them even
index 7ce7a5ff141c320e5274ad9fcba9d55fcf16f885..d44aeebf4268ca539ea6127bd1c106c631b0c5a0 100644 (file)
@@ -4,5 +4,5 @@
 # PPU side of the Cell BE and debugging the SPU side.
 
 NATDEPFILES = spu-linux-nat.o fork-child.o inf-ptrace.o \
-             linux-procfs.o linux-ptrace.o linux-waitpid.o
+             linux-procfs.o linux-ptrace.o linux-waitpid.o linux-personality.o
 
index 59bab838b643856d031561fffb3476b1d54fc38e..e1ad8990ebabf9ef9a76e27301f4296ba2ea8830 100644 (file)
@@ -3,6 +3,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o s390-linux-nat.o \
        linux-thread-db.o proc-service.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
+       linux-personality.o \
        linux-waitpid.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 LOADLIBES = -ldl $(RDYNAMIC)
index 009ea9576c68806d11a121c4e38ae09667bab9b9..bd7fc86614823568bcec96a7b8d00fac92707350 100644 (file)
@@ -4,7 +4,8 @@ NATDEPFILES= sparc-nat.o sparc-linux-nat.o \
        fork-child.o inf-ptrace.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o \
-       linux-procfs.o linux-ptrace.o linux-waitpid.o
+       linux-procfs.o linux-ptrace.o linux-waitpid.o \
+       linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 # The dynamically loaded libthread_db needs access to symbols in the
index aad77c7ae707f1f67ed4490a1580f1a102e385f0..86f984fb3c2e7e045368e9db584b271738f83ad3 100644 (file)
@@ -4,7 +4,8 @@ NATDEPFILES= sparc-nat.o sparc64-nat.o sparc64-linux-nat.o \
        fork-child.o inf-ptrace.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o \
-       linux-procfs.o linux-ptrace.o linux-waitpid.o
+       linux-procfs.o linux-ptrace.o linux-waitpid.o \
+       linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 # The dynamically loaded libthread_db needs access to symbols in the
index 1ed9c33c1207394629a472edec0d300e465722d2..b5edcd493791ff97b560aa285918aa4d8e0afd05 100644 (file)
@@ -5,7 +5,8 @@ NATDEPFILES= inf-ptrace.o fork-child.o \
        tilegx-linux-nat.o \
        proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o \
-       linux-procfs.o linux-ptrace.o linux-waitpid.o
+       linux-procfs.o linux-ptrace.o linux-waitpid.o \
+       linux-personality.o
 
 # The dynamically loaded libthread_db needs access to symbols in the
 # gdb executable.
index 8ef84b46cdf9aac53134369ee3e3e4ee34de3165..b4e59b369d723de37ef66adbae810b4530bc5625 100644 (file)
@@ -5,7 +5,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o xtensa-linux-nat.o \
        linux-thread-db.o proc-service.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o
+       linux-waitpid.o linux-personality.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 LOADLIBES = -ldl $(RDYNAMIC)
index 99db465b93d37f37df777679960ff5e1ad7b2cee..a1cd45f91f4abae19c27bac0c7af8600c1af89c6 100644 (file)
@@ -29,7 +29,6 @@
 
 #include <sys/types.h>
 #include <limits.h>
-#include <stdint.h>
 
 /* The libdecnumber library, on which GDB depends, includes a header file
    called gstdint.h instead of relying directly on stdint.h.  GDB, on the
index c2d8e5672b4f7e6e5a7c899b918099cbda14ab37..51f1019777d7f985da8b9446e3e356092184067e 100644 (file)
@@ -1,3 +1,14 @@
+2015-01-15  Sergio Durigan Junior  <sergiodj@redhat.com>
+
+       * Makefile.in (SFILES): Add linux-personality.c.
+       (linux-personality.o): New rule.
+       * configure.srv (srv_linux_obj): Add linux-personality.o to the
+       list of objects to be built.
+       * linux-low.c: Include nat/linux-personality.h.
+       (linux_create_inferior): Remove code to disable address space
+       randomization (moved to ../nat/linux-personality.c).  Create
+       cleanup to disable address space randomization.
+
 2015-01-15  Sergio Durigan Junior  <sergiodj@redhat.com>
 
        * Makefile.in (posix-strerror.o): New rule.
index 8406ff4db5fed22f7db285a58f77af62c3d97b54..b455c5b34584e4bc3115acffe16a411d44ba1870 100644 (file)
@@ -167,6 +167,7 @@ SFILES=     $(srcdir)/gdbreplay.c $(srcdir)/inferiors.c $(srcdir)/dll.c \
        $(srcdir)/common/vec.c $(srcdir)/common/gdb_vecs.c \
        $(srcdir)/common/common-utils.c $(srcdir)/common/xml-utils.c \
        $(srcdir)/nat/linux-osdata.c $(srcdir)/common/ptid.c \
+       $(srcdir)/nat/linux-personality.c \
        $(srcdir)/common/buffer.c $(srcdir)/nat/linux-btrace.c \
        $(srcdir)/common/filestuff.c $(srcdir)/target/waitstatus.c \
        $(srcdir)/nat/mips-linux-watch.c $(srcdir)/common/print-utils.c \
@@ -590,6 +591,9 @@ mips-linux-watch.o: ../nat/mips-linux-watch.c
 ppc-linux.o: ../nat/ppc-linux.c
        $(COMPILE) $<
        $(POSTCOMPILE)
+linux-personality.o: ../nat/linux-personality.c
+       $(COMPILE) $<
+       $(POSTCOMPILE)
 
 aarch64.c : $(srcdir)/../regformats/aarch64.dat $(regdat_sh)
        $(SHELL) $(regdat_sh) $(srcdir)/../regformats/aarch64.dat aarch64.c
index f52aee25e3dcf34bd9e09cf294cbf510b2f6e03e..127786e8859ba7bbba4c357dc6fd28eb19916f69 100644 (file)
@@ -42,7 +42,7 @@ srv_amd64_linux_xmlfiles="i386/amd64-linux.xml i386/amd64-avx-linux.xml i386/amd
 
 # Linux object files.  This is so we don't have to repeat
 # these files over and over again.
-srv_linux_obj="linux-low.o linux-osdata.o linux-procfs.o linux-ptrace.o linux-waitpid.o"
+srv_linux_obj="linux-low.o linux-osdata.o linux-procfs.o linux-ptrace.o linux-waitpid.o linux-personality.o"
 
 # Input is taken from the "${target}" variable.
 
index 4d19c870da2f447d019b3e899f8a53f283dbcb98..5e37dd55a409a52e5bcfdc53ce70a85f9d05a5a5 100644 (file)
@@ -27,6 +27,7 @@
 #include <sys/ptrace.h>
 #include "nat/linux-ptrace.h"
 #include "nat/linux-procfs.h"
+#include "nat/linux-personality.h"
 #include <signal.h>
 #include <sys/ioctl.h>
 #include <fcntl.h>
@@ -597,29 +598,11 @@ add_lwp (ptid_t ptid)
 static int
 linux_create_inferior (char *program, char **allargs)
 {
-#ifdef HAVE_PERSONALITY
-  int personality_orig = 0, personality_set = 0;
-#endif
   struct lwp_info *new_lwp;
   int pid;
   ptid_t ptid;
-
-#ifdef HAVE_PERSONALITY
-  if (disable_randomization)
-    {
-      errno = 0;
-      personality_orig = personality (0xffffffff);
-      if (errno == 0 && !(personality_orig & ADDR_NO_RANDOMIZE))
-       {
-         personality_set = 1;
-         personality (personality_orig | ADDR_NO_RANDOMIZE);
-       }
-      if (errno != 0 || (personality_set
-                        && !(personality (0xffffffff) & ADDR_NO_RANDOMIZE)))
-       warning ("Error disabling address space randomization: %s",
-                strerror (errno));
-    }
-#endif
+  struct cleanup *restore_personality
+    = maybe_disable_address_space_randomization (disable_randomization);
 
 #if defined(__UCLIBC__) && defined(HAS_NOMMU)
   pid = vfork ();
@@ -665,16 +648,7 @@ linux_create_inferior (char *program, char **allargs)
       _exit (0177);
     }
 
-#ifdef HAVE_PERSONALITY
-  if (personality_set)
-    {
-      errno = 0;
-      personality (personality_orig);
-      if (errno != 0)
-       warning ("Error restoring address space randomization: %s",
-                strerror (errno));
-    }
-#endif
+  do_cleanups (restore_personality);
 
   linux_add_process (pid, 0);
 
index a8a63cf07382e5ec9fbaf6261454aea1388cb9d2..be524704516900d536755b2645cafb97e8da3fa1 100644 (file)
@@ -32,6 +32,7 @@
 #include "linux-nat.h"
 #include "nat/linux-ptrace.h"
 #include "nat/linux-procfs.h"
+#include "nat/linux-personality.h"
 #include "linux-fork.h"
 #include "gdbthread.h"
 #include "gdbcmd.h"
 #define SPUFS_MAGIC 0x23c9b64e
 #endif
 
-#ifdef HAVE_PERSONALITY
-# include <sys/personality.h>
-# if !HAVE_DECL_ADDR_NO_RANDOMIZE
-#  define ADDR_NO_RANDOMIZE 0x0040000
-# endif
-#endif /* HAVE_PERSONALITY */
-
 /* This comment documents high-level logic of this file.
 
 Waiting for events in sync mode
@@ -1103,45 +1097,18 @@ linux_nat_create_inferior (struct target_ops *ops,
                           char *exec_file, char *allargs, char **env,
                           int from_tty)
 {
-#ifdef HAVE_PERSONALITY
-  int personality_orig = 0, personality_set = 0;
-#endif /* HAVE_PERSONALITY */
+  struct cleanup *restore_personality
+    = maybe_disable_address_space_randomization (disable_randomization);
 
   /* The fork_child mechanism is synchronous and calls target_wait, so
      we have to mask the async mode.  */
 
-#ifdef HAVE_PERSONALITY
-  if (disable_randomization)
-    {
-      errno = 0;
-      personality_orig = personality (0xffffffff);
-      if (errno == 0 && !(personality_orig & ADDR_NO_RANDOMIZE))
-       {
-         personality_set = 1;
-         personality (personality_orig | ADDR_NO_RANDOMIZE);
-       }
-      if (errno != 0 || (personality_set
-                        && !(personality (0xffffffff) & ADDR_NO_RANDOMIZE)))
-       warning (_("Error disabling address space randomization: %s"),
-                safe_strerror (errno));
-    }
-#endif /* HAVE_PERSONALITY */
-
   /* Make sure we report all signals during startup.  */
   linux_nat_pass_signals (ops, 0, NULL);
 
   linux_ops->to_create_inferior (ops, exec_file, allargs, env, from_tty);
 
-#ifdef HAVE_PERSONALITY
-  if (personality_set)
-    {
-      errno = 0;
-      personality (personality_orig);
-      if (errno != 0)
-       warning (_("Error restoring address space randomization: %s"),
-                safe_strerror (errno));
-    }
-#endif /* HAVE_PERSONALITY */
+  do_cleanups (restore_personality);
 }
 
 /* Callback for linux_proc_attach_tgid_threads.  Attach to PTID if not
diff --git a/gdb/nat/linux-personality.c b/gdb/nat/linux-personality.c
new file mode 100644 (file)
index 0000000..f61a2c6
--- /dev/null
@@ -0,0 +1,94 @@
+/* Disable address space randomization based on inferior personality.
+
+   Copyright (C) 2008-2015 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+#include "common-defs.h"
+#include "nat/linux-personality.h"
+
+#ifdef HAVE_PERSONALITY
+# include <sys/personality.h>
+# ifndef HAVE_DECL_ADDR_NO_RANDOMIZE
+#  define ADDR_NO_RANDOMIZE 0x0040000
+# endif /* ! HAVE_DECL_ADDR_NO_RANDOMIZE */
+#endif /* HAVE_PERSONALITY */
+
+#ifdef HAVE_PERSONALITY
+
+/* Restore address space randomization of the inferior.  ARG is the
+   original inferior's personality value before the address space
+   randomization was disabled.  */
+
+static void
+restore_personality (void *arg)
+{
+  int personality_orig = (int) (uintptr_t) arg;
+
+  errno = 0;
+  personality (personality_orig);
+  if (errno != 0)
+    warning (_("Error restoring address space randomization: %s"),
+            safe_strerror (errno));
+}
+#endif /* HAVE_PERSONALITY */
+
+/* Return a cleanup responsible for restoring the inferior's
+   personality (and restoring the inferior's address space
+   randomization) if HAVE_PERSONALITY and if PERSONALITY_SET is not
+   zero; return a null cleanup if not HAVE_PERSONALITY or if
+   PERSONALITY_SET is zero.  */
+
+static struct cleanup *
+make_disable_asr_cleanup (int personality_set, int personality_orig)
+{
+#ifdef HAVE_PERSONALITY
+  if (personality_set != 0)
+    return make_cleanup (restore_personality,
+                        (void *) (uintptr_t) personality_orig);
+#endif /* HAVE_PERSONALITY */
+
+  return make_cleanup (null_cleanup, NULL);
+}
+
+/* See comment on nat/linux-personality.h.  */
+
+struct cleanup *
+maybe_disable_address_space_randomization (int disable_randomization)
+{
+  int personality_orig = 0;
+  int personality_set = 0;
+
+#ifdef HAVE_PERSONALITY
+  if (disable_randomization)
+    {
+      errno = 0;
+      personality_orig = personality (0xffffffff);
+      if (errno == 0 && !(personality_orig & ADDR_NO_RANDOMIZE))
+       {
+         personality_set = 1;
+         personality (personality_orig | ADDR_NO_RANDOMIZE);
+       }
+      if (errno != 0 || (personality_set
+                        && !(personality (0xffffffff) & ADDR_NO_RANDOMIZE)))
+       warning (_("Error disabling address space randomization: %s"),
+                safe_strerror (errno));
+    }
+#endif /* HAVE_PERSONALITY */
+
+  return make_disable_asr_cleanup (personality_set,
+                                  personality_orig);
+}
diff --git a/gdb/nat/linux-personality.h b/gdb/nat/linux-personality.h
new file mode 100644 (file)
index 0000000..009e614
--- /dev/null
@@ -0,0 +1,31 @@
+/* Disable address space randomization based on inferior personality.
+
+   Copyright (C) 2008-2015 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+#ifndef NAT_LINUX_PERSONALITY_H
+#define NAT_LINUX_PERSONALITY_H
+
+/* Disable the inferior's address space randomization if
+   DISABLE_RANDOMIZATION is not zero and if we have
+   <sys/personality.h>.  Return a cleanup which, when called, will
+   re-enable the inferior's address space randomization.  */
+
+extern struct cleanup *maybe_disable_address_space_randomization
+  (int disable_randomization);
+
+#endif /* ! NAT_LINUX_PERSONALITY_H */