Backed out changeset 192cc007fef6 (bug 1742864) for xpcshell failure on test_dmd.js

mozglue/misc/StackWalk.cpp

@@ -876,20 +876,6 @@ static void DoFramePointerStackWalk(MozWalkStackCallback aCallback,
   FrameSkipper skipper(aFirstFramePC);
   uint32_t numFrames = 0;
 
-  // Sanitize the given aBp. Assume that something reasonably close to
-  // but before the stack end is going be a valid frame pointer. Also
-  // check that it is an aligned address. This increases the chances
-  // that if the pointer is not valid (which might happen if the caller
-  // called __builtin_frame_address(1) and its frame is busted for some
-  // reason), we won't read it, leading to a crash. Because the calling
-  // code is not using frame pointers when returning, it might actually
-  // recover just fine.
-  static const uintptr_t kMaxStackSize = 8 * 1024 * 1024;
-  if (uintptr_t(aBp) < uintptr_t(aStackEnd) - kMaxStackSize ||
-      aBp >= aStackEnd || (uintptr_t(aBp) & 3)) {
-    return;
-  }
-
   while (aBp) {
     void** next = (void**)*aBp;
     // aBp may not be a frame pointer on i386 if code was compiled with