Closed Bug 1202484 Opened 5 years ago Closed 5 years ago

Implement backport fix for 'lock-immeditaly' bug 1186100 as FMD hotfix


(Firefox OS Graveyard :: FindMyDevice, defect)

Gonk (Firefox OS)
Not set


(b2g-v2.0 affected, b2g-v2.1 affected, b2g-v2.2 fixed, b2g-v2.2r fixed, b2g-master unaffected)

FxOS-S7 (18Sep)
Tracking Status
b2g-v2.0 --- affected
b2g-v2.1 --- affected
b2g-v2.2 --- fixed
b2g-v2.2r --- fixed
b2g-master --- unaffected


(Reporter: cr, Assigned: cr)



(Keywords: sec-other)


(1 file)

The current patch for bug 1186100 works with the new lockscreen state manager which is not available in earlier version.

To keep things simple, the backport can be implemented as hotfix in Find My Device instead: When it requests to lock immediately, it would need to set the pass code timeout setting to 'immediately' before signalling 'lock-immediately'.
Assignee: nobody → cr
Blocks: 1186100
Keywords: sec-other
Comment on attachment 8657929 [details] [review]
[gaia] cr:immediatelockfix-v2.2 > mozilla-b2g:v2.2

Alexandre, could you review this trivial hotfix, please? STR in bug 1186100.
Attachment #8657929 - Flags: review?(lissyx+mozillians)
Landing this depends on bug 1195872 being fixed.
Depends on: 1195872
Attachment #8657929 - Flags: review?(lissyx+mozillians) → review+
Comment on attachment 8657929 [details] [review]
[gaia] cr:immediatelockfix-v2.2 > mozilla-b2g:v2.2

[Approval Request Comment]

[Bug caused by] (feature/regressing bug #):
Design flaw. Hotfix for bug 1186100.

[User impact] if declined:
FindMyDevice lock command will not passcode-lock immediately when the user had configured a long timeout. Thief can continue to use the device until timeout occurs.

[Testing completed]:
Local tests passing.

[Risk to taking this patch] (and alternatives if risky):
Low risk. One-liner change in FMD which forced the pass code timeout to 'immediately'.

[String changes made]:

Please also give v2.2r approval.
Attachment #8657929 - Flags: approval-gaia-v2.2?(mpotharaju)
Hey CR, is the dependent bug not required before we land this?
Flags: needinfo?(cr)
(In reply to Mahendra Potharaju [:mahe] from comment #5)
> Hey CR, is the dependent bug not required before we land this?

This patch isn't technically dependent on 1195872 which is about broken test automation on all non-master branches. Meanwhile sheriffs have been landing a few other in the blind on v2.2. I marked my 2.2 patches as dependent just to raise awareness.
Flags: needinfo?(cr)
Thank you Christiane.
Comment on attachment 8657929 [details] [review]
[gaia] cr:immediatelockfix-v2.2 > mozilla-b2g:v2.2

Approved to apply patch on v2.2. 

Attachment #8657929 - Flags: approval-gaia-v2.2?(mpotharaju) → approval-gaia-v2.2+
Keywords: checkin-needed
Keywords: checkin-needed
Target Milestone: --- → FxOS-S7 (18Sep)
You need to log in before you can comment on or make changes to this bug.