wukenaihe / google-breakpad

Automatically exported from code.google.com/p/google-breakpad
0 stars 0 forks source link

Small fix for Solaris handler #200

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
The Solaris client has some little faults while making the dump file,
1. The module count is different with actual number written to the dump,
for some modules without a name was omitted when writing.
2. The thread list stream may not write to the dump file because of a
problem with the return value.
3. Wrongly use the process id as thread id in one place.  
These could cause error when processing the dump file.

What version of the product are you using? On what operating system?
rev 198

Please provide any additional information below.
BTW, I really think the '}' at minidump_processor.cc:835 should be at 734
to close the switch for WIN32.
Is that right?

Original issue reported on code.google.com by satisfy...@gmail.com on 20 Aug 2007 at 3:11

Attachments:

GoogleCodeExporter commented 9 years ago
Checked in with issue #201 using the consolidated patch there, trunk revision 
[215].

Original comment by mmento...@gmail.com on 26 Sep 2007 at 6:28