826872 - OOM crash while syncing cjones's calendar

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Calendar, defect, P2)

Description

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

I wish there were a better way to share STR, but this is all I've got.  gal and I share calendars so it's also syncing gal's calendar.

bb? -> radar

Comment 1

[James Lal [:lightsofapollo]]

Is there someway to export the ical document for your calendar(s) ?

Comment 2

[Jason Smith [:jsmith]]

We're going to need some way to debug this if it's reproducible.

Are you guys both going to the work week? If so, we could quickly meet up and debug this as another option.

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

I'm fine with sharing my calendar but the export will include gal's too, so I need a thumbs-up from him before sharing privately.

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Nice try! ;)

Comment 5

[David Scravaglieri [:scravag]]

Chris, James, could you try to see this together this week ?

Comment 6

[David Scravaglieri [:scravag]]

Just reproduced So blocking.

Comment 7

[James Lal [:lightsofapollo]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/7313

Pointer to Github pull-request

Comment 8

[James Lal [:lightsofapollo]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/7313

Pointer to Github pull-request

Comment 9

[James Lal [:lightsofapollo]]

I have a simple patch which makes it possible to sync his calendar but I was still able to crash it sometimes. 819628 should improve this further.

Comment 10

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

(In reply to David Scravaglieri [:scravag] from comment #5)
> Chris, James, could you try to see this together this week ?

We've been communicating out of band, but the crash reproduces pretty quickly 100% of the time.

Comment 11

[Jason Smith [:jsmith]]

(In reply to Chris Jones [:cjones] [:warhammer] from comment #10)
> (In reply to David Scravaglieri [:scravag] from comment #5)
> > Chris, James, could you try to see this together this week ?
> 
> We've been communicating out of band, but the crash reproduces pretty
> quickly 100% of the time.

I confirmed Faramarz's calendars (which happens to include Andreas's calendar) is causing this OOM crash repeatedly as well. Definitely reproducible.

Comment 12

[James Lal [:lightsofapollo]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/7419

Pointer to Github pull-request

Comment 13

[James Lal [:lightsofapollo]]

Note: with Andreas's calendar I see an ical parse error that is localized to his own calendar and is probably is either invalid ical or a unrelated parser bug.

With this patch I could sync Faramarz's calendar but it would show the pending bar forever until I rebooted the calendar app and re-synced (which seems to work around the parse issues).

Comment 14

[James Lal [:lightsofapollo]]

https://github.com/mozilla-b2g/gaia/commit/f10ca860f261f1072092c7638eafdb03384c3101

Comment 15

[amiller]

Device: Unagi
Build: 20130113070202.
Does not repro.