[PATCH] Stop use after free

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Thu Sep 26 01:51:40 MDT 2013


Hi Volker,

On 26/09/2013 12:48 AM, Volker Lendecke wrote:
> On Wed, Sep 25, 2013 at 04:16:47PM +1000, Alistair Leslie-Hughes wrote:
> While this looks good, it does fix two things. Why don't you
> just talloc_strdup session_info->unix_info->unix_name to
> state->gensec_state? Is session_info->unix_info->unix_name
> something different from the talloc_asprintf args?
This is two files called ntlm_auth.c, one in source3 and other source4.

So my initial though was to make this function the same (which it is 
mostly).  I'll change it to use the talloc_strdup function which makes
more sense.

I'll still finding my way around samba code, so thanks for the advice.


Best Regards
  Alistair Leslie-Hughes


More information about the samba-technical mailing list