1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Client: Fix potential file descriptor leak

Browse Source 
Qt doesn't use the ICC file information, but the file descriptor should
be still closed.

Change-Id: Ib793e7cc12dbd7dd338c4ab4528c17f30e48abcd
Reviewed-by: David Edmundson <davidedmundson@kde.org>
This commit is contained in:
Vlad Zahorodnii 2025-02-18 15:22:41 +02:00
parent a0d203cf89
commit 71fc85f594
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/plugins/platforms/wayland/qwaylandcolormanagement.cpp
View File

@ -6,6 +6,8 @@
#include <QDebug>
#include <unistd.h>
QT_BEGIN_NAMESPACE
namespace QtWaylandClient {
@ -146,6 +148,12 @@ void ImageDescriptionInfo::xx_image_description_info_v4_done()
Q_EMIT done();
}
void ImageDescriptionInfo::xx_image_description_info_v4_icc_file(int32_t icc, uint32_t icc_size)
{
Q_UNUSED(icc_size)
close(icc);
}
void ImageDescriptionInfo::xx_image_description_info_v4_primaries(int32_t r_x, int32_t r_y, int32_t g_x, int32_t g_y, int32_t b_x, int32_t b_y, int32_t w_x, int32_t w_y)
{
mContainerRed = QPointF(r_x, r_y) / 10'000.0;

1
src/plugins/platforms/wayland/qwaylandcolormanagement_p.h
View File

@ -89,6 +89,7 @@ public:
private:
void xx_image_description_info_v4_done() override;
void xx_image_description_info_v4_icc_file(int32_t icc, uint32_t icc_size) override;
void xx_image_description_info_v4_primaries(int32_t r_x, int32_t r_y, int32_t g_x, int32_t g_y, int32_t b_x, int32_t b_y, int32_t w_x, int32_t w_y) override;
void xx_image_description_info_v4_tf_named(uint32_t transferFunction) override;
void xx_image_description_info_v4_luminances(uint32_t min_lum, uint32_t max_lum, uint32_t reference_lum) override;