This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 2/2 Take-2][BZ #12416] Use stack boundaries from/proc/PID/maps to make stack executable


On Fri,  4 May 2012 16:08:45 -0700 (PDT), Roland wrote:
> > +  old_stack_addr += stack_size;
> 
> That looks like it's assuming the stack grows down.  I think your
> actual intent is just to verify that both the address and size were
> the same in both calls.  But you're not doing that, you're only
> verifying the sum. Why not just save both values and check them both
> below?
> 
> If your intent is something subtler than that, then it needs comments.

Thanks for the review. My intent is to check if the bottom of stack
changes, since it is possible that the stack vma may have grown between
the two calls and hence the top of stack (which is computed and
returned as bottom - size) may have changed.

Thanks,
Siddhesh


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]