From: Alan T. DeKok <aland@freeradius.org>
Date: Mon, 18 Feb 2013 19:07:04 +0000 (-0500)
Subject: No longer need to free conf file stuff manually.
X-Git-Tag: release_2_2_1~148
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c9333d89d516fef52d5aea0bb8b5ced36bf5d409;p=thirdparty%2Ffreeradius-server.git

No longer need to free conf file stuff manually.
---

diff --git a/src/modules/rlm_sql/rlm_sql.c b/src/modules/rlm_sql/rlm_sql.c
index d51857f22ef..9d0328f626b 100644
--- a/src/modules/rlm_sql/rlm_sql.c
+++ b/src/modules/rlm_sql/rlm_sql.c
@@ -814,28 +814,6 @@ static int rlm_sql_detach(void *instance)
 			free(inst->config->xlat_name);
 		}
 
-		/*
-		 *	Free up dynamically allocated string pointers.
-		 */
-		for (i = 0; module_config[i].name != NULL; i++) {
-			char **p;
-			if (module_config[i].type != PW_TYPE_STRING_PTR) {
-				continue;
-			}
-
-			/*
-			 *	Treat 'config' as an opaque array of bytes,
-			 *	and take the offset into it.  There's a
-			 *      (char*) pointer at that offset, and we want
-			 *	to point to it.
-			 */
-			p = (char **) (((char *)inst->config) + module_config[i].offset);
-			if (!*p) { /* nothing allocated */
-				continue;
-			}
-			free(*p);
-			*p = NULL;
-		}
 		/*
 		 *	Catch multiple instances of the module.
 		 */