Bug 1417442 - P3. Don't enable nv12 surfaces if we failed to parse the version string. r?mattwoodrow draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Wed, 15 Nov 2017 23:33:37 +0100
changeset 699104 994983fae251bd111a31a0de7efc8830a91c4777
parent 699103 d9f8808cd4ef6706c947e133d7aece5c371b55f8
child 699105 83ccc24b4d86eee4ee3289473c04924fb7a35604
push id89468
push userbmo:jyavenard@mozilla.com
push dateThu, 16 Nov 2017 17:28:05 +0000
reviewersmattwoodrow
bugs1417442
milestone59.0a1
Bug 1417442 - P3. Don't enable nv12 surfaces if we failed to parse the version string. r?mattwoodrow MozReview-Commit-ID: 1MKtCOO1gcS
dom/media/platforms/wmf/DXVA2Manager.cpp
--- a/dom/media/platforms/wmf/DXVA2Manager.cpp
+++ b/dom/media/platforms/wmf/DXVA2Manager.cpp
@@ -1260,17 +1260,17 @@ DXVA2Manager::IsNV12Supported(uint32_t a
     for (const auto& model : sAMDPreUVD4) {
       if (aDeviceID == model) {
         return false;
       }
     }
     // AMD driver earlier than 21.19.411.0 have bugs in their handling of NV12
     // surfaces.
     uint64_t driverVersion;
-    if (widget::ParseDriverVersion(aDriverVersionString, &driverVersion) &&
+    if (!widget::ParseDriverVersion(aDriverVersionString, &driverVersion) ||
         driverVersion < widget::V(21, 19, 411, 0)) {
       return false;
     }
   } else if (aVendorID == 0x10DE) {
     // NVidia
     for (const auto& model : sNVIDIABrokenNV12) {
       if (aDeviceID == model) {
         return false;