crash [@ nsAbOSXDirectory::AssertCard]

RESOLVED FIXED in Thunderbird 8.0

Status

defect
--
critical
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: wsmwk, Assigned: mconley)

Tracking

({crash})

Trunk
Thunderbird 8.0
x86_64
macOS

Thunderbird Tracking Flags

(thunderbird7 fixed)

Details

(crash signature)

Attachments

(1 attachment)

Reporter

Description

8 years ago
crash [@ nsAbOSXDirectory::AssertCard]
#82 crash for TB5

none of the crash comments say they were using anything address book related. 
https://crash-stats.mozilla.com/report/list?product=Thunderbird&query_search=signature&query_type=exact&query=nsAbOSXDirectory%3A%3AAssertCard&reason_type=contains&date=08%2F07%2F2011%2007%3A51%3A55&range_value=4&range_unit=weeks&hang_type=any&process_type=all&do_query=1&signature=nsAbOSXDirectory%3A%3AAssertCard

bp-ecf57b53-5df2-4100-9b97-b7e7e2110712 (kk7)
EXC_BAD_ACCESS / KERN_INVALID_ADDRESS
0x0
0	XUL	nsAbOSXDirectory::AssertCard	mailnews/addrbook/src/nsAbOSXDirectory.mm:755
1	XUL	-[ABChangedMonitor ABChanged:]	mailnews/addrbook/src/nsAbOSXDirectory.mm:177
2	Foundation	Foundation@0x10bc4	
3	CoreFoundation	CoreFoundation@0xe8cf	
4	CoreFoundation	CoreFoundation@0x58fff	
5	libobjc.A.dylib	libobjc.A.dylib@0x79a0	
6	Foundation	Foundation@0x289eb	
7	CoreFoundation	CoreFoundation@0x45577	
8	libobjc.A.dylib	libobjc.A.dylib@0x65f2	
9	libobjc.A.dylib	libobjc.A.dylib@0x7bdc	

bp-nsAbOSXDirectory::AssertCard (gmoses)
I think I see the problem.  For TB <= 6.0, it looks like we're not checking the return value of ConvertToCard, so even if it fails, we go ahead and try to manipulate a NULL pointer:  http://hg.mozilla.org/releases/comm-miramar/diff/e4f4569d451a/mailnews/addrbook/src/nsAbOSXDirectory.mm

Similarly, in TB 7.0, while we capture the return value of CreateCard, we're not actually checking it for success or failure:  http://mxr.mozilla.org/comm-central/source/mailnews/addrbook/src/nsAbOSXDirectory.mm#221

Patch en route.
Assignee: nobody → mconley
Whoops - quick correction for that last comment.  Should have been:

I think I see the problem.  For TB <= 5.0, it looks like we're not checking the return value of ConvertToCard, so even if it fails, we go ahead and try to manipulate a NULL pointer:  http://hg.mozilla.org/releases/comm-miramar/diff/e4f4569d451a/mailnews/addrbook/src/nsAbOSXDirectory.mm

Similarly, in TB 6.0+, while we capture the return value of CreateCard, we're not actually checking it for success or failure:  http://mxr.mozilla.org/comm-central/source/mailnews/addrbook/src/nsAbOSXDirectory.mm#221

Patch en route.
Ack, disregard that last comment.  I had it right the first time.
Assignee

Updated

8 years ago
Attachment #551451 - Flags: review?(mbanner)
Attachment #551451 - Flags: approval-comm-aurora?
Comment on attachment 551451 [details] [diff] [review]
Patch v1

Ok, lets try it. I'm a bit concerned that something bad is happening here, and we're going to miss logging it, but I guess we'll have to see if we get users reporting any weird issues.
Attachment #551451 - Flags: review?(mbanner)
Attachment #551451 - Flags: review+
Attachment #551451 - Flags: approval-comm-aurora?
Attachment #551451 - Flags: approval-comm-aurora+
Assignee

Updated

8 years ago
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/bbf265b99212
Status: NEW → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 8.0
You need to log in before you can comment on or make changes to this bug.