751957 - Leak in dom/system/gonk/RadioInterfaceLayer.js

Status: RESOLVED INVALID

(Firefox OS Graveyard :: RIL, defect)

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

After launching b2g desktop I got:
 nsStringStats
 => mAllocCount:          65643
 => mReallocCount:         2428
 => mFreeCount:           65636  --  LEAKED 7 !!!
 => mShareCount:          23318
 => mAdoptCount:           2036
 => mAdoptFreeCount:       2036

After a basic investigation it seems like this is related to RadioInterfaceLayer.js

Comment 1

[Philipp von Weitershausen [:philikon]]

(In reply to Vivien Nicolas (:vingtetun) from comment #0)
> After a basic investigation it seems like this is related to
> RadioInterfaceLayer.js

How did you arrive at that conlusion?

I'll take a look to see if there's anything obvious that we're not cleaning up (e.g. some service reference.)

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Philipp von Weitershausen [:philikon] from comment #1)
> (In reply to Vivien Nicolas (:vingtetun) from comment #0)
> > After a basic investigation it seems like this is related to
> > RadioInterfaceLayer.js
> 
> How did you arrive at that conlusion?

Simply by commenting some code in RadioInterfaceLayer.js to prevent it to work. I have not found any obvious cause btw.

Comment 3

[Dietrich Ayala (:dietrich)]

RIL is probably holding a reference to some XPCOM service. However, RIL stays resident for the lifetime of a device being live, so this shouldn't be a problem on phones. Not going to block on this for now.

Comment 4

[Ken Chang[:kenkai]]

Move it to backlog.

Comment 5

[Hsin-Yi Tsai (she/her) [:hsinyi]]

since Bug 920551 - B2G RIL: allow building B2G without all RIL functions,  there isn't RIL running on b2g-desktop. Close it as invalid.