commit | 7d2355af35d7c397f53e3913602c170a57fa07c6 | [log] [tgz] |
---|---|---|
author | etienneb <etienneb@chromium.org> | Thu Jul 26 18:18:54 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Jul 26 18:18:54 2018 |
tree | 266bb7ee8b8f8603c8ea6f05c6feb0a9f195c42a | |
parent | b76242f6845736e4fd77834ed5e752ed721481fa [diff] |
Fix potential missing nul character on resolved symbol names The symbol name returned by SymFromName may not contains a NUL character when the symbol name is exactly the size of the buffer. It believe this may also happen when the symbol name is too long and truncated. The original code is based on: https://msdn.microsoft.com/en-us/library/windows/desktop/ms680580(v=vs.85).aspx A right implementation can be found here: https://cs.chromium.org/chromium/src/base/debug/stack_trace_win.cc?l=145&rcl=f4ecb9e37e9e2d59e32b8b96f23ac4a1e33b9552 As described here: https://msdn.microsoft.com/en-us/library/windows/desktop/ms680686(v=vs.85).aspx NameLen The length of the name, in characters, not including the null-terminating character. MaxNameLen The size of the Name buffer, in characters. If this member is 0, the Name member is not used. This issue was causing the catapult symbolisation script to encode incorrect (random) characters into the symbol names. See the example in the bug. Original BUG=713741 Review-Url: https://codereview.chromium.org/2832643004 Cr-Commit-Position: refs/heads/master@{#466098} BUG=724399,b:70905156 Change-Id: Ia6de4ddbd5c66a966b74ecec3fda5e853f36073b Reviewed-on: https://chromium-review.googlesource.com/1147669 Reviewed-by: Will Harris <wfh@chromium.org> Commit-Queue: Gabriel Marin <gmx@chromium.org> Cr-Commit-Position: refs/heads/master@{#578373}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .