Author: jra
Date: 2006-03-12 19:16:45 +0000 (Sun, 12 Mar 2006)
New Revision: 14250

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=14250

Log:
Fix coverity bug #107. Resource leak on error path.
Jeremy.

Modified:
   branches/SAMBA_3_0/source/rpc_server/srv_spoolss_nt.c


Changeset:
Modified: branches/SAMBA_3_0/source/rpc_server/srv_spoolss_nt.c
===================================================================
--- branches/SAMBA_3_0/source/rpc_server/srv_spoolss_nt.c       2006-03-12 
18:01:36 UTC (rev 14249)
+++ branches/SAMBA_3_0/source/rpc_server/srv_spoolss_nt.c       2006-03-12 
19:16:45 UTC (rev 14250)
@@ -4862,8 +4862,10 @@
        if((printer=SMB_MALLOC_P(PRINTER_INFO_5))==NULL)
                return WERR_NOMEM;
 
-       if (!construct_printer_info_5(print_hnd, printer, snum))
+       if (!construct_printer_info_5(print_hnd, printer, snum)) {
+               free_printer_info_5(printer);
                return WERR_NOMEM;
+       }
        
        /* check the required size. */  
        *needed += spoolss_size_printer_info_5(printer);

Reply via email to