[Libreoffice-commits] core.git: oox/source xmloff/source

Caolán McNamara (via logerrit) logerrit at kemper.freedesktop.org
Fri Jul 19 09:42:04 UTC 2019


 oox/source/export/ColorPropertySet.cxx   |    3 ++-
 xmloff/source/chart/ColorPropertySet.cxx |    3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

New commits:
commit 6dfc659c5e468ad969291ad39085b9c3ec53bcac
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Thu Jul 18 11:55:08 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Jul 19 11:41:24 2019 +0200

    cid#1448264 silence Out-of-bounds access
    
    Change-Id: I12267e8709d8f36acbbd6c70bfdb80a2a500e5d3
    Reviewed-on: https://gerrit.libreoffice.org/75910
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/oox/source/export/ColorPropertySet.cxx b/oox/source/export/ColorPropertySet.cxx
index 8c48022c4288..e1a861d52ff6 100644
--- a/oox/source/export/ColorPropertySet.cxx
+++ b/oox/source/export/ColorPropertySet.cxx
@@ -158,7 +158,8 @@ PropertyState SAL_CALL ColorPropertySet::getPropertyState( const OUString& /* Pr
 Sequence< PropertyState > SAL_CALL ColorPropertySet::getPropertyStates( const Sequence< OUString >& /* aPropertyName */ )
 {
     PropertyState aState = PropertyState_DIRECT_VALUE;
-    return Sequence< PropertyState >( & aState, 1 );
+    // coverity[overrun-buffer-arg : FALSE] - coverity has difficulty with css::uno::Sequence
+    return Sequence<PropertyState>(&aState, 1);
 }
 
 void SAL_CALL ColorPropertySet::setPropertyToDefault( const OUString& PropertyName )
diff --git a/xmloff/source/chart/ColorPropertySet.cxx b/xmloff/source/chart/ColorPropertySet.cxx
index 10f62e9aa81d..388e33d60e01 100644
--- a/xmloff/source/chart/ColorPropertySet.cxx
+++ b/xmloff/source/chart/ColorPropertySet.cxx
@@ -137,7 +137,8 @@ PropertyState SAL_CALL ColorPropertySet::getPropertyState( const OUString& /* Pr
 Sequence< PropertyState > SAL_CALL ColorPropertySet::getPropertyStates( const Sequence< OUString >& /* aPropertyName */ )
 {
     PropertyState aState = PropertyState_DIRECT_VALUE;
-    return Sequence< PropertyState >( & aState, 1 );
+    // coverity[overrun-buffer-arg : FALSE] - coverity has difficulty with css::uno::Sequence
+    return Sequence<PropertyState>(&aState, 1);
 }
 
 void SAL_CALL ColorPropertySet::setPropertyToDefault( const OUString& PropertyName )


More information about the Libreoffice-commits mailing list