Skip to content

Commit e768e2e

Browse files
author
Porcupiney Hairs
committed
Include changes from review
1 parent a7cdf0e commit e768e2e

File tree

3 files changed

+29
-17
lines changed

3 files changed

+29
-17
lines changed
Lines changed: 28 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,43 @@
1-
#include<iostream>
2-
#include<memory>
1+
#include "../../../../../library-tests/string_concat/stl.h"
32

4-
#include<curl/curl.h>
3+
namespace std{
4+
struct CURL {};
5+
typedef CURL curl;
6+
enum curl_constant{
7+
CURLOPT_URL,
8+
CURLOPT_SSL_VERIFYHOST,
9+
CURLOPT_SSL_VERIFYPEER
10+
};
11+
12+
CURL *curl_easy_init();
13+
void curl_easy_cleanup(CURL *handle);
14+
void curl_easy_perform(CURL *handle);
15+
void curl_easy_setopt(CURL *handle, curl_constant param, int p);
16+
void curl_easy_setopt(CURL *handle, curl_constant param, char* p);
17+
}
518

6-
using namespace std;
719

8-
string host = "codeql.com"
20+
using namespace std;
21+
char host[] = "codeql.com";
922

1023
void bad(void) {
11-
std::unique_ptr<CURL, void(*)(CURL*)> curl =
12-
std::unique_ptr<CURL, void(*)(CURL*)>(curl_easy_init(), curl_easy_cleanup);
24+
std::unique_ptr<CURL> curl = std::unique_ptr<CURL>(curl_easy_init());
1325
curl_easy_setopt(curl.get(), CURLOPT_SSL_VERIFYPEER, 0);
1426
curl_easy_setopt(curl.get(), CURLOPT_SSL_VERIFYHOST, 0);
15-
curl_easy_setopt(curl.get(), CURLOPT_URL, host.c_str());
16-
curl_easy_perform(curl.get());
27+
curl_easy_setopt(curl.get(), CURLOPT_URL, host);
28+
curl_easy_perform(curl.get());
1729
}
1830

1931
void good(void) {
20-
std::unique_ptr<CURL, void(*)(CURL*)> curl =
21-
std::unique_ptr<CURL, void(*)(CURL*)>(curl_easy_init(), curl_easy_cleanup);
32+
std::unique_ptr<CURL> curl = std::unique_ptr<CURL>(curl_easy_init());
2233
curl_easy_setopt(curl.get(), CURLOPT_SSL_VERIFYPEER, 2);
2334
curl_easy_setopt(curl.get(), CURLOPT_SSL_VERIFYHOST, 2);
24-
curl_easy_setopt(curl.get(), CURLOPT_URL, host.c_str());
25-
curl_easy_perform(curl.get());
35+
curl_easy_setopt(curl.get(), CURLOPT_URL, host);
36+
curl_easy_perform(curl.get());
2637
}
2738

2839
int main(int c, char** argv){
29-
bad();
30-
good();
31-
}
40+
bad();
41+
good();
42+
}
43+
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
a

cpp/ql/test/experimental/query-tests/Security/CWE/CWE-295/options

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)