Attachment 12168 Details for Bug 11959 – git-am fix for master. Submitted to samba-technical.

[patch] git-am fix for master. Submitted to samba-technical.

0001-s3-krb5-keytab-The-done-label-can-be-jumped-to-with-.patch (text/plain), 1.36 KB, created by Jeremy Allison on 2016-06-08 22:01:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2016-06-08 22:01:42 UTC

Size: 1.36 KB

patch

obsolete

>From 7f98ecb6e208401a311bbede7c94e95316635e8a Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 8 Jun 2016 14:50:59 -0700
>Subject: [PATCH] s3: krb5: keytab - The done label can be jumped to with
> context == NULL.
>
>Ensure we don't crash in this case.
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>---
> source3/libads/kerberos_keytab.c | 18 ++++++++----------
> 1 file changed, 8 insertions(+), 10 deletions(-)
>
>diff --git a/source3/libads/kerberos_keytab.c b/source3/libads/kerberos_keytab.c
>index ae3d80e39..2068d03 100644
>--- a/source3/libads/kerberos_keytab.c
>+++ b/source3/libads/kerberos_keytab.c
>@@ -742,26 +742,24 @@ done:
> 	TALLOC_FREE(oldEntries);
> 	TALLOC_FREE(frame);
> 
>-	{
>+	if (context) {
> 		krb5_keytab_entry zero_kt_entry;
>+		krb5_kt_cursor zero_csr;
>+
> 		ZERO_STRUCT(zero_kt_entry);
>+		ZERO_STRUCT(zero_csr);
>+
> 		if (memcmp(&zero_kt_entry, &kt_entry,
> 				sizeof(krb5_keytab_entry))) {
> 			smb_krb5_kt_free_entry(context, &kt_entry);
> 		}
>-	}
>-	{
>-		krb5_kt_cursor zero_csr;
>-		ZERO_STRUCT(zero_csr);
> 		if ((memcmp(&cursor, &zero_csr,
> 				sizeof(krb5_kt_cursor)) != 0) && keytab) {
> 			krb5_kt_end_seq_get(context, keytab, &cursor);
> 		}
>-	}
>-	if (keytab) {
>-		krb5_kt_close(context, keytab);
>-	}
>-	if (context) {
>+		if (keytab) {
>+			krb5_kt_close(context, keytab);
>+		}
> 		krb5_free_context(context);
> 	}
> 	return ret;
>-- 
>2.8.0.rc3.226.g39d4020
>

Actions: View

Attachments on bug 11959: 12168 | 12173