Something is wrong here. If `ssl_servername` is set to `nullptr`, why `release` it later? There's nothing there. This applies to `sni_servername` as well. Further, even if not set to `nullptr`, what is accomplished by the `release`?
Outside scope, but we should change this to support move semantics to avoid making extra copies of the source strings. [ Full content available at: https://github.com/apache/trafficserver/pull/4216 ] This message was relayed via gitbox.apache.org for [email protected]
