summaryrefslogtreecommitdiffstats
path: root/clang/test
diff options
context:
space:
mode:
authorTed Kremenek <kremenek@apple.com>2012-04-26 05:08:26 +0000
committerTed Kremenek <kremenek@apple.com>2012-04-26 05:08:26 +0000
commit468365b6bea6f041cd0911d065c308fa4478d96b (patch)
tree14f89e3b64a96a41f35cfe2e188bd6187d2847f3 /clang/test
parent01d152f43fc919ff648c70b24a3883d178c707c7 (diff)
downloadbcm5719-llvm-468365b6bea6f041cd0911d065c308fa4478d96b.tar.gz
bcm5719-llvm-468365b6bea6f041cd0911d065c308fa4478d96b.zip
[analyzer] check lazy bindings in RegionStore first before looking for default values. Fixes <rdar://problem/11269741>.
llvm-svn: 155615
Diffstat (limited to 'clang/test')
-rw-r--r--clang/test/Analysis/malloc.c16
1 files changed, 16 insertions, 0 deletions
diff --git a/clang/test/Analysis/malloc.c b/clang/test/Analysis/malloc.c
index 3b4712320b7..32c6171f8c4 100644
--- a/clang/test/Analysis/malloc.c
+++ b/clang/test/Analysis/malloc.c
@@ -760,6 +760,22 @@ void radar10978247_positive(int myValueSize) {
return;
}
+// <rdar://problem/11269741> Previously this triggered a false positive
+// because malloc() is known to return uninitialized memory and the binding
+// of 'o' to 'p->n' was not getting propertly handled. Now we report a leak.
+struct rdar11269741_a_t {
+ struct rdar11269741_b_t {
+ int m;
+ } n;
+};
+
+int rdar11269741(struct rdar11269741_b_t o)
+{
+ struct rdar11269741_a_t *p = (struct rdar11269741_a_t *) malloc(sizeof(*p));
+ p->n = o;
+ return p->n.m; // expected-warning {{leak}}
+}
+
// ----------------------------------------------------------------------------
// Below are the known false positives.
OpenPOWER on IntegriCloud