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);