Clang Project

clang_source_code/test/Analysis/dispatch-once.m
1// RUN: %clang_analyze_cc1 -w -fblocks -analyzer-checker=core,osx.API,unix.Malloc -verify %s
2// RUN: %clang_analyze_cc1 -w -fblocks -fobjc-arc -analyzer-checker=core,osx.API,unix.Malloc -verify %s
3
4#include "Inputs/system-header-simulator-objc.h"
5
6typedef unsigned long size_t;
7void *calloc(size_t nmemb, size_t size);
8
9typedef void (^dispatch_block_t)(void);
10typedef long dispatch_once_t;
11void dispatch_once(dispatch_once_t *predicate, dispatch_block_t block);
12
13void test_stack() {
14  dispatch_once_t once;
15  dispatch_once(&once, ^{}); // expected-warning{{Call to 'dispatch_once' uses the local variable 'once' for the predicate value.  Using such transient memory for the predicate is potentially dangerous.  Perhaps you intended to declare the variable as 'static'?}}
16}
17
18void test_static_local() {
19  static dispatch_once_t once;
20  dispatch_once(&once, ^{}); // no-warning
21}
22
23void test_heap_var() {
24  dispatch_once_t *once = calloc(1, sizeof(dispatch_once_t));
25  // Use regexps to check that we're NOT suggesting to make this static.
26  dispatch_once(once, ^{}); // expected-warning-re{{{{^Call to 'dispatch_once' uses heap-allocated memory for the predicate value.  Using such transient memory for the predicate is potentially dangerous$}}}}
27}
28
29void test_external_pointer(dispatch_once_t *once) {
30  // External pointer does not necessarily point to the heap.
31  dispatch_once(once, ^{}); // no-warning
32}
33
34typedef struct {
35  dispatch_once_t once;
36} Struct;
37
38void test_local_struct() {
39  Struct s;
40  dispatch_once(&s.once, ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the local variable 's' for the predicate value.}}
41}
42
43void test_heap_struct() {
44  Struct *s = calloc(1, sizeof(Struct));
45  dispatch_once(&s->once, ^{}); // expected-warning{{Call to 'dispatch_once' uses heap-allocated memory for the predicate value.}}
46}
47
48@interface Object : NSObject {
49@public
50  dispatch_once_t once;
51  Struct s;
52  dispatch_once_t once_array[2];
53}
54- (void)test_ivar_from_inside;
55- (void)test_ivar_struct_from_inside;
56@end
57
58@implementation Object
59- (void)test_ivar_from_inside {
60  dispatch_once(&once, ^{}); // expected-warning{{Call to 'dispatch_once' uses the instance variable 'once' for the predicate value.}}
61}
62- (void)test_ivar_struct_from_inside {
63  dispatch_once(&s.once, ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the instance variable 's' for the predicate value.}}
64}
65- (void)test_ivar_array_from_inside {
66  dispatch_once(&once_array[1], ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the instance variable 'once_array' for the predicate value.}}
67}
68@end
69
70void test_ivar_from_alloc_init() {
71  Object *o = [[Object alloc] init];
72  dispatch_once(&o->once, ^{}); // expected-warning{{Call to 'dispatch_once' uses the instance variable 'once' for the predicate value.}}
73}
74void test_ivar_struct_from_alloc_init() {
75  Object *o = [[Object alloc] init];
76  dispatch_once(&o->s.once, ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the instance variable 's' for the predicate value.}}
77}
78void test_ivar_array_from_alloc_init() {
79  Object *o = [[Object alloc] init];
80  dispatch_once(&o->once_array[1], ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the instance variable 'once_array' for the predicate value.}}
81}
82
83void test_ivar_from_external_obj(Object *o) {
84  // ObjC object pointer always points to the heap.
85  dispatch_once(&o->once, ^{}); // expected-warning{{Call to 'dispatch_once' uses the instance variable 'once' for the predicate value.}}
86}
87void test_ivar_struct_from_external_obj(Object *o) {
88  dispatch_once(&o->s.once, ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the instance variable 's' for the predicate value.}}
89}
90void test_ivar_array_from_external_obj(Object *o) {
91  dispatch_once(&o->once_array[1], ^{}); // expected-warning{{Call to 'dispatch_once' uses memory within the instance variable 'once_array' for the predicate value.}}
92}
93
94void test_block_var_from_block() {
95  __block dispatch_once_t once;
96  ^{
97    dispatch_once(&once, ^{}); // expected-warning{{Call to 'dispatch_once' uses the block variable 'once' for the predicate value.}}
98  };
99}
100
101void use_block_var(dispatch_once_t *once);
102
103void test_block_var_from_outside_block() {
104  __block dispatch_once_t once;
105  ^{
106    use_block_var(&once);
107  };
108  dispatch_once(&once, ^{}); // expected-warning{{Call to 'dispatch_once' uses the block variable 'once' for the predicate value.}}
109}
110
111void test_static_var_from_outside_block() {
112  static dispatch_once_t once;
113  ^{
114    dispatch_once(&once, ^{}); // no-warning
115  };
116}
117
CodeThinker, Enjoy your reading code