core.git: 2 commits - desktop/unx sw/source

[Caolán McNamara (via logerrit)]

 desktop/unx/source/start.c         |    1 +
 sw/source/core/unocore/unoport.cxx |    2 +-
 sw/source/core/unocore/unotbl.cxx  |    4 ++--
 3 files changed, 4 insertions(+), 3 deletions(-)

New commits:
commit d7cfb08c1ff7bfe0da477783fc2c0fbb661b8b29
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Thu Nov 21 08:54:48 2024 +0000
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Fri Nov 22 20:34:31 2024 +0100

    cid#1634864 avoid Missing move assignment operator
    
    and
    
    cid#1634866 Missing move assignment operator
    
    Change-Id: Iaf588bdd9185da6afeee26eea0607e72a9792bab
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177045
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/sw/source/core/unocore/unoport.cxx 
b/sw/source/core/unocore/unoport.cxx
index ce9f2bfb5698..54c58c112207 100644
--- a/sw/source/core/unocore/unoport.cxx
+++ b/sw/source/core/unocore/unoport.cxx
@@ -52,7 +52,7 @@ using namespace ::com::sun::star;
 
 void SwXTextPortion::init(const SwUnoCursor* pPortionCursor)
 {
-    m_pUnoCursor = 
pPortionCursor->GetDoc().CreateUnoCursor(*pPortionCursor->GetPoint());
+    
m_pUnoCursor.reset(pPortionCursor->GetDoc().CreateUnoCursor(*pPortionCursor->GetPoint()));
     if (pPortionCursor->HasMark())
     {
         m_pUnoCursor->SetMark();
diff --git a/sw/source/core/unocore/unotbl.cxx 
b/sw/source/core/unocore/unotbl.cxx
index 3d3f52d60516..d2c31d2474b7 100644
--- a/sw/source/core/unocore/unotbl.cxx
+++ b/sw/source/core/unocore/unotbl.cxx
@@ -1454,7 +1454,7 @@ SwXTextTableCursor::SwXTextTableCursor(SwFrameFormat* 
pFrameFormat, SwTableBox c
     SwDoc* pDoc = m_pFrameFormat->GetDoc();
     const SwStartNode* pSttNd = pBox->GetSttNd();
     SwPosition aPos(*pSttNd);
-    m_pUnoCursor = pDoc->CreateUnoCursor(aPos, true);
+    m_pUnoCursor.reset(pDoc->CreateUnoCursor(aPos, true));
     m_pUnoCursor->Move( fnMoveForward, GoInNode );
     SwUnoTableCursor& rTableCursor = 
dynamic_cast<SwUnoTableCursor&>(*m_pUnoCursor);
     rTableCursor.MakeBoxSels();
@@ -1465,7 +1465,7 @@ SwXTextTableCursor::SwXTextTableCursor(SwFrameFormat& 
rTableFormat, const SwTabl
     , m_pPropSet(aSwMapProvider.GetPropertySet(PROPERTY_MAP_TEXT_TABLE_CURSOR))
 {
     StartListening(m_pFrameFormat->GetNotifier());
-    m_pUnoCursor = 
pTableSelection->GetDoc().CreateUnoCursor(*pTableSelection->GetPoint(), true);
+    
m_pUnoCursor.reset(pTableSelection->GetDoc().CreateUnoCursor(*pTableSelection->GetPoint(),
 true));
     if(pTableSelection->HasMark())
     {
         m_pUnoCursor->SetMark();
commit 36c2ce13445d4326d97237a23ad3e517320360e8
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Thu Nov 21 09:29:19 2024 +0000
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Fri Nov 22 20:34:25 2024 +0100

    cid#1608081 silence Overflowed array index write
    
    Change-Id: I40d4207eafe46736a122e07c56f6db94cb517697
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177044
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/desktop/unx/source/start.c b/desktop/unx/source/start.c
index a8d6b9ff0139..ee4358bc904d 100644
--- a/desktop/unx/source/start.c
+++ b/desktop/unx/source/start.c
@@ -516,6 +516,7 @@ static ProgressStatus read_percent(ChildInfo const *info, 
int *pPercent)
     memmove(pBuffer, pNext, nNotProcessed);
 
     /* read data */
+    // coverity[ tainted_data_return : FALSE ] version 2023.12.2
     ssize_t nThisRead = read(child_info_get_status_fd(info),
                              pBuffer + nNotProcessed, BUFFER_LEN - 
nNotProcessed);