1767894 - Intermittent comm/mail/components/addrbook/test/browser/browser_mailing_lists.js | application crashed [@ XPCConvert::NativeData2JS(JSContext*, JS::MutableHandle<JS::Value>, void const*, nsXPTType const&, nsID const*, unsigned int, nsresult*)]

Status: RESOLVED FIXED

(Thunderbird :: Address Book, defect)

Description

[Treeherder Bug Filer]

Filed by: geoff [at] darktrojan.net
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=376766954&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/X2afknhqTdCJtxV0Y6dt7w/runs/0/artifacts/public/logs/live_backing.log

---

Comment 1

[Geoff Lankow (:darktrojan)]

Also appears on TreeHerder as mail/components/addrbook/test/browser/new/browser_search.js.

I can reproduce this by starting with browser_edit_card.js and it will crash at browser_search.js but if I remove any of the tests in between (except browser_edit_photo.js) it stops happening.

Comment 2

[Geoff Lankow (:darktrojan)]

Regression range is:

• https://hg.mozilla.org/comm-central/pushloghtml?fromchange=a3eba558343fcd7890c26a59f00a850d3a035972&tochange=749de9101c761732db07f9f9c0e14e1165f1e257 (very unlikely IMO)
• https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ff375ced41f5e49d7a30ee1f59dcf7f52b7b469c&tochange=7fafa8a52cb0c24a35e224f9929ffe4561c06a03

Comment 3

[Geoff Lankow (:darktrojan)]

Nika, it seems these 16 patches from bug 1754004 are the cause. I can't see why they would, but I've narrowed the regression range to include only them so far.

Comment 4

[Geoff Lankow (:darktrojan)]

hg bisect took me to https://hg.mozilla.org/mozilla-central/rev/017e1552d5493438ee4be08aba0248672f33060f

Comment 5

[Andrew Sutherland [:asuth] (he/him)]

The relevant portion of the crash stack is:

• libxul.so!XPCConvert::NativeData2JS(JSContext*, JS::MutableHandle<JS::Value>, void const*, nsXPTType const&, nsID const*, unsigned int, nsresult*) [XPCConvert.cpp:7fafa8a52cb0c24a35e224f9929ffe4561c06a03 : 352 + 0x1e]
• libxul.so!nsXPCWrappedJS::CallMethod(unsigned short, nsXPTMethodInfo const*, nsXPTCMiniVariant*) [XPCWrappedJSClass.cpp:7fafa8a52cb0c24a35e224f9929ffe4561c06a03 : 933 + 0x28]
• libxul.so!PrepareAndDispatch [xptcstubs_x86_64_linux.cpp:7fafa8a52cb0c24a35e224f9929ffe4561c06a03 : 115 + 0x16]
• libxul.so!SharedStub + 0x5a
• libxul.so!nsInputStreamReadyEvent::Run() [nsStreamUtils.cpp:7fafa8a52cb0c24a35e224f9929ffe4561c06a03 : 96 + 0x5]

The test is: https://searchfox.org/comm-central/source/mail/components/addrbook/test/browser/browser_mailing_lists.js

There's a few onInputStreamReady handlers in mail/mailnews.

Comment 6

[Andrew Sutherland [:asuth] (he/him)]

(In reply to Geoff Lankow (:darktrojan) from comment #1)

Also appears on TreeHerder as mail/components/addrbook/test/browser/new/browser_search.js.

I can reproduce this by starting with browser_edit_card.js and it will crash at browser_search.js but if I remove any of the tests in between (except browser_edit_photo.js) it stops happening.

Given that this might involve multiple tests and they're addressbook based, maybe it could be something like https://searchfox.org/comm-central/source/mail/components/addrbook/test/browser/new/browser_ldap_search.js using LDAP which involves the LDAP server's onInputStreamReady.

Comment 7

[Geoff Lankow (:darktrojan)]

I suspect it's browser_ldap_search.js + some time, maybe garbage collection or something is involved?

Comment 8

[Geoff Lankow (:darktrojan)]

Yes, if I add Cu.forceGC() to the end of browser_ldap_search.js, it crashes.

Comment 9

[Geoff Lankow (:darktrojan)]

I tried this with the latest version of bug 1754004 and it's fixed.

Comment 10

[Intermittent Failures Robot]

1 failures in 3540 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 1

Platform and build breakdown:

• linux1804-64-qr: 1
  • opt: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1767894&startday=2022-05-02&endday=2022-05-08&tree=all