Re: [1/2] git commit: TS-2802: Additional fixups. Changed NetVCOptions to use ats_scoped_str instead of a raw pointer. Added comments.

[James Peach]

On Sep 19, 2014, at 8:38 AM, Alan M. Carroll <a...@network-geographics.com> 
wrote:

> Friday, September 19, 2014, 10:12:29 AM, you wrote:
> 
> 
>>> +     sni_servername.release(); // otherwise we'll free the source string.
> 
> 
>> Why don't you want to free the source string in this case?
> 
> Because it's still in the source object and therefore it would be freed again 
> at some point in the future.

Oh, because of the memcpy ... yeh