[Libreoffice-commits] core.git: Branch 'libreoffice-5-1' - formula/source

[Eike Rathke]

 formula/source/ui/dlg/parawin.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 221504d3a76d63e668c4cc287e0bd5580218020a
Author: Eike Rathke <erack at redhat.com>
Date:   Thu Dec 17 15:41:31 2015 +0100

    out-of-bounds arg mapping access, this was always wrong but never hit it seems
    
    Change-Id: I15843b320ac8ddcefce8094da7aa7c6b81b42e4f
    (cherry picked from commit 5b404cc2adcea04938c8159c0bc84a8cd44ec5df)

diff --git a/formula/source/ui/dlg/parawin.cxx b/formula/source/ui/dlg/parawin.cxx
index e32d257..fe41e6b 100644
--- a/formula/source/ui/dlg/parawin.cxx
+++ b/formula/source/ui/dlg/parawin.cxx
@@ -122,7 +122,7 @@ void ParaWin::UpdateArgDesc( sal_uInt16 nArg )
 
         if ( nArgs < VAR_ARGS )
         {
-            sal_uInt16 nRealArg = (aVisibleArgMapping.size() < nArg) ? aVisibleArgMapping[nArg] : nArg;
+            sal_uInt16 nRealArg = (nArg < aVisibleArgMapping.size()) ? aVisibleArgMapping[nArg] : nArg;
             aArgDesc  = pFuncDesc->getParameterDescription(nRealArg);
             aArgName  = pFuncDesc->getParameterName(nRealArg);
             aArgName += " ";