From 7e974221de906eccb5b2b10cf744f8768963b761 Mon Sep 17 00:00:00 2001 From: Michael Meeks Date: Thu, 7 Jul 2016 20:23:41 +0100 Subject: tdf#99512 - opencl - restrict scope of vlookup optimization to doubles. Change-Id: Iab7316cb167f34c13adafe142af0fdd73eb7d04c Reviewed-on: https://gerrit.libreoffice.org/27100 Tested-by: Jenkins Reviewed-by: Michael Meeks (cherry picked from commit dead5dc1ae3baab5e25d641322d138dd3d242bff) Reviewed-on: https://gerrit.libreoffice.org/27104 Reviewed-by: Jan Holesovsky (cherry picked from commit f250c06228f4d79af849ebe2ada565e42d29b8cd) Reviewed-on: https://gerrit.libreoffice.org/27143 Reviewed-by: Eike Rathke Tested-by: Michael Meeks --- sc/source/core/opencl/op_spreadsheet.cxx | 31 ++++++++++++++++++++++++++++++- 1 file changed, 30 insertions(+), 1 deletion(-) diff --git a/sc/source/core/opencl/op_spreadsheet.cxx b/sc/source/core/opencl/op_spreadsheet.cxx index c18b2ba87a52..243d2993bca3 100644 --- a/sc/source/core/opencl/op_spreadsheet.cxx +++ b/sc/source/core/opencl/op_spreadsheet.cxx @@ -45,12 +45,41 @@ void OpVLookup::GenSlidingWindowFunction(std::stringstream &ss, CheckSubArgumentIsNan(ss,vSubArguments,arg++); int secondParaWidth = 1; + // tdf#99512 - for now only allow non-dynamic indicees (the + // common-case) to validate consistent return types vs. the input. + int index = 0; + int indexArg = vSubArguments.size() - 2; + if (vSubArguments[indexArg]->GetFormulaToken()->GetType() == formula::svDouble) + { + const formula::FormulaDoubleToken *dblToken = static_cast(vSubArguments[indexArg]->GetFormulaToken()); + index = ::rtl::math::approxFloor(dblToken->GetDouble()); + } + if (vSubArguments[1]->GetFormulaToken()->GetType() == formula::svDoubleVectorRef) { FormulaToken *tmpCur = vSubArguments[1]->GetFormulaToken(); const formula::DoubleVectorRefToken*pCurDVR = static_cast(tmpCur); - secondParaWidth = pCurDVR->GetArrays().size(); + const std::vector items = pCurDVR->GetArrays(); + + secondParaWidth = items.size(); + + if (index < 1 || index > secondParaWidth) + throw Unhandled(__FILE__, __LINE__); // oob index. + + if (items[index - 1].mpStringArray) + { + rtl_uString **pStrings = items[index - 1].mpStringArray; + for (size_t i = 0; i < pCurDVR->GetArrayLength(); ++i) + { + if (pStrings[i] != nullptr) + { // TODO: the GroupTokenConverter should do better. + throw Unhandled(__FILE__, __LINE__); // mixed arguments. + } + } + } } + else + throw Unhandled(__FILE__, __LINE__); // unusual vlookup. arg += secondParaWidth; CheckSubArgumentIsNan(ss,vSubArguments,arg++); -- cgit v1.2.3