[Libreoffice-commits] core.git: sc/source

[Caolán McNamara (via logerrit)]

 sc/source/core/data/dpsave.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 2a7a576d19b9c212fd6f906f5e275fdf2266608e
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Fri Aug 26 09:06:58 2022 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Fri Aug 26 11:37:03 2022 +0200

    cid#1500451 silence Resource leak
    
    Change-Id: I80851372e8cb27ada07cfacd2294287e9d7df965
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138865
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sc/source/core/data/dpsave.cxx b/sc/source/core/data/dpsave.cxx
index e25af1f1500c..cf694258bcd9 100644
--- a/sc/source/core/data/dpsave.cxx
+++ b/sc/source/core/data/dpsave.cxx
@@ -964,9 +964,9 @@ void ScDPSaveData::SetPosition( ScDPSaveDimension* pDim, 
tools::Long nNew )
         [&pDim](const std::unique_ptr<ScDPSaveDimension>& rxDim) { return pDim 
== rxDim.get(); });
     if (it != m_DimList.end())
     {
-        // Tell vector<unique_ptr> to give up ownership of this element.
-        // Don't delete this instance as it is re-inserted into the
-        // container later.
+        // Tell vector<unique_ptr> to give up ownership of this element.  Don't
+        // delete this instance as it is re-inserted into the container later.
+        // coverity[leaked_storage] - re-inserted into the container later
         it->release();
         m_DimList.erase(it);
     }