commit | 0cf76ed0b54f4b64d3a0575a1c85ef8617d4278d | [log] [tgz] |
---|---|---|
author | Paul Pluzhnikov <ppluzhnikov@google.com> | Tue Dec 01 13:59:45 2009 -0800 |
committer | Arun Sharma <aruns@google.com> | Tue Dec 01 13:59:45 2009 -0800 |
tree | 52ec6fd9a574a9d9964e92794713515ceb1605b9 | |
parent | 9626d66019599de925b9ad1f216388d1c890d193 [diff] |
Check for NULL when validating addresses This is rather on the obvious side. While doing strace on an executable using libunwind, I noticed a lot of: msync(0, 1, MS_SYNC) = -1 ENOMEM (Cannot allocate memory) Since we know that the first page isn't mapped (or at least doesn't contain the data we are looking for), we can eliminate all such msync calls. Tested on Linux/x86_64 with no regressions.