From 47d93e4f65493977217cfed53ff68eb926cf9bb7 Mon Sep 17 00:00:00 2001
From: Nat Goodspeed <nat@lindenlab.com>
Date: Tue, 19 Jul 2016 14:08:43 -0400
Subject: [PATCH] DRTVWR-418: Remove rogue getMessage() from llsecapi.h
 exceptions.

The LLProtectedDataException and LLCertException exception classes didn't used
to be derived from std::exception, so they followed their own getMessage()
convention instead of the standard what() convention. Now that they're derived
from std::exception, remove getMessage() and change its few consumers to use
what() instead. Thanks NickyD for suggesting.
---
 indra/newview/llappcorehttp.cpp | 4 ++--
 indra/newview/llsecapi.cpp      | 2 +-
 indra/newview/llsecapi.h        | 2 --
 3 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/indra/newview/llappcorehttp.cpp b/indra/newview/llappcorehttp.cpp
index 49291ea5642..dbcae57de76 100644
--- a/indra/newview/llappcorehttp.cpp
+++ b/indra/newview/llappcorehttp.cpp
@@ -534,7 +534,7 @@ LLCore::HttpStatus LLAppCoreHttp::sslVerify(const std::string &url,
 		// somewhat clumsy, as we may run into errors that do not map directly to curl
 		// error codes.  Should be refactored with login refactoring, perhaps.
 		result = LLCore::HttpStatus(LLCore::HttpStatus::EXT_CURL_EASY, CURLE_SSL_CACERT);
-		result.setMessage(cert_exception.getMessage());
+		result.setMessage(cert_exception.what());
 		LLPointer<LLCertificate> cert = cert_exception.getCert();
 		cert->ref(); // adding an extra ref here
 		result.setErrorData(cert.get());
@@ -544,7 +544,7 @@ LLCore::HttpStatus LLAppCoreHttp::sslVerify(const std::string &url,
 	catch (LLCertException &cert_exception)
 	{
 		result = LLCore::HttpStatus(LLCore::HttpStatus::EXT_CURL_EASY, CURLE_SSL_PEER_CERTIFICATE);
-		result.setMessage(cert_exception.getMessage());
+		result.setMessage(cert_exception.what());
 		LLPointer<LLCertificate> cert = cert_exception.getCert();
 		cert->ref(); // adding an extra ref here
 		result.setErrorData(cert.get());
diff --git a/indra/newview/llsecapi.cpp b/indra/newview/llsecapi.cpp
index c27709a57bd..e8715707865 100644
--- a/indra/newview/llsecapi.cpp
+++ b/indra/newview/llsecapi.cpp
@@ -64,7 +64,7 @@ void initializeSecHandler()
 		}
 		catch (LLProtectedDataException e)
 		{
-			exception_msg = e.getMessage();
+			exception_msg = e.what();
 		}
 	}
 	if (!exception_msg.empty())  // an exception was thrown.
diff --git a/indra/newview/llsecapi.h b/indra/newview/llsecapi.h
index 55c6d95cd8e..535a1126381 100644
--- a/indra/newview/llsecapi.h
+++ b/indra/newview/llsecapi.h
@@ -124,7 +124,6 @@ struct LLProtectedDataException: public std::runtime_error
 	{
 		LL_WARNS("SECAPI") << "Protected Data Error: " << msg << LL_ENDL;
 	}
-	std::string getMessage() const { return what(); }
 };
 
 // class LLCertificate
@@ -345,7 +344,6 @@ class LLCertException: public std::runtime_error
 	}
 	virtual ~LLCertException() throw() {}
 	LLPointer<LLCertificate> getCert() const { return mCert; }
-	std::string getMessage() const { return what(); }
 protected:
 	LLPointer<LLCertificate> mCert;
 };
-- 
GitLab