Author: jra Date: 2006-06-15 02:06:46 +0000 (Thu, 15 Jun 2006) New Revision: 16242
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=16242 Log: Fix Klocwork #106 and others like it. Make 2 important changes. pdb_get_methods() returning NULL is a *fatal* error. Don't try and cope with it just call smb_panic. This removes a *lot* of pointless "if (!pdb)" handling code. Secondly, ensure that if samu_init() fails we *always* back out of a function. That way we are never in a situation where the pdb_XXX() functions need to start with a "if (sampass)" test - this was just bad design, not defensive programming. Jeremy. Modified: trunk/source/auth/auth_util.c trunk/source/pam_smbpass/pam_smb_acct.c trunk/source/pam_smbpass/pam_smb_auth.c trunk/source/passdb/passdb.c trunk/source/passdb/pdb_get_set.c trunk/source/passdb/pdb_interface.c trunk/source/rpc_server/srv_netlog_nt.c trunk/source/smbd/lanman.c trunk/source/utils/smbpasswd.c Changeset: Sorry, the patch is too large (1947 lines) to include; please use WebSVN to see it! WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=16242
