[GH-667] core: Avoid value truncation on 32bit architectures

Replace `labs()` with `llabs()` to avoid truncation on 32bit architectures. Also do not tolerate value truncation in the CI build.
eclipse-wakaama · Jul 6, 2022 · a782907 · a782907
1 parent fd7c070
commit a782907
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/core/utils.c b/core/utils.c
@@ -218,7 +218,7 @@ size_t utils_intToText(int64_t data,
             string[int64_min_strlen] = '\0';
             return int64_min_strlen;
         }
-        result = utils_uintToText((uint64_t)labs(data), string + 1, length - 1);
+        result = utils_uintToText(llabs(data), string + 1, length - 1);
         if(result != 0)
         {
             result += 1;

diff --git a/wakaama.cmake b/wakaama.cmake
@@ -124,6 +124,7 @@ add_compile_options(
     -Wshadow
     -Wswitch-default
     -Wwrite-strings
+    -Werror=absolute-value
     -pedantic
 )