Don't free struct clsrvconf members rewritein and rewriteout.

They are pointers into static struct hash *rewriteconfs and should live forever. Patch by Fabian Mauchle.
mariux64 · Sep 2, 2013 · edaa77b · edaa77b
1 parent 6fe5339
commit edaa77b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -8,6 +8,8 @@
 	Bug fixes:
 	- The duplication cache is purged properly. Patch by Fabian
 	Mauchle.
+	- Stop freeing a shared piece of memory manifesting itself as a
+	crash when using dynamic discovery. Patch by Fabian Mauchle.
 
 2012-10-25 1.6.2
 	Bug fixes (security):

diff --git a/radsecproxy.c b/radsecproxy.c
@@ -2646,8 +2646,8 @@ void freeclsrvconf(struct clsrvconf *conf) {
 	free(conf->rewriteusername);
     }
     free(conf->dynamiclookupcommand);
-    free(conf->rewritein);
-    free(conf->rewriteout);
+    conf->rewritein=NULL;
+    conf->rewriteout=NULL;
     if (conf->hostports)
 	freehostports(conf->hostports);
     if (conf->lock) {