Skip to content

Commit 734646d

Browse files
authored
Merge pull request #3096 from gberkes/v3/sonar_return_never_will_be_executed
Clean up 'return' never will be executed.
2 parents f474cc5 + 64dfe41 commit 734646d

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

examples/reading_logs_via_rule_message/reading_logs_via_rule_message.h

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ struct data_ms {
7373
};
7474

7575

76-
static void *process_request(void *data) {
76+
[[noreturn]] static void *process_request(void *data) {
7777
struct data_ms *a = (struct data_ms *)data;
7878
modsecurity::ModSecurity *modsec = a->modsec;
7979
modsecurity::RulesSet *rules = a->rules;
@@ -102,8 +102,7 @@ static void *process_request(void *data) {
102102
delete modsecTransaction;
103103
}
104104

105-
pthread_exit(NULL);
106-
return NULL;
105+
pthread_exit(nullptr);
107106
}
108107

109108

@@ -165,7 +164,6 @@ class ReadingLogsViaRuleMessage {
165164

166165
delete rules;
167166
delete modsec;
168-
pthread_exit(NULL);
169167
return 0;
170168
}
171169

examples/reading_logs_via_rule_message/simple_request.cc

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,5 @@ int main(int argc, char **argv) {
3535
response_headers, response_body, ip, rules);
3636
rlvrm.process();
3737

38-
39-
40-
pthread_exit(NULL);
4138
return 0;
4239
}

0 commit comments

Comments
 (0)