Bug 1417467 - P1. 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 21:49:50 +0100
changeset 698479 a1cca977cf1e3418ae662a75e928d316fd6827b6
parent 698478 0e459f72022fc5ba731c6e4be6511a5b7f30abaa
child 698480 5e4094ca94999236a26d9a7cb67d057cfff3f813
push id89303
push userbmo:jyavenard@mozilla.com
push dateWed, 15 Nov 2017 21:07:49 +0000
reviewersmattwoodrow
bugs1417467
milestone57.0.1
Bug 1417467 - P1. 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
@@ -1258,17 +1258,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;