I agree with vl's #if 0 here, and am not quite sure what I was

thinking with regard to the original code.

Let's keep samba simple, and just remove it.

Andrew Bartlett
(This used to be commit 156cf8bede52e1d33be980fd8841367253a60b9e)

diff --git a/source3/passdb/pdb_get_set.c b/source3/passdb/pdb_get_set.c
index 46c49be8b1166e95d716d5bf58592903530004f6..4a5a5759d4f63e568bcbe77aa8e7936c59b57797 100644 (file)
--- a/source3/passdb/pdb_get_set.c
+++ b/source3/passdb/pdb_get_set.c
@@ -1045,19 +1045,6 @@ BOOL pdb_set_backend_private_data (SAM_ACCOUNT *sampass, void *private_data,
        if (!sampass)
                return False;
 
-#if 0
-       /* With this check backend_private_data_free_fn is *never* set
-          as the methods are never set anywhere. What is this
-          supposed to do ????
-
-          Volker
-       */
-
-       /* does this backend 'own' this SAM_ACCOUNT? */
-       if (my_methods != sampass->private.backend_private_methods)
-               return False;
-#endif
-
        if (sampass->private.backend_private_data && sampass->private.backend_private_data_free_fn) {
                sampass->private.backend_private_data_free_fn(&sampass->private.backend_private_data);
        }