Attachment 11446 Details for Bug 11502 – patch for 4.3

[patch] patch for 4.3

43.patch (text/plain), 3.16 KB, created by Andreas Schneider on 2015-09-17 14:02:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andreas Schneider

Created: 2015-09-17 14:02:28 UTC

Size: 3.16 KB

patch

obsolete

>From c4b381e3c5a088813110b9db5fb5828ed14f4b8d Mon Sep 17 00:00:00 2001
>From: Andreas Schneider <asn@samba.org>
>Date: Tue, 8 Sep 2015 16:48:08 +0200
>Subject: [PATCH] pam_winbind: Fix a segfault if initialization fails
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=11502
>
>Signed-off-by: Andreas Schneider <asn@samba.org>
>Reviewed-by: Michael Adam <obnox@samba.org>
>
>Autobuild-User(master): Michael Adam <obnox@samba.org>
>Autobuild-Date(master): Tue Sep  8 21:39:21 CEST 2015 on sn-devel-104
>
>(cherry picked from commit 7d84cd6e40024fd361ea21635f7befed40f0e41f)
>---
> nsswitch/pam_winbind.c | 19 ++++++++-----------
> 1 file changed, 8 insertions(+), 11 deletions(-)
>
>diff --git a/nsswitch/pam_winbind.c b/nsswitch/pam_winbind.c
>index 1e16741..a2d9f3b 100644
>--- a/nsswitch/pam_winbind.c
>+++ b/nsswitch/pam_winbind.c
>@@ -2489,7 +2489,7 @@ static int _pam_delete_cred(pam_handle_t *pamh, int flags,
> 
> 	retval = _pam_winbind_init_context(pamh, flags, argc, argv, type, &ctx);
> 	if (retval) {
>-		goto out;
>+		return retval;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("_pam_delete_cred", ctx);
>@@ -2625,7 +2625,7 @@ int pam_sm_authenticate(pam_handle_t *pamh, int flags,
> 	retval = _pam_winbind_init_context(pamh, flags, argc, argv,
> 					   PAM_WINBIND_AUTHENTICATE, &ctx);
> 	if (retval) {
>-		goto out;
>+		return retval;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("pam_sm_authenticate", ctx);
>@@ -2777,7 +2777,7 @@ int pam_sm_setcred(pam_handle_t *pamh, int flags,
> 	ret = _pam_winbind_init_context(pamh, flags, argc, argv,
> 					PAM_WINBIND_SETCRED, &ctx);
> 	if (ret) {
>-		goto out;
>+		return ret;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("pam_sm_setcred", ctx);
>@@ -2808,8 +2808,6 @@ int pam_sm_setcred(pam_handle_t *pamh, int flags,
> 			break;
> 	}
> 
>- out:
>-
> 	_PAM_LOG_FUNCTION_LEAVE("pam_sm_setcred", ctx, ret);
> 
> 	TALLOC_FREE(ctx);
>@@ -2833,7 +2831,7 @@ int pam_sm_acct_mgmt(pam_handle_t *pamh, int flags,
> 	ret = _pam_winbind_init_context(pamh, flags, argc, argv,
> 					PAM_WINBIND_ACCT_MGMT, &ctx);
> 	if (ret) {
>-		goto out;
>+		return ret;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("pam_sm_acct_mgmt", ctx);
>@@ -2929,7 +2927,7 @@ int pam_sm_open_session(pam_handle_t *pamh, int flags,
> 	ret = _pam_winbind_init_context(pamh, flags, argc, argv,
> 					PAM_WINBIND_OPEN_SESSION, &ctx);
> 	if (ret) {
>-		goto out;
>+		return ret;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("pam_sm_open_session", ctx);
>@@ -2938,7 +2936,7 @@ int pam_sm_open_session(pam_handle_t *pamh, int flags,
> 		/* check and create homedir */
> 		ret = _pam_mkhomedir(ctx);
> 	}
>- out:
>+
> 	_PAM_LOG_FUNCTION_LEAVE("pam_sm_open_session", ctx, ret);
> 
> 	TALLOC_FREE(ctx);
>@@ -2956,12 +2954,11 @@ int pam_sm_close_session(pam_handle_t *pamh, int flags,
> 	ret = _pam_winbind_init_context(pamh, flags, argc, argv,
> 					PAM_WINBIND_CLOSE_SESSION, &ctx);
> 	if (ret) {
>-		goto out;
>+		return ret;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("pam_sm_close_session", ctx);
> 
>-out:
> 	_PAM_LOG_FUNCTION_LEAVE("pam_sm_close_session", ctx, ret);
> 
> 	TALLOC_FREE(ctx);
>@@ -3043,7 +3040,7 @@ int pam_sm_chauthtok(pam_handle_t * pamh, int flags,
> 	ret = _pam_winbind_init_context(pamh, flags, argc, argv,
> 					PAM_WINBIND_CHAUTHTOK, &ctx);
> 	if (ret) {
>-		goto out;
>+		return ret;
> 	}
> 
> 	_PAM_LOG_FUNCTION_ENTER("pam_sm_chauthtok", ctx);
>-- 
>2.5.2
>

Flags:

obnox: review+

Actions: View

Attachments on bug 11502: 11445 | 11446