Bug 1417442 - P3. Don't enable nv12 surfaces if we failed to parse the version string. r?mattwoodrow
MozReview-Commit-ID: 1MKtCOO1gcS
--- 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;