Closed Bug 1380201 Opened 7 years ago Closed 7 years ago

[Fennec][HLS] Make access to mIsPlayerInitDone & mResourceCallback guarded.

Categories

(Firefox for Android Graveyard :: Audio/Video, enhancement, P3)

enhancement

Tracking

(firefox56 fixed)

RESOLVED FIXED
Firefox 56
Tracking Status
firefox56 --- fixed

People

(Reporter: kikuo, Assigned: kikuo)

References

Details

Attachments

(1 file)

From https://scan4.coverity.com/reports.htm#v27748/p11134/fileInstanceId=22191825&defectInstanceId=4435480&mergedDefectId=147724, there are 4 unguarded read/write defects (CID : 147724 / 147725 / 147726 / 147727). It won't lead to an unexpected result, It's better to make them safe.
Priority: -- → P3
Comment on attachment 8885596 [details] Bug 1380201 - Make access to some members in GeckoHlsPlayer guarded. https://reviewboard.mozilla.org/r/156448/#review161566
Attachment #8885596 - Flags: review?(jacheng) → review+
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again. hg error in cmd: hg rebase -s 34c4ded4c997 -d 3b419fb92d30: rebasing 406756:34c4ded4c997 "Bug 1380201 - Make access to some members in GeckoHlsPlayer guarded. r=JamesCheng" (tip) merging mobile/android/geckoview/src/main/java/org/mozilla/gecko/media/BaseHlsPlayer.java merging mobile/android/geckoview/src/main/java/org/mozilla/gecko/media/GeckoHlsPlayer.java warning: conflicts while merging mobile/android/geckoview/src/main/java/org/mozilla/gecko/media/GeckoHlsPlayer.java! (edit, then use 'hg resolve --mark') unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment on attachment 8885596 [details] Bug 1380201 - Make access to some members in GeckoHlsPlayer guarded. https://reviewboard.mozilla.org/r/156448/#review161660 rebase
Pushed by kikuo@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/835d645c14f0 Make access to some members in GeckoHlsPlayer guarded. r=JamesCheng
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: