android-cloexec-inotify-init1.cpp
1.78 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
// RUN: %check_clang_tidy %s android-cloexec-inotify-init1 %t
#define IN_NONBLOCK 1
#define __O_CLOEXEC 3
#define IN_CLOEXEC __O_CLOEXEC
#define TEMP_FAILURE_RETRY(exp) \
({ \
int _rc; \
do { \
_rc = (exp); \
} while (_rc == -1); \
})
extern "C" int inotify_init1(int flags);
void a() {
inotify_init1(IN_NONBLOCK);
// CHECK-MESSAGES: :[[@LINE-1]]:28: warning: 'inotify_init1' should use IN_CLOEXEC where possible [android-cloexec-inotify-init1]
// CHECK-FIXES: inotify_init1(IN_NONBLOCK | IN_CLOEXEC);
TEMP_FAILURE_RETRY(inotify_init1(IN_NONBLOCK));
// CHECK-MESSAGES: :[[@LINE-1]]:47: warning: 'inotify_init1'
// CHECK-FIXES: TEMP_FAILURE_RETRY(inotify_init1(IN_NONBLOCK | IN_CLOEXEC));
}
void f() {
inotify_init1(0);
// CHECK-MESSAGES: :[[@LINE-1]]:18: warning: 'inotify_init1'
// CHECK-FIXES: inotify_init1(IN_CLOEXEC);
TEMP_FAILURE_RETRY(inotify_init1(0));
// CHECK-MESSAGES: :[[@LINE-1]]:37: warning: 'inotify_init1'
// CHECK-FIXES: TEMP_FAILURE_RETRY(inotify_init1(IN_CLOEXEC));
int flag = 1;
inotify_init1(flag);
TEMP_FAILURE_RETRY(inotify_init1(flag));
}
namespace i {
int inotify_init1(int flags);
void d() {
inotify_init1(IN_NONBLOCK);
TEMP_FAILURE_RETRY(inotify_init1(IN_NONBLOCK));
}
} // namespace i
void e() {
inotify_init1(IN_CLOEXEC);
TEMP_FAILURE_RETRY(inotify_init1(IN_CLOEXEC));
inotify_init1(IN_NONBLOCK | IN_CLOEXEC);
TEMP_FAILURE_RETRY(inotify_init1(IN_NONBLOCK | IN_CLOEXEC));
}
class G {
public:
int inotify_init1(int flags);
void d() {
inotify_init1(IN_CLOEXEC);
TEMP_FAILURE_RETRY(inotify_init1(IN_CLOEXEC));
inotify_init1(IN_NONBLOCK | IN_CLOEXEC);
TEMP_FAILURE_RETRY(inotify_init1(IN_NONBLOCK | IN_CLOEXEC));
}
};