Closed Bug 641378 Opened 9 years ago Closed 9 years ago

Crash if I import bookmarks backup in offline mode [@ nsXPCWrappedJS::`scalar deleting destructor''(unsigned int) ]

Categories

(Firefox :: Bookmarks & History, defect, critical)

x86
Windows 7
defect
Not set
critical

Tracking

()

VERIFIED FIXED

People

(Reporter: stupidtrashmails, Assigned: mrbkap)

Details

(Keywords: crash, reproducible)

Crash Data

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0

Firefox crashes if I try to import bookmarks from an json file while disconnected from the internet

Reproducible: Always

Steps to Reproduce:
1.make a json backup file of bookmarks with the bookmarks manager
2.disconnect from internet
3.import the backup file
Actual Results:  
firefox crashes


Also tried this in safe mode without any addons
Works for me on:
Mozilla/5.0 (Windows NT 6.1; rv:2.0b13pre) Gecko/20110313 Firefox/4.0b13pre

*Note: Can you provide more clear steps. After disconnecting the internet, do you run a new profile (without history and bookmarks)?
No it crashed all the times with my old profile.

I think I now found out that it has something to do with Firefox Sync.

I created a new profile, and the error was gone, no crash anymore.

But then I created a new Sync Account, repeated the steps above and it causes Firefox to crash again.

Hope this was helpful.
did you get the crash reporter dialog and submitted a crash report? Can you please give us the crash id? (you can get it by visiting about:crashes with that profile)
Reproducible on:
Mozilla/5.0 (Windows NT 6.1; rv:2.0b13pre) Gecko/20110313 Firefox/4.0b13pre

*Note: I could reproduce the issue. I got the following crash report:
ID: baa4669c-d476-434a-9557-077be2110314
Able to reproduce using Mozilla/5.0 (Windows NT 6.1; rv:2.0b13pre) Gecko/20110313 Firefox/4.0b13pre.
Setting resolution to NEW.
Status: UNCONFIRMED → NEW
Ever confirmed: true
The stack reported in Comment 4 is the same stack as in Bug 592998.
Summary: Crash if I import bookmarks backup in offline mode → Crash if I import bookmarks backup in offline mode [@ nsXPCWrappedJS::`scalar deleting destructor''(unsigned int) ]
Keywords: crash
Keywords: reproducible
Version: unspecified → Trunk
Attached patch patch v1 (obsolete) — Splinter Review
I don't know how this ever worked. I'm also not entirely sure why this doesn't always crash, but we don't hit this codepath very often. In short, the lesson here is that an nsXPCWrappedJS is not its underlying XPTCStub and calling virtual methods on the wrapped JS doesn't forward correctly.

Peter, another way of fixing this would be to call QI on the returned nsXPCWrappedJS, but I think that's equivalent to this approach. Does it look right to you as well?
Assignee: nobody → mrbkap
Status: NEW → ASSIGNED
Attachment #522028 - Flags: review?(peterv)
Attached patch Real patchSplinter Review
Sorry for the spam.

By the way, it seems to me like someone should file a bug on sync throwing exceptions all over the place (and generally not gracefully falling back) when the browser is in offline mode. But that's separate from this.
Attachment #522028 - Attachment is obsolete: true
Attachment #522028 - Flags: review?(peterv)
Attachment #522030 - Flags: review?(peterv)
Comment on attachment 522030 [details] [diff] [review]
Real patch

In the testcase I'd check exception.tries for non-zero.
Attachment #522030 - Flags: review?(peterv) → review+
http://hg.mozilla.org/mozilla-central/rev/a089731c47d5
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Verified fixed on Mozilla/5.0 (Windows NT 6.1; rv:2.2a1pre) Gecko/20110411 Firefox/4.2a1pre
Status: RESOLVED → VERIFIED
Crash Signature: [@ nsXPCWrappedJS::`scalar deleting destructor''(unsigned int) ]
You need to log in before you can comment on or make changes to this bug.