New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Denials of service with SLD parameter in WMS requests #4703

Closed
wants to merge 7 commits into
base: master
from

Only allow HTTP and HTTPS for curl request (#4703)

  • Loading branch information...
tbonfort committed Jul 27, 2013
commit 8c5828a189c03bf3d337d0ec66bf295582ccf8f5
View
@@ -529,6 +529,8 @@ int msHTTPExecuteRequests(httpRequestObj *pasReqInfo, int numRequests,
/* set URL, note that curl keeps only a ref to our string buffer */
curl_easy_setopt(http_handle, CURLOPT_URL, pasReqInfo[i].pszGetUrl );
curl_easy_setopt(http_handle, CURLOPT_PROTOCOLS, CURLPROTO_HTTP|CURLPROTO_HTTPS );
/* Set User-Agent (auto-generate if not set by caller */
if (pasReqInfo[i].pszUserAgent == NULL) {
curl_version_info_data *psCurlVInfo;
ProTip! Use n and p to navigate between commits in a pull request.