[PATCH] D40941: [ubsan] Use pass_object_size info in bounds checks (compiler-rt)

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 6 19:47:01 PST 2017


vsk created this revision.
Herald added subscribers: dberris, kubamracek.

This is a test update for the clang change in https://reviews.llvm.org/D40940


https://reviews.llvm.org/D40941

Files:
  test/ubsan/TestCases/Misc/bounds.cpp


Index: test/ubsan/TestCases/Misc/bounds.cpp
===================================================================
--- test/ubsan/TestCases/Misc/bounds.cpp
+++ test/ubsan/TestCases/Misc/bounds.cpp
@@ -5,7 +5,23 @@
 // RUN: %run %t 0 3 0 2>&1 | FileCheck %s --check-prefix=CHECK-B-3
 // RUN: %run %t 0 0 4 2>&1 | FileCheck %s --check-prefix=CHECK-C-4
 
+int get_int(int *const p __attribute__((pass_object_size(0))), int i) {
+  // CHECK-A-2: bounds.cpp:[[@LINE+1]]:10: runtime error: index 2 out of bounds for type 'int *'
+  return p[i];
+}
+
+int get_double(double *const p __attribute__((pass_object_size(0))), int i) {
+  // CHECK-A-2: bounds.cpp:[[@LINE+1]]:10: runtime error: index 2 out of bounds for type 'double *'
+  return p[i];
+}
+
 int main(int argc, char **argv) {
+  int bar[2];
+  get_int(bar, argv[1][0] - '0');
+
+  double baz[2];
+  get_double(baz, argv[1][0] - '0');
+
   int arr[2][3][4] = {};
 
   return arr[argv[1][0] - '0'][argv[2][0] - '0'][argv[3][0] - '0'];


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40941.125882.patch
Type: text/x-patch
Size: 984 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171207/5493c9eb/attachment-0001.bin>


More information about the cfe-commits mailing list