CVS update: samba/source/rpc_server

[Simo Sorce]

On Fri, Dec 21, 2001 at 11:34:34AM -0800, Jeremy Allison wrote:
> 
> Date:	Fri Dec 21 11:34:33 2001
> Author:	jra
> 
> Update of /data/cvs/samba/source/rpc_server
> In directory va:/tmp/cvs-serv555/rpc_server
> 
> Modified Files:
> 	srv_samr_nt.c 
> Log Message:
> Use pdb_free_sam() to free a SAM_ACCOUNT struct, not safe_free(). There
> are pointers withing the SAM_ACCOUNT struct that also need freeing.
> Jeremy.
> 
> 
> Revisions:
> srv_samr_nt.c		1.65 => 1.66
> 	http://www.samba.org/cgi-bin/cvsweb/samba/source/rpc_server/srv_samr_nt.c?r1=1.65&r2=1.66
> 

I do not know if this would be a nice thing or a burden, but I was thinking to 
convert safe_free to support structs and any kind of data we want to define.
Basically we only need to add an int field to any structure we alloc that will identify the structure type. and then make safe_struct_free() have a table of calls to call the correct free function. we will alloc the structure with a macro that set the correct reference.
Do this sound useful? it would reduce greatly the number of free/alloc functions names to remember.


-- 
Simo Sorce       idra at samba.org
-------------------------------
Samba Team http://www.samba.org