Bug 1417442 - P3. Don't enable nv12 surfaces if we failed to parse the version string. r=mattwoodrow, a=sledru
authorJean-Yves Avenard <jyavenard@mozilla.com>
Wed, 15 Nov 2017 23:33:37 +0100
changeset 435369 c971c6beda8d
parent 435368 c40c1a52ec1b
child 435370 7e3aa9b5cf6f
push id1588
push usersledru@mozilla.com
push dateFri, 24 Nov 2017 07:58:32 +0000
treeherdermozilla-release@c971c6beda8d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow, sledru
bugs1417442
milestone57.0.1
Bug 1417442 - P3. Don't enable nv12 surfaces if we failed to parse the version string. r=mattwoodrow, a=sledru 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;