Singularity-Viewer/SingularityViewer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't feed empty, non-working callback to CURL

Browse Source
This commit is contained in:
Siana Gearz
2012-08-07 16:48:22 +02:00
parent 6ed9a62501
commit 96a4e16a4b
5 changed files with 10 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
indra/llmessage/llhttpclient.cpp
View File

@@ -42,8 +42,6 @@
const F32 HTTP_REQUEST_EXPIRY_SECS = 60.0f; const F32 HTTP_REQUEST_EXPIRY_SECS = 60.0f;
LLURLRequest::SSLCertVerifyCallback LLHTTPClient::mCertVerifyCallback = NULL;
//////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////
// Responder class moved to LLCurl // Responder class moved to LLCurl
@@ -206,11 +204,6 @@ namespace
LLPumpIO* theClientPump = NULL; LLPumpIO* theClientPump = NULL;
} }
void LLHTTPClient::setCertVerifyCallback(LLURLRequest::SSLCertVerifyCallback callback)
{
LLHTTPClient::mCertVerifyCallback = callback;
}
static void request( static void request(
const std::string& url, const std::string& url,
LLURLRequest::ERequestAction method, LLURLRequest::ERequestAction method,
@@ -234,7 +227,7 @@ static void request(
return ; return ;
} }
req->setSSLVerifyCallback(LLHTTPClient::getCertVerifyCallback(), (void *)req); req->checkRootCertificate(true);
lldebugs << LLURLRequest::actionAsVerb(method) << " " << url << " " lldebugs << LLURLRequest::actionAsVerb(method) << " " << url << " "

6
indra/llmessage/llhttpclient.h
View File

@@ -153,12 +153,6 @@ public:
///< for testing ///< for testing
static LLPumpIO &getPump(); static LLPumpIO &getPump();
///< Hippo special ///< Hippo special
static void setCertVerifyCallback(LLURLRequest::SSLCertVerifyCallback callback);
static LLURLRequest::SSLCertVerifyCallback getCertVerifyCallback() { return mCertVerifyCallback; }
protected:
static LLURLRequest::SSLCertVerifyCallback mCertVerifyCallback;
}; };
#endif // LL_LLHTTPCLIENT_H #endif // LL_LLHTTPCLIENT_H

38
indra/llmessage/llurlrequest.cpp
View File

@@ -100,37 +100,6 @@ LLURLRequestDetail::~LLURLRequestDetail()
mLastRead = NULL; mLastRead = NULL;
} }
void LLURLRequest::setSSLVerifyCallback(SSLCertVerifyCallback callback, void *param)
{
mDetail->mSSLVerifyCallback = callback;
mDetail->mCurlRequest->setSSLCtxCallback(LLURLRequest::_sslCtxCallback, (void *)this);
mDetail->mCurlRequest->setopt(CURLOPT_SSL_VERIFYPEER, true);
mDetail->mCurlRequest->setopt(CURLOPT_SSL_VERIFYHOST, 2);
}
// _sslCtxFunction
// Callback function called when an SSL Context is created via CURL
// used to configure the context for custom cert validation
CURLcode LLURLRequest::_sslCtxCallback(CURL * curl, void *sslctx, void *param)
{
LLURLRequest *req = (LLURLRequest *)param;
if(req == NULL || req->mDetail->mSSLVerifyCallback == NULL)
{
SSL_CTX_set_cert_verify_callback((SSL_CTX *)sslctx, NULL, NULL);
return CURLE_OK;
}
SSL_CTX * ctx = (SSL_CTX *) sslctx;
// disable any default verification for server certs
SSL_CTX_set_verify(ctx, SSL_VERIFY_NONE, NULL);
// set the verification callback.
SSL_CTX_set_cert_verify_callback(ctx, req->mDetail->mSSLVerifyCallback, (void *)req);
// the calls are void
return CURLE_OK;
}
/** /**
* class LLURLRequest * class LLURLRequest
*/ */
@@ -196,6 +165,13 @@ void LLURLRequest::addHeader(const char* header)
mDetail->mCurlRequest->slist_append(header); mDetail->mCurlRequest->slist_append(header);
} }
void LLURLRequest::checkRootCertificate(bool check)
{
mDetail->mCurlRequest->setopt(CURLOPT_SSL_VERIFYPEER, (check? TRUE : FALSE));
mDetail->mCurlRequest->setoptString(CURLOPT_ENCODING, "");
}
void LLURLRequest::setBodyLimit(U32 size) void LLURLRequest::setBodyLimit(U32 size)
{ {
mDetail->mBodyLimit = size; mDetail->mBodyLimit = size;

4
indra/llmessage/llurlrequest.h
View File

@@ -143,7 +143,7 @@ public:
* Set whether request will check that remote server * Set whether request will check that remote server
* certificates are signed by a known root CA when using HTTPS. * certificates are signed by a known root CA when using HTTPS.
*/ */
void setSSLVerifyCallback(SSLCertVerifyCallback callback, void * param); void checkRootCertificate(bool check);
/** /**
@@ -226,8 +226,6 @@ protected:
S32 mRequestTransferedBytes; S32 mRequestTransferedBytes;
S32 mResponseTransferedBytes; S32 mResponseTransferedBytes;
static CURLcode _sslCtxCallback(CURL * curl, void *sslctx, void *param);
private: private:
/** /**
* @brief Initialize the object. Called during construction. * @brief Initialize the object. Called during construction.

2
indra/newview/hipporestrequest.cpp
View File

@@ -256,7 +256,7 @@ static void request(const std::string &url,
LLPumpIO::chain_t chain; LLPumpIO::chain_t chain;
LLURLRequest *req = new LLURLRequest(method, url); LLURLRequest *req = new LLURLRequest(method, url);
req->setSSLVerifyCallback(LLHTTPClient::getCertVerifyCallback(), (void *)req); req->checkRootCertificate(true);
/* /*
// Insert custom headers if the caller sent any // Insert custom headers if the caller sent any