dovecot: Factor out duplicated verify_callback() function.

dovecot at dovecot.org dovecot at dovecot.org
Fri Jun 29 16:07:33 EEST 2007


details:   http://hg.dovecot.org/dovecot/rev/0a08fa294c3b
changeset: 5830:0a08fa294c3b
user:      Andrey Panin <pazke at donpac.ru>
date:      Fri Jun 29 16:47:36 2007 +0400
description:
Factor out duplicated verify_callback() function.

diffstat:

5 files changed, 33 insertions(+), 33 deletions(-)
src/auth/Makefile.am    |    2 ++
src/auth/mech-login.c   |   18 ++----------------
src/auth/mech-plain.c   |   19 ++-----------------
src/auth/plain-common.c |   20 ++++++++++++++++++++
src/auth/plain-common.h |    7 +++++++

diffs (126 lines):

diff -r 1d73153584d2 -r 0a08fa294c3b src/auth/Makefile.am
--- a/src/auth/Makefile.am	Fri Jun 29 03:39:27 2007 +0300
+++ b/src/auth/Makefile.am	Fri Jun 29 16:47:36 2007 +0400
@@ -63,6 +63,7 @@ dovecot_auth_SOURCES = \
 	mech-rpa.c \
 	mech-apop.c \
 	otp-skey-common.c \
+	plain-common.c \
 	passdb.c \
 	passdb-blocking.c \
 	passdb-bsdauth.c \
@@ -107,6 +108,7 @@ headers = \
 	mech.h \
 	mycrypt.h \
 	otp-skey-common.h \
+	plain-common.h \
 	passdb.h \
 	passdb-blocking.h \
 	passdb-cache.h \
diff -r 1d73153584d2 -r 0a08fa294c3b src/auth/mech-login.c
--- a/src/auth/mech-login.c	Fri Jun 29 03:39:27 2007 +0300
+++ b/src/auth/mech-login.c	Fri Jun 29 16:47:36 2007 +0400
@@ -10,22 +10,8 @@
 #include "mech.h"
 #include "passdb.h"
 #include "safe-memset.h"
+#include "plain-common.h"
 
-static void verify_callback(enum passdb_result result,
-			    struct auth_request *request)
-{
-	switch (result) {
-	case PASSDB_RESULT_OK:
-		auth_request_success(request, NULL, 0);
-		break;
-	case PASSDB_RESULT_INTERNAL_FAILURE:
-		auth_request_internal_failure(request);
-		break;
-	default:
-		auth_request_fail(request);
-		break;
-	}
-}
 
 static void
 mech_login_auth_continue(struct auth_request *request,
@@ -47,7 +33,7 @@ mech_login_auth_continue(struct auth_req
 				  prompt2, strlen(prompt2));
 	} else {
 		char *pass = p_strndup(unsafe_data_stack_pool, data, data_size);
-		auth_request_verify_plain(request, pass, verify_callback);
+		auth_request_verify_plain(request, pass, plain_verify_callback);
 		safe_memset(pass, 0, strlen(pass));
 	}
 }
diff -r 1d73153584d2 -r 0a08fa294c3b src/auth/mech-plain.c
--- a/src/auth/mech-plain.c	Fri Jun 29 03:39:27 2007 +0300
+++ b/src/auth/mech-plain.c	Fri Jun 29 16:47:36 2007 +0400
@@ -4,22 +4,7 @@
 #include "safe-memset.h"
 #include "mech.h"
 #include "passdb.h"
-
-static void verify_callback(enum passdb_result result,
-			    struct auth_request *request)
-{
-	switch (result) {
-	case PASSDB_RESULT_OK:
-		auth_request_success(request, NULL, 0);
-		break;
-	case PASSDB_RESULT_INTERNAL_FAILURE:
-		auth_request_internal_failure(request);
-		break;
-	default:
-		auth_request_fail(request);
-		break;
-	}
-}
+#include "plain-common.h"
 
 static void
 mech_plain_auth_continue(struct auth_request *request,
@@ -71,7 +56,7 @@ mech_plain_auth_continue(struct auth_req
                 auth_request_fail(request);
         } else {
                 auth_request_verify_plain(request, pass,
-                                          verify_callback);
+                                          plain_verify_callback);
 	}
 
         /* make sure it's cleared */
diff -r 1d73153584d2 -r 0a08fa294c3b src/auth/plain-common.c
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/src/auth/plain-common.c	Fri Jun 29 16:47:36 2007 +0400
@@ -0,0 +1,20 @@
+#include "common.h"
+#include "mech.h"
+#include "passdb.h"
+#include "plain-common.h"
+
+void plain_verify_callback(enum passdb_result result,
+			   struct auth_request *request)
+{
+	switch (result) {
+	case PASSDB_RESULT_OK:
+		auth_request_success(request, NULL, 0);
+		break;
+	case PASSDB_RESULT_INTERNAL_FAILURE:
+		auth_request_internal_failure(request);
+		break;
+	default:
+		auth_request_fail(request);
+		break;
+	}
+}
diff -r 1d73153584d2 -r 0a08fa294c3b src/auth/plain-common.h
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/src/auth/plain-common.h	Fri Jun 29 16:47:36 2007 +0400
@@ -0,0 +1,7 @@
+#ifndef __PLAIN_COMMON_H__
+#define __PLAIN_COMMON_H__
+
+void plain_verify_callback(enum passdb_result result,
+			   struct auth_request *request);
+
+#endif	/* __PLAIN_COMMON_H__ */


More information about the dovecot-cvs mailing list