https://bugs.freedesktop.org/show_bug.cgi?id=60063
Julien Nabet <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] | |r, [email protected], | |[email protected] --- Comment #3 from Julien Nabet <[email protected]> --- Here are 2 possible patches: 1) diff --git a/oox/source/docprop/docprophandler.cxx b/oox/source/docprop/docprophandler.cxx index 1067315..6c01d4c 100644 --- a/oox/source/docprop/docprophandler.cxx +++ b/oox/source/docprop/docprophandler.cxx @@ -174,9 +174,9 @@ uno::Sequence< ::rtl::OUString > OOXMLDocPropHandler::GetKeywordsSet( const ::rt // unfortunately I did not find any specification for the possible delimiters if ( !aResult[nCounter].isEmpty() ) { - if ( nCounter >= aResult.getLength() ) - aResult.realloc( nCounter + 10 ); nCounter++; + if ( nCounter >= aResult.getLength()) + aResult.realloc( nCounter + 10 ); 2) diff --git a/oox/source/docprop/docprophandler.cxx b/oox/source/docprop/docprophandler.cxx index 1067315..b96ed6b 100644 --- a/oox/source/docprop/docprophandler.cxx +++ b/oox/source/docprop/docprophandler.cxx @@ -174,7 +174,7 @@ uno::Sequence< ::rtl::OUString > OOXMLDocPropHandler::GetKeywordsSet( const ::rt // unfortunately I did not find any specification for the possible delimiters if ( !aResult[nCounter].isEmpty() ) { - if ( nCounter >= aResult.getLength() ) + if ( nCounter >= (aResult.getLength() - 1) ) aResult.realloc( nCounter + 10 ); nCounter++; } Both work but I'm not sure if one is wrong or if they're equivalent. Michael/Cédric: one for you? If one of them is ok, tell me, I can push the patch on master. -- You are receiving this mail because: You are the assignee for the bug.
_______________________________________________ Libreoffice-bugs mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
