Changeset 223 for branches/samba-3.3.x/source/winbindd

Timestamp:

May 24, 2009, 7:51:24 AM (16 years ago)

Author:

Herwig Bauernfeind

Message:

Update Samba 3.3 branch to 3.3.3

Location:

branches/samba-3.3.x/source/winbindd

Files:

• winbindd_cm.c (modified) (1 diff)
• winbindd_pam.c (modified) (1 diff)
• winbindd_rpc.c (modified) (1 diff)

branches/samba-3.3.x/source/winbindd/winbindd_cm.c

@@ -1764 +1764 @@
         WERROR werr;
         TALLOC_CTX              *mem_ctx = NULL;
-        struct rpc_pipe_client  *cli;
+        struct rpc_pipe_client  *cli = NULL;
         POLICY_HND pol;
         union dssetup_DsRoleInfo info;

branches/samba-3.3.x/source/winbindd/winbindd_pam.c

@@ -1855 +1855 @@
         if (state->request.data.auth_crap.lm_resp_len > sizeof(state->request.data.auth_crap.lm_resp)
                 || state->request.data.auth_crap.nt_resp_len > sizeof(state->request.data.auth_crap.nt_resp)) {
-                if (!state->request.flags & WBFLAG_BIG_NTLMV2_BLOB ||
+                if (!(state->request.flags & WBFLAG_BIG_NTLMV2_BLOB) ||
                      state->request.extra_len != state->request.data.auth_crap.nt_resp_len) {
                         DEBUG(0, ("winbindd_pam_auth_crap: invalid password length %u/%u\n",

branches/samba-3.3.x/source/winbindd/winbindd_rpc.c

@@ -856 +856 @@
 
                 for (r=0; r<tmp_names.count; r++) {
-                        (*names)[i+r] = fill_domain_username_talloc(mem_ctx,
-                                                domain->name,
-                                                tmp_names.names[r].string,
-                                                true);
-                        (*name_types)[i+r] = tmp_types.ids[r];
-                }
-
-                total_names += tmp_names.count;
+                        if (tmp_types.ids[r] == SID_NAME_UNKNOWN) {
+                                continue;
+                        }
+                        (*names)[total_names] = fill_domain_username_talloc(
+                                mem_ctx, domain->name,
+                                tmp_names.names[r].string, true);
+                        (*name_types)[total_names] = tmp_types.ids[r];
+                        total_names += 1;
+                }
         }