[Libreoffice-commits] core.git: solenv/sanitizers

[Stephan Bergmann]

 solenv/sanitizers/ubsan-suppressions |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 052423984614911c7b711dba106315b1b67e7047
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Fri Mar 2 15:18:06 2018 +0100

    More sc files relying on float-divide-by-zero resulting in inf
    
    ...triggering -fsanitize=float-divide-by-zero during e.g.
    CppunitTest_sc_statistical_functions_test.  (Found only now after I removed
    -fno-sanitize-recover=ALL -fsanitize-recover=float-divide-by-zero from CXX and
    instead added halt_on_error=1 to UBSAN_OPTIONS.)
    
    Change-Id: Iead2147e42af33b987edb8e10804bc4eefa1aeca
    Reviewed-on: https://gerrit.libreoffice.org/50637
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>
    Tested-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/solenv/sanitizers/ubsan-suppressions b/solenv/sanitizers/ubsan-suppressions
index 05adbd33ef06..057178cdfb09 100644
--- a/solenv/sanitizers/ubsan-suppressions
+++ b/solenv/sanitizers/ubsan-suppressions
@@ -7,7 +7,7 @@
 #
 
 # Various places in sc rely on floating-point division by zero to produce
-# infinity (as triggered by CppunitTest_sc_functions_test); silence those UBSan
+# infinity (as triggered by e.g. CppunitTest_sc_functions_test); silence those UBSan
 # reports by adding
 # suppressions=/path-to-this/solenv/sanitizers/ubsan-suppressions to the
 # UBSAN_OPTIONS environment variable (and making sure to compile with
@@ -15,5 +15,7 @@
 
 float-divide-by-zero:sc/source/core/tool/interpr1.cxx
 float-divide-by-zero:sc/source/core/tool/interpr2.cxx
+float-divide-by-zero:sc/source/core/tool/interpr3.cxx
+float-divide-by-zero:sc/source/core/tool/interpr8.cxx
 float-divide-by-zero:scaddins/source/analysis/analysis.cxx
 float-divide-by-zero:scaddins/source/analysis/financial.cxx