Closed Bug 841634 Opened 7 years ago Closed 7 years ago
Race condition in Content
Parent: m Child ID should be set in the constructor
Right now mChildID is set to a unique value only during RecvGetProcessAttributes. This sets us up for a race condition where we pass -1 to the wake lock manager and acquire a wake lock on behalf of the main process instead of on behalf of the child. Patch in a moment.
Racey race race...
Attachment #714208 - Flags: review?(jones.chris.g)
Comment on attachment 714208 [details] [diff] [review] Patch, v1 (Whew, I'm not in the blame for this one! ;) ) Nice find.
Attachment #714208 - Flags: review?(jones.chris.g) → review+
Indeed, I reviewed the change. :)
Assignee: nobody → justin.lebar+bug
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Can you please provide steps to verify this fix - as we will blackbox test from the UI?
(In reply to croesch from comment #7) > Can you please provide steps to verify this fix - as we will blackbox test > from the UI? Not possible.
No need to create a TC in Moztrap for this defect.
You need to log in before you can comment on or make changes to this bug.