From: Jean-Marc Pigeon (Delson) <jmp@safe.ca>
Date: Mon, 14 Jul 2025 22:15:55 +0000 (-0400)
Subject: scarmt.c no memory leak
X-Git-Tag: tag-0.14~94
X-Git-Url: https://jmp-git.ovh.safe.ca/?a=commitdiff_plain;h=f35b3c3ebbb408ae5500099d9a43fc2950ef19ca;p=jmp%2Fmailleur

scarmt.c no memory leak
---

diff --git a/Makefile.dbg b/Makefile.dbg
index 25abf78..bf201df 100644
--- a/Makefile.dbg
+++ b/Makefile.dbg
@@ -178,6 +178,16 @@ dbgscar:  clean debug
 		   -f					\
 		   $(DBGPAR)
 
+valscar:  clean debug
+	  @						\
+	      valgrind 					\
+		--leak-check=full                       \
+                --show-leak-kinds=$(VALKIND) 		\
+	  						\
+	        bin/scarmt				\
+		   -f					\
+		   $(DBGPAR)
+
 onefeed	:  debug
 TODO	=  `basename -a 				\
 	   $(TESTDIR)/var/spool/$(APPNAME)/queue/*.todo|\
diff --git a/app/scarmt.c b/app/scarmt.c
index 82a17bf..faa1f98 100644
--- a/app/scarmt.c
+++ b/app/scarmt.c
@@ -303,7 +303,8 @@ while (proceed==true) {
     case 8      :       //let continue
       scanref.logptr=log_closelog(scanref.logptr);
       scanref.sqlptr=sql_closesql(scanref.sqlptr);
-      phase=1;          //looping quickly to do next batch
+      if (foreground==false)
+        phase=1;        //looping quickly to do next batch
       break;
     default     :       //SAFE Guard
       proceed=false;
diff --git a/lib/subafn.c b/lib/subafn.c
index 22a843f..9da2d0e 100644
--- a/lib/subafn.c
+++ b/lib/subafn.c
@@ -491,7 +491,6 @@ if (afnnum!=(AFNTYP *)0) {
   int i;
   char revers[200];
 
-  reversip=strdup("");
   (void) memset(revers,'\000',sizeof(revers));
   switch (afnnum->afntype) {
     case AF_INET6       :
diff --git a/lib/unisql.c b/lib/unisql.c
index a413ea7..26f25f9 100644
--- a/lib/unisql.c
+++ b/lib/unisql.c
@@ -150,6 +150,7 @@ PUBLIC SRVTYP *sql_freesrv(SRVTYP *smtpdata)
 
 {
 if (smtpdata!=(SRVTYP *)0) {
+  smtpdata->reverse=rou_freestr(smtpdata->reverse);
   smtpdata->listing=rou_freestr(smtpdata->listing);
   smtpdata->rmtip=rou_freestr(smtpdata->rmtip);
   (void) free(smtpdata);