Side by Side Diff: src/DefaultWebRequestCurl.cpp
Issue 10310030:
Convert references to FileSystem & Co. into shared pointers (avoid use after free) (Closed)
Patch Set:
Created April 18, 2013, 11:59 a.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« src/DefaultErrorCallback.cpp ('k')
|
src/FileSystemJsObject.cpp » ('j')
|
src/JsEngine.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| OLD | NEW |
|---|---|
| 1 #include <sstream> | 1 #include <sstream> |
| 2 #include <cctype> | 2 #include <cctype> |
| 3 #include <algorithm> | 3 #include <algorithm> |
| 4 #include <curl/curl.h> | 4 #include <curl/curl.h> |
| 5 #include <AdblockPlus/WebRequest.h> | 5 #include <AdblockPlus/DefaultWebRequest.h> |
| 6 | 6 |
| 7 namespace | 7 namespace |
| 8 { | 8 { |
| 9 struct HeaderData | 9 struct HeaderData |
| 10 { | 10 { |
| 11 int status; | 11 int status; |
| 12 bool expectingStatus; | 12 bool expectingStatus; |
| 13 std::vector<std::string> headers; | 13 std::vector<std::string> headers; |
| 14 | 14 |
| 15 HeaderData() | 15 HeaderData() |
| (...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 164 } | 164 } |
| 165 } | 165 } |
| 166 } | 166 } |
| 167 | 167 |
| 168 if (headerList) | 168 if (headerList) |
| 169 curl_slist_free_all(headerList); | 169 curl_slist_free_all(headerList); |
| 170 curl_easy_cleanup(curl); | 170 curl_easy_cleanup(curl); |
| 171 } | 171 } |
| 172 return result; | 172 return result; |
| 173 } | 173 } |
| OLD | NEW |
« no previous file with comments
|
« src/DefaultErrorCallback.cpp ('k')
|
src/FileSystemJsObject.cpp » ('j')
|
src/JsEngine.cpp » ('J')
