package/source/zipapi/ZipFile.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 8080459c1c91a1ccddb1b1607bb310d29e621548
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Mon Nov 11 12:19:54 2024 +0100
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Mon Nov 11 17:48:07 2024 +0100

    tdf#163486: PVS: exception thrown by pointer
    
    Since
    commit a6ad198d097fb4a503c8d5831d484ff46721134b
    Author: Noel Grandin <noel.gran...@collabora.co.uk>
    Date:   Sat Aug 17 13:19:54 2024 +0200
    
        tdf#158556 use more comphelper::ByteReader
    
    V1022   An exception was thrown by pointer. Consider throwing it by value 
instead.
    
    Change-Id: I9218d79097975e47822f48838472ca2096d527d7
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176398
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/package/source/zipapi/ZipFile.cxx 
b/package/source/zipapi/ZipFile.cxx
index ea674f642c29..24241e66af2e 100644
--- a/package/source/zipapi/ZipFile.cxx
+++ b/package/source/zipapi/ZipFile.cxx
@@ -952,7 +952,7 @@ sal_uInt64 ZipFile::readLOC_Impl(ZipEntry &rEntry, 
std::vector<sal_Int8>& rNameB
     aGrabber.seek(nPos);
     std::array<sal_Int8, 30> aHeader;
     if (aGrabber.readBytes(aHeader.data(), 30) != 30)
-        throw new uno::RuntimeException();
+        throw uno::RuntimeException();
     MemoryByteGrabber headerMemGrabber(aHeader.data(), 30);
 
     sal_Int32 nTestSig = headerMemGrabber.ReadInt32();

Reply via email to