gensec: Avoid exposing lp_ctx on the API level.

1 files changed, 10 insertions, 12 deletions

diff --git a/source4/auth/gensec/gensec.c b/source4/auth/gensec/gensec.c
index 1addf937d0..5df9128d0a 100644
--- a/source4/auth/gensec/gensec.c
+++ b/source4/auth/gensec/gensec.c
@@ -40,10 +40,9 @@ _PUBLIC_ struct gensec_security_ops **gensec_security_all(void)
 	return generic_security_ops;
 }
 
-bool gensec_security_ops_enabled(struct gensec_security_ops *ops, 
-				 struct loadparm_context *lp_ctx)
+bool gensec_security_ops_enabled(struct gensec_security_ops *ops, struct gensec_security *security)
 {
-	return lp_parm_bool(lp_ctx, NULL, "gensec", ops->name, ops->enabled);
+	return lp_parm_bool(security->settings->lp_ctx, NULL, "gensec", ops->name, ops->enabled);
 }
 
 /* Sometimes we want to force only kerberos, sometimes we want to
@@ -147,9 +146,8 @@ static const struct gensec_security_ops *gensec_security_by_authtype(struct gens
 	}
 	backends = gensec_security_mechs(gensec_security, mem_ctx);
 	for (i=0; backends && backends[i]; i++) {
-	    	if (!gensec_security_ops_enabled(backends[i], 
-											 gensec_security->settings->lp_ctx))
-		    continue;
+	    	if (!gensec_security_ops_enabled(backends[i], gensec_security))
+				continue;
 		if (backends[i]->auth_type == auth_type) {
 			backend = backends[i];
 			talloc_free(mem_ctx);
@@ -175,7 +173,7 @@ const struct gensec_security_ops *gensec_security_by_oid(struct gensec_security
 	for (i=0; backends && backends[i]; i++) {
 	    	if (gensec_security != NULL && 
 				!gensec_security_ops_enabled(backends[i], 
-											 gensec_security->settings->lp_ctx))
+											 gensec_security))
 		    continue;
 		if (backends[i]->oid) {
 			for (j=0; backends[i]->oid[j]; j++) { 
@@ -205,7 +203,7 @@ const struct gensec_security_ops *gensec_security_by_sasl_name(struct gensec_sec
 	}
 	backends = gensec_security_mechs(gensec_security, mem_ctx);
 	for (i=0; backends && backends[i]; i++) {
-	    	if (!gensec_security_ops_enabled(backends[i], gensec_security->settings->lp_ctx))
+	    	if (!gensec_security_ops_enabled(backends[i], gensec_security))
 		    continue;
 		if (backends[i]->sasl_name 
 		    && (strcmp(backends[i]->sasl_name, sasl_name) == 0)) {
@@ -232,7 +230,7 @@ static const struct gensec_security_ops *gensec_security_by_name(struct gensec_s
 	backends = gensec_security_mechs(gensec_security, mem_ctx);
 	for (i=0; backends && backends[i]; i++) {
 	    	if (gensec_security != NULL && 
-				!gensec_security_ops_enabled(backends[i], gensec_security->settings->lp_ctx))
+				!gensec_security_ops_enabled(backends[i], gensec_security))
 		    continue;
 		if (backends[i]->name 
 		    && (strcmp(backends[i]->name, name) == 0)) {
@@ -278,7 +276,7 @@ const struct gensec_security_ops **gensec_security_by_sasl_list(struct gensec_se
 	 * then looking in the supplied list */
 	for (i=0; backends && backends[i]; i++) {
 	    	if (gensec_security != NULL &&
-				!gensec_security_ops_enabled(backends[i], gensec_security->settings->lp_ctx))
+				!gensec_security_ops_enabled(backends[i], gensec_security))
 		    continue;
 		for (sasl_idx = 0; sasl_names[sasl_idx]; sasl_idx++) {
 			if (!backends[i]->sasl_name ||
@@ -349,7 +347,7 @@ const struct gensec_security_ops_wrapper *gensec_security_by_oid_list(struct gen
 	 * then looking in the supplied list */
 	for (i=0; backends && backends[i]; i++) {
 	    	if (gensec_security != NULL && 
-				!gensec_security_ops_enabled(backends[i], gensec_security->settings->lp_ctx))
+				!gensec_security_ops_enabled(backends[i], gensec_security))
 		    continue;
 		if (!backends[i]->oid) {
 			continue;
@@ -418,7 +416,7 @@ const char **gensec_security_oids_from_ops(struct gensec_security *gensec_securi
 	
 	for (i=0; ops && ops[i]; i++) {
 		if (gensec_security != NULL && 
-			!gensec_security_ops_enabled(ops[i], gensec_security->settings->lp_ctx)) {
+			!gensec_security_ops_enabled(ops[i], gensec_security)) {
 			continue;
 		}
 		if (!ops[i]->oid) {