Details
-
Task
-
Resolution: Unresolved
-
Major - P3
-
None
-
None
-
None
-
Server Security
Description
native_sasl_client_session.cpp has a number of function calls like this:
_saslConversation.reset(new SaslPLAINClientConversation(this));
|
'new' is considered bad form, because it creates objects which for a split second can leak if an exception is thrown. Instead, the code in this file should use make_unique. For example:
_saslConversation = std::make_unique<SaslPLAINClientConversation>(this);
|