From 97939ceae864aef2f50e117496f548555877b4ea Mon Sep 17 00:00:00 2001 From: Xaver Hugl Date: Thu, 4 Aug 2022 15:36:26 +0200 Subject: [PATCH] backends/drm: fix blob updating Two things about the code were wrong: 1. m_current is used in updateBlob() but was only updated afterwards 2. the assumption that the property having the same ID means it has the same blob contents is not always true BUG: 449285 --- src/backends/drm/drm_property.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/backends/drm/drm_property.cpp b/src/backends/drm/drm_property.cpp index 42d4bafc2d..f5b88317d0 100644 --- a/src/backends/drm/drm_property.cpp +++ b/src/backends/drm/drm_property.cpp @@ -112,10 +112,8 @@ bool DrmProperty::needsCommit() const void DrmProperty::setCurrent(uint64_t value) { - if (m_current != value) { - updateBlob(); - m_current = value; - } + m_current = value; + updateBlob(); } uint64_t DrmProperty::current() const