diff options
author | Caolán McNamara <caolanm@redhat.com> | 2018-08-13 16:41:31 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-08-14 10:52:59 +0200 |
commit | 148b3cfd0b948163416f1d19ee6876d3459f1d9f (patch) | |
tree | 5317eac46ec42be79a03fbb39a7328b8a209b3f8 | |
parent | 3fc47cda232e396b026f8f26f56f96e0f65dbccc (diff) |
ofz: check sequence length before use
Change-Id: I137a9de49c5a73eb5f277dc1519e5e036abba31c
Reviewed-on: https://gerrit.libreoffice.org/58946
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
-rw-r--r-- | writerfilter/source/ooxml/OOXMLDocumentImpl.cxx | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx b/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx index 4848d1d59aad..4858570b8572 100644 --- a/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx +++ b/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx @@ -633,6 +633,12 @@ void OOXMLDocumentImpl::resolveGlossaryStream(Stream & /*rStream*/) OOXMLStream::Pointer_t gStream; uno::Sequence< beans::StringPair > aSeq = aSeqs[j]; //Follows following aSeq[0] is Id, aSeq[1] is Type, aSeq[2] is Target + if (aSeq.getLength() < 3) + { + SAL_WARN("writerfilter.ooxml", "too short sequence"); + continue; + } + OUString gId(aSeq[0].Second); OUString gType(aSeq[1].Second); OUString gTarget(aSeq[2].Second); |