s3-auth remove extra from auth3_session_info
authorAndrew Bartlett <abartlet@samba.org>
Mon, 18 Jul 2011 00:20:25 +0000 (10:20 +1000)
committerAndrew Bartlett <abartlet@samba.org>
Tue, 19 Jul 2011 23:17:12 +0000 (09:17 +1000)
Signed-off-by: Andrew Tridgell <tridge@samba.org>
source3/auth/auth_util.c
source3/include/auth.h

index 379cdf42ceaefd1c7e660e05045e296a2ba27b4b..421768d96d6753c46f33e3cb5161f4ad80b4620c 100644 (file)
@@ -911,8 +911,12 @@ NTSTATUS make_session_info_from_username(TALLOC_CTX *mem_ctx,
  * guest.
  *
  * This is a lossy conversion.  Variables known to be lost so far
- * include: nss_token (not needed because the only read doesn't happen
+ * include:
+ *
+ * - nss_token (not needed because the only read doesn't happen
  * for the GUEST user, as this routine populates ->security_token
+ *
+ * - extra (not needed because the guest account mut have a valid RID per the output of get_guest_info3())
  */
 static struct auth_serversupplied_info *copy_session_info_serverinfo(TALLOC_CTX *mem_ctx,
                                                              const struct auth3_session_info *src)
@@ -967,7 +971,6 @@ static struct auth_serversupplied_info *copy_session_info_serverinfo(TALLOC_CTX
                TALLOC_FREE(dst);
                return NULL;
        }
-       dst->extra = src->extra;
 
        dst->unix_name = talloc_strdup(dst, src->unix_info->unix_name);
        if (!dst->unix_name) {
@@ -1026,7 +1029,6 @@ static struct auth3_session_info *copy_serverinfo_session_info(TALLOC_CTX *mem_c
                TALLOC_FREE(dst);
                return NULL;
        }
-       dst->extra = src->extra;
 
        dst->unix_info = talloc_zero(dst, struct auth_user_info_unix);
        if (!dst->unix_info) {
@@ -1098,7 +1100,6 @@ struct auth3_session_info *copy_session_info(TALLOC_CTX *mem_ctx,
                TALLOC_FREE(dst);
                return NULL;
        }
-       dst->extra = src->extra;
 
        if (src->unix_info) {
                dst->unix_info = talloc_zero(dst, struct auth_user_info_unix);
index dc6d344cb9b52a8cb8561afa3185163031b641f7..da3a443a163b476614021525a2a8604401d13bfd 100644 (file)
@@ -96,13 +96,6 @@ struct auth3_session_info {
 
        struct netr_SamInfo3 *info3;
 
-       /* this structure is filled *only* in pathological cases where the user
-        * sid or the primary group sid are not sids of the domain. Normally
-        * this happens only for unix accounts that have unix domain sids.
-        * This is checked only when info3.rid and/or info3.primary_gid are set
-        * to the special invalid value of 0xFFFFFFFF */
-       struct extra_auth_info extra;
-
        struct auth_user_info_unix *unix_info;
 };