Seems hackish

Author: brenoguim

src/patchelf.cc

@@ -960,7 +960,9 @@ void ElfFile<ElfFileParamNames>::rewriteSectionsExecutable()
 
         /* Calculate how many bytes are needed out of the additional pages. */
         size_t extraSpace = neededSpace - startOffset; 
-        unsigned int neededPages = roundUp(extraSpace, getPageSize()) / getPageSize();
+        // Always give one extra page to avoid colliding with segments that start at
+        // unaligned addresses and will be rounded down when loaded
+        unsigned int neededPages = 1 + roundUp(extraSpace, getPageSize()) / getPageSize();
         debug("needed pages is %d\n", neededPages);
         if (neededPages * getPageSize() > firstPage)
             error("virtual address space underrun!");

tests/Makefile.am

@@ -45,7 +45,9 @@ src_TESTS = \
   add-debug-tag.sh \
   empty-note.sh \
   print-execstack.sh \
-  modify-execstack.sh
+  modify-execstack.sh \
+  overlapping-segments-after-rounding.sh \
+  empty-note.sh
 
 build_TESTS = \
   $(no_rpath_arch_TESTS)

tests/overlapping-segments-after-rounding.sh

@@ -0,0 +1,22 @@
+#! /bin/sh -e
+
+PATCHELF=$(readlink -f "../src/patchelf")
+SCRATCH="scratch/$(basename "$0" .sh)"
+READELF=${READELF:-readelf}
+
+EXEC_NAME="overlapping-segments-after-rounding"
+
+rm -rf "${SCRATCH}"
+mkdir -p "${SCRATCH}"
+
+cp "${EXEC_NAME}.xz" ${SCRATCH}/
+cd ${SCRATCH}
+xz --decompress "${EXEC_NAME}.xz"
+
+${PATCHELF} --force-rpath --remove-rpath --output modified1 "${EXEC_NAME}"
+
+ldd modified1
+
+${PATCHELF} --force-rpath --set-rpath "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx" --output modified2 modified1
+
+ldd modified2