Merge pull request #1761 from Robotechnic/master

Fix for #1758

Author: sim642

src/analyses/base.ml

@@ -2492,7 +2492,9 @@ struct
           | _ ->
             set ~man st lv_a lv_typ (VD.top_value (unrollType lv_typ))
         end
-    in
+    in 
+    (* Evaluate each functions arguments. `eval_rv` is only called for its side effects, we ignore the result. *)
+    List.iter (fun arg -> eval_rv ~man st arg |> ignore) args; 
     let st = match desc.special args, f.vname with
     | Memset { dest; ch; count; }, _ ->
       (* TODO: check count *)

tests/regression/26-undefined_behavior/02-array-out-of-bounds.c

@@ -11,6 +11,7 @@ int main()
     for (int i = 0; i < 10; ++i)
     {
         arr[i] = 5; //WARN
+        printf("%d\n", arr[i]); //WARN
     }
     return 0;
 }

tests/regression/39-signed-overflows/14-printf-evaluation.c

@@ -0,0 +1,9 @@
+//PARAM: --enable ana.int.interval
+
+#include <limits.h>
+#include <stdio.h>
+
+int main() {
+  printf("%d\n", INT_MAX + 1); //WARN
+  return 0;
+}