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)
         {

Reply via email to