[PATCH 2/2, RESEND] source4/libcli: Only set ctemp set on success

Volker Lendecke Volker.Lendecke at SerNet.DE
Tue Apr 23 01:27:50 MDT 2013


On Mon, Apr 22, 2013 at 10:23:16AM -0500, samlang at gmail.com wrote:
> From: Sam Lang <sam.lang at inktank.com>
> 
> If open fails ctemp.out.name probably won't be valid and strdup
> will cause a segv.  Only set the path if open succeeds.
> 
> Signed-off-by: Sam Lang <sam.lang at inktank.com>

Reviewed-by: Volker Lendecke <vl at samba.org>

Please another team member also review this and if it is
okay and someone has the time please push.

Thanks,

Volker Lendecke

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de


More information about the samba-technical mailing list