summaryrefslogtreecommitdiff
path: root/kde-frameworks/kimageformats/files/kimageformats-5.108.0-psd-fix-UB-type-punning.patch
diff options
context:
space:
mode:
Diffstat (limited to 'kde-frameworks/kimageformats/files/kimageformats-5.108.0-psd-fix-UB-type-punning.patch')
-rw-r--r--kde-frameworks/kimageformats/files/kimageformats-5.108.0-psd-fix-UB-type-punning.patch89
1 files changed, 89 insertions, 0 deletions
diff --git a/kde-frameworks/kimageformats/files/kimageformats-5.108.0-psd-fix-UB-type-punning.patch b/kde-frameworks/kimageformats/files/kimageformats-5.108.0-psd-fix-UB-type-punning.patch
new file mode 100644
index 000000000000..68d853a5f447
--- /dev/null
+++ b/kde-frameworks/kimageformats/files/kimageformats-5.108.0-psd-fix-UB-type-punning.patch
@@ -0,0 +1,89 @@
+From c3a91c3bc62bdd913c55dd83f4e1159ed25310c5 Mon Sep 17 00:00:00 2001
+From: Mirco Miranda <mirco.miranda@systemceramics.com>
+Date: Sun, 16 Jul 2023 08:03:58 +0000
+Subject: [PATCH] psd: Fix UB type punning
+
+BUGS: 471829
+---
+ src/imageformats/psd.cpp | 33 ++++++++++++++++++++++++++-------
+ 1 file changed, 26 insertions(+), 7 deletions(-)
+
+diff --git a/src/imageformats/psd.cpp b/src/imageformats/psd.cpp
+index 7280aea..b0c76e2 100644
+--- a/src/imageformats/psd.cpp
++++ b/src/imageformats/psd.cpp
+@@ -42,6 +42,7 @@
+ #include <QColorSpace>
+
+ #include <cmath>
++#include <cstring>
+
+ typedef quint32 uint;
+ typedef quint16 ushort;
+@@ -808,6 +809,26 @@ inline quint32 xchg(quint32 v) {
+ #endif
+ }
+
++inline float xchg(float v)
++{
++#if Q_BYTE_ORDER == Q_LITTLE_ENDIAN
++# ifdef Q_CC_MSVC
++ float *pf = &v;
++ quint32 f = xchg(*reinterpret_cast<quint32*>(pf));
++ quint32 *pi = &f;
++ return *reinterpret_cast<float*>(pi);
++# else
++ quint32 t;
++ std::memcpy(&t, &v, sizeof(quint32));
++ t = xchg(t);
++ std::memcpy(&v, &t, sizeof(quint32));
++ return v;
++# endif
++#else
++ return v; // never tested
++#endif
++}
++
+ template<class T>
+ inline void planarToChunchy(uchar *target, const char *source, qint32 width, qint32 c, qint32 cn)
+ {
+@@ -818,15 +839,13 @@ inline void planarToChunchy(uchar *target, const char *source, qint32 width, qin
+ }
+ }
+
+-template<class T, T min = 0, T max = 1>
+-inline void planarToChunchyFloat(uchar *target, const char *source, qint32 width, qint32 c, qint32 cn)
++template<class T>
++inline void planarToChunchyFloatToUInt16(uchar *target, const char *source, qint32 width, qint32 c, qint32 cn)
+ {
+ auto s = reinterpret_cast<const T*>(source);
+ auto t = reinterpret_cast<quint16*>(target);
+ for (qint32 x = 0; x < width; ++x) {
+- auto tmp = xchg(s[x]);
+- auto ftmp = (*reinterpret_cast<float*>(&tmp) - double(min)) / (double(max) - double(min));
+- t[x * cn + c] = quint16(std::min(ftmp * std::numeric_limits<quint16>::max() + 0.5, double(std::numeric_limits<quint16>::max())));
++ t[x * cn + c] = quint16(std::min(xchg(s[x]) * std::numeric_limits<quint16>::max() + 0.5, double(std::numeric_limits<quint16>::max())));
+ }
+ }
+
+@@ -1140,7 +1159,7 @@ static bool LoadPSD(QDataStream &stream, const PSDHeader &header, QImage &img)
+ } else if (header.depth == 16) {
+ planarToChunchy<quint16>(scanLine, rawStride.data(), header.width, c, header.channel_count);
+ } else if (header.depth == 32) {
+- planarToChunchyFloat<quint32>(scanLine, rawStride.data(), header.width, c, header.channel_count);
++ planarToChunchyFloatToUInt16<float>(scanLine, rawStride.data(), header.width, c, header.channel_count);
+ }
+ }
+
+@@ -1204,7 +1223,7 @@ static bool LoadPSD(QDataStream &stream, const PSDHeader &header, QImage &img)
+ } else if (header.depth == 16) { // 16-bits integer images: Grayscale, RGB/RGBA
+ planarToChunchy<quint16>(scanLine, rawStride.data(), header.width, c, imgChannels);
+ } else if (header.depth == 32) { // 32-bits float images: Grayscale, RGB/RGBA (coverted to equivalent integer 16-bits)
+- planarToChunchyFloat<quint32>(scanLine, rawStride.data(), header.width, c, imgChannels);
++ planarToChunchyFloatToUInt16<float>(scanLine, rawStride.data(), header.width, c, imgChannels);
+ }
+ }
+ }
+--
+GitLab
+