From 967c50523329266e92d9ae590733b423cd0f646f Mon Sep 17 00:00:00 2001 From: Alexander Golubev Date: Thu, 22 Feb 2024 04:54:37 +0300 Subject: [PATCH] Fix a crash when loading a PPMRAW image The crash is caused by the fact that TQImageIO::imageFormat() may return some values not present in neighter KImageIO::types(KImageIO::Reading) nor in TQImage::inputFormatList(), e.g. different flavours of ppm like PPMRAW. Besides that it's possible theat TQt could support other formats unknown to TDE. Signed-off-by: Alexander Golubev --- src/gvcore/imageloader.cpp | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/gvcore/imageloader.cpp b/src/gvcore/imageloader.cpp index 406a378..17aedf2 100644 --- a/src/gvcore/imageloader.cpp +++ b/src/gvcore/imageloader.cpp @@ -444,7 +444,12 @@ static TQString mimeTypeFromFormat(const char* format) { TQStringList formats = KImageIO::types(KImageIO::Reading); TQStringList mimeTypes = KImageIO::mimeTypes(KImageIO::Reading); int pos = formats.findIndex(TQString::fromAscii(format)); - Q_ASSERT(pos != -1); + if(pos < 0) { + // TQImageIO::imageFormat() nay return some different types than listed by + // TQImage::inputFormats(), e.g. "PPMRAW". Also TDE might simly be ignorant + // abous some types known to TQt. + return TQString::null; + } return mimeTypes[pos]; } @@ -463,13 +468,18 @@ void ImageLoader::slotDataReceived(TDEIO::Job* job, const TQByteArray& chunk) { const char* format = TQImageIO::imageFormat(&buffer); if (format) { // This is a raster image, get the mime type now - d->mURLKind = MimeTypeUtils::KIND_RASTER_IMAGE; d->mMimeType = mimeTypeFromFormat(format); + if(d->mMimeType.isNull()) { // fall back to TDE's autoguess of mime by content + KMimeType::Ptr ptr = KMimeType::findByContent(d->mRawData); + d->mMimeType = ptr->name(); + } + d->mURLKind = MimeTypeUtils::KIND_RASTER_IMAGE; } else { KMimeType::Ptr ptr = KMimeType::findByContent(d->mRawData); d->mMimeType = ptr->name(); d->mURLKind = MimeTypeUtils::mimeTypeKind(d->mMimeType); } + if (d->mURLKind!=MimeTypeUtils::KIND_RASTER_IMAGE) { Q_ASSERT(!d->mDecoderTimer.isActive()); job->kill(true /* quietly */);