diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2018-10-20 21:34:14 +0100 |
|---|---|---|
| committer | Michael Stahl <Michael.Stahl@cib.de> | 2018-10-26 22:01:10 +0200 |
| commit | cae6fbaaea522fca72627bb1627b9b6c59b2e71a (patch) | |
| tree | f7b76f0223c9c7e187cce2a6f149d3a896efb3cf /io | |
| parent | pvs-studio: 'memcpy' function will lead to underflow of the buffer (diff) | |
| download | core-cae6fbaaea522fca72627bb1627b9b6c59b2e71a.tar.gz core-cae6fbaaea522fca72627bb1627b9b6c59b2e71a.zip | |
pvs-studio: V557 Array overrun is possible
Change-Id: Ia4a7c857b3c94490364ed979ba2fcb9e192e3796
Reviewed-on: https://gerrit.libreoffice.org/62145
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
Diffstat (limited to 'io')
| -rw-r--r-- | io/source/stm/odata.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/io/source/stm/odata.cxx b/io/source/stm/odata.cxx index aaa4178d3b0f..78d4cde4f90f 100644 --- a/io/source/stm/odata.cxx +++ b/io/source/stm/odata.cxx @@ -1149,7 +1149,7 @@ Reference< XPersistObject > OObjectInputStream::readObject() } } else { - if( m_aPersistVector.size() < nId ) + if (nId >= m_aPersistVector.size()) { // id unknown, load failure ! bLoadSuccessful = false; |