sdext/source/pdfimport/filterdet.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit 45a7c21ed8d7588a676878b52d12113a3fe2e077 Author: Xisco Fauli <xiscofa...@libreoffice.org> AuthorDate: Wed Nov 27 12:31:12 2024 +0100 Commit: Xisco Fauli <xiscofa...@libreoffice.org> CommitDate: Wed Nov 27 14:02:00 2024 +0100 PVS: V560 A part of conditional expression is always true: bSuccess Since commit 217ef2ed9b8a757b7b02feac799621d20d0f312e Author: Tomaž Vajngerl <tomaz.vajng...@collabora.co.uk> Date: Mon Jan 23 12:32:25 2023 +0900 pdfimport: refactor pdf and hybrid format detection code Change-Id: Ib8bd09417eb07bd395b38a04245de30e8abda5a6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177408 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org> diff --git a/sdext/source/pdfimport/filterdet.cxx b/sdext/source/pdfimport/filterdet.cxx index 3a5e0f2ec480..0654cc8beec6 100644 --- a/sdext/source/pdfimport/filterdet.cxx +++ b/sdext/source/pdfimport/filterdet.cxx @@ -316,7 +316,7 @@ OUString SAL_CALL PDFDetector::detect( uno::Sequence< beans::PropertyValue >& rF oslFileHandle aFileHandle = nullptr; // check for hybrid PDF - if (bSuccess && (aURL.isEmpty() || !comphelper::isFileUrl(aURL))) + if (aURL.isEmpty() || !comphelper::isFileUrl(aURL)) { if (osl_createTempFile(nullptr, &aFileHandle, &aURL.pData) != osl_File_E_None) {