|
 |
e9d329 |
diff --git a/src/gui/image/qxbmhandler.cpp b/src/gui/image/qxbmhandler.cpp
|
|
|
e9d329 |
index 7ba44049..8c4be4f0 100644
|
|
|
e9d329 |
--- a/src/gui/image/qxbmhandler.cpp
|
|
|
e9d329 |
+++ b/src/gui/image/qxbmhandler.cpp
|
|
|
e9d329 |
@@ -158,7 +158,9 @@ static bool read_xbm_body(QIODevice *device, int w, int h, QImage *outImage)
|
|
|
e9d329 |
w = (w+7)/8; // byte width
|
|
|
e9d329 |
|
|
|
e9d329 |
while (y < h) { // for all encoded bytes...
|
|
|
e9d329 |
- if (p) { // p = "0x.."
|
|
|
e9d329 |
+ if (p && p < (buf + readBytes - 3)) { // p = "0x.."
|
|
|
e9d329 |
+ if (!isxdigit(p[2]) || !isxdigit(p[3]))
|
|
|
e9d329 |
+ return false;
|
|
|
e9d329 |
*b++ = hex2byte(p+2);
|
|
|
e9d329 |
p += 2;
|
|
|
e9d329 |
if (++x == w && ++y < h) {
|
|
|
e9d329 |
diff --git a/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp b/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp
|
|
|
e9d329 |
index 1eee2f27..f801f3cd 100644
|
|
|
e9d329 |
--- a/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp
|
|
|
e9d329 |
+++ b/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp
|
|
|
e9d329 |
@@ -167,6 +167,8 @@ private slots:
|
|
|
e9d329 |
void devicePixelRatio_data();
|
|
|
e9d329 |
void devicePixelRatio();
|
|
|
e9d329 |
|
|
|
e9d329 |
+ void xbmBufferHandling();
|
|
|
e9d329 |
+
|
|
|
e9d329 |
private:
|
|
|
e9d329 |
QString prefix;
|
|
|
e9d329 |
QTemporaryDir m_temporaryDir;
|
|
|
e9d329 |
@@ -2002,5 +2004,41 @@ void tst_QImageReader::devicePixelRatio()
|
|
|
e9d329 |
QCOMPARE(img.devicePixelRatio(), dpr);
|
|
|
e9d329 |
}
|
|
|
e9d329 |
|
|
|
e9d329 |
+void tst_QImageReader::xbmBufferHandling()
|
|
|
e9d329 |
+{
|
|
|
e9d329 |
+ uint8_t original_buffer[256];
|
|
|
e9d329 |
+ for (int i = 0; i < 256; ++i)
|
|
|
e9d329 |
+ original_buffer[i] = i;
|
|
|
e9d329 |
+
|
|
|
e9d329 |
+ QImage image(original_buffer, 256, 8, QImage::Format_MonoLSB);
|
|
|
e9d329 |
+ image.setColorTable({0xff000000, 0xffffffff});
|
|
|
e9d329 |
+
|
|
|
e9d329 |
+ QByteArray buffer;
|
|
|
e9d329 |
+ {
|
|
|
e9d329 |
+ QBuffer buf(&buffer);
|
|
|
e9d329 |
+ QImageWriter writer(&buf, "xbm");
|
|
|
e9d329 |
+ writer.write(image);
|
|
|
e9d329 |
+ }
|
|
|
e9d329 |
+
|
|
|
e9d329 |
+ QCOMPARE(QImage::fromData(buffer, "xbm"), image);
|
|
|
e9d329 |
+
|
|
|
e9d329 |
+ auto i = buffer.indexOf(',');
|
|
|
e9d329 |
+ buffer.insert(i + 1, " ");
|
|
|
e9d329 |
+ QCOMPARE(QImage::fromData(buffer, "xbm"), image);
|
|
|
e9d329 |
+ buffer.insert(i + 1, " ");
|
|
|
e9d329 |
+ QCOMPARE(QImage::fromData(buffer, "xbm"), image);
|
|
|
e9d329 |
+ buffer.insert(i + 1, " ");
|
|
|
e9d329 |
+#if 0 // Lines longer than 300 chars not supported currently
|
|
|
e9d329 |
+ QCOMPARE(QImage::fromData(buffer, "xbm"), image);
|
|
|
e9d329 |
+#endif
|
|
|
e9d329 |
+
|
|
|
e9d329 |
+ i = buffer.lastIndexOf("\n ");
|
|
|
e9d329 |
+ buffer.truncate(i + 1);
|
|
|
e9d329 |
+ buffer.append(QByteArray(297, ' '));
|
|
|
e9d329 |
+ buffer.append("0x");
|
|
|
e9d329 |
+ // Only check we get no buffer overflow
|
|
|
e9d329 |
+ QImage::fromData(buffer, "xbm");
|
|
|
e9d329 |
+}
|
|
|
e9d329 |
+
|
|
|
e9d329 |
QTEST_MAIN(tst_QImageReader)
|
|
|
e9d329 |
#include "tst_qimagereader.moc"
|