diff options
| author | jimblandy <jimblandy@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-02-18 07:57:53 +0000 |
|---|---|---|
| committer | jimblandy <jimblandy@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-02-18 07:57:53 +0000 |
| commit | 6de1b75da4d9fb29eb8e00f41beab3cbc539874f (patch) | |
| tree | f56d9ce695d3daec15c8d82057abf9de9c3451e2 /src/common/mac/file_id.cc | |
| parent | Breakpad DWARF parser: Expand comments for ByteReader class. (diff) | |
| download | breakpad-6de1b75da4d9fb29eb8e00f41beab3cbc539874f.tar.xz | |
Breakpad Linux dumper: Compare section names correctly.
FindSectionByName will return the first section whose name starts with
NAME, because strncmp stops the comparison once NAME's characters have
been found to match. The comparison stops before the terminating '\0'.
For example, if we search for the section named ".eh_frame", we may
get the section named ".eh_frame_hdr".
Instead, check that the section name section has enough space to store
the complete name with its terminating '\0', and then use strcmp,
which will never examine more than strlen(NAME) + 1 bytes from the
section name section, regardless of its contents, and will require the
terminating '\0' to match as well.
a=jimblandy, r=nealsid
git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@525 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/common/mac/file_id.cc')
0 files changed, 0 insertions, 0 deletions