diff options
author | Moody Liu <mooodyhunter@outlook.com> | 2022-06-17 23:21:45 +0100 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2022-06-20 13:23:54 +0000 |
commit | 4f6ad68b2e7869d15a94b0482233cceb71669bd2 (patch) | |
tree | b7c1176bb3642feea5b1966ae5506295cf9b9e5f | |
parent | 65faf9a3d8b7ce428947bd11719141c1f036131c (diff) |
jp2: use correct buffer_size for jas_stream_memopen
According to jasper's documentation, the second argument should
be 0 instead of -1, to acquire such a dynamically growing
buffer. This also causes a runtime crash due to the signedness of
size_t.
Fixes: QTBUG-104398
Change-Id: I173c6b7c9802c22fe0fa01083a71bf3b6ca7b134
Reviewed-by: Liang Qi <liang.qi@qt.io>
(cherry picked from commit ad2e721683b7c68748b05862fe8cf33bf63ebf55)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
-rw-r--r-- | src/plugins/imageformats/jp2/qjp2handler.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/imageformats/jp2/qjp2handler.cpp b/src/plugins/imageformats/jp2/qjp2handler.cpp index ab58604..acf5bd9 100644 --- a/src/plugins/imageformats/jp2/qjp2handler.cpp +++ b/src/plugins/imageformats/jp2/qjp2handler.cpp @@ -857,7 +857,7 @@ bool Jpeg2000JasperReader::write(const QImage &image, int quality) } // Open an empty jasper stream that grows automatically - jas_stream_t * memory_stream = jas_stream_memopen(0, -1); + jas_stream_t * memory_stream = jas_stream_memopen(0, 0); // Jasper wants a non-const string. char *str = qstrdup(jasperFormatString.toLatin1().constData()); |