oysteikt/heimdal
0
0
Fork 0
Code Issues 5 Pull Requests Releases Activity

s4/heimdal/lib/krb5/pac.c: Align PAC buffers to match Windows

Browse Source 
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14642

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
(cherry picked from Samba commit 28a5a586c8e9cd155d676dcfcb81a2587ace99d1)
This commit is contained in:
Joseph Sutton
2021-08-11 13:27:11 +12:00
committed by Luke Howard
parent c7a8ee1439
commit 24a7a82e82
1 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lib/krb5/pac.c
View File

@@ -62,10 +62,12 @@ struct krb5_pac_data {
#define PACTYPE_SIZE 8 #define PACTYPE_SIZE 8
#define PAC_INFO_BUFFER_SIZE 16 #define PAC_INFO_BUFFER_SIZE 16
#define PAC_LOGON_INFO 1
#define PAC_SERVER_CHECKSUM 6 #define PAC_SERVER_CHECKSUM 6
#define PAC_PRIVSVR_CHECKSUM 7 #define PAC_PRIVSVR_CHECKSUM 7
#define PAC_LOGON_NAME 10 #define PAC_LOGON_NAME 10
#define PAC_CONSTRAINED_DELEGATION 11 #define PAC_CONSTRAINED_DELEGATION 11
#define PAC_UPN_DNS_INFO 12
#define PAC_TICKET_CHECKSUM 16 #define PAC_TICKET_CHECKSUM 16
#define CHECK(r,f,l) \ #define CHECK(r,f,l) \
@@ -1199,7 +1201,17 @@ _krb5_pac_sign(krb5_context context,
ret = krb5_enomem(context); ret = krb5_enomem(context);
goto out; goto out;
} }
/* XXX if not aligned, fill_zeros */
if (p->pac->buffers[i].type == PAC_LOGON_INFO
|| p->pac->buffers[i].type == PAC_UPN_DNS_INFO)
{
uint32_t rounded = (len + PAC_ALIGNMENT - 1) / PAC_ALIGNMENT
* PAC_ALIGNMENT;
uint32_t remaining = rounded - len;
CHECK(ret, fill_zeros(context, spdata, remaining), out);
len = rounded;
}
} }
/* write header */ /* write header */