Browse code

Fix problem of possible confused pointers if large number of recipients given

git-svn-id: file:///var/lib/svn/clamav-devel/trunk/clamav-devel@131 77e5149b-7576-45b1-b177-96237e5ba77b

Nigel Horne authored on 2003/11/29 20:53:44
Showing 3 changed files
... ...
@@ -1,3 +1,8 @@
1
+Sat Nov 29 12:52:21 GMT 2003 (njh)
2
+----------------------------------
3
+  * clamav-milter: Fix problem of possible confused pointers if large number of
4
+	recipients given. Fix by Michael Dankov <misha@btrc.ru>.
5
+
1 6
 Sat Nov 29 04:35:03 CET 2003 (tk)
2 7
 ---------------------------------
3 8
   * clamd: UseProcesses: use processes instead of threads (initial version)
... ...
@@ -48,6 +48,7 @@ Install into /usr/local/sbin/clamav-milter, mode 744
48 48
 See http://www.nmt.edu/~wcolburn/sendmail-8.12.5/libmilter/docs/sample.html
49 49
 
50 50
 Installations for RedHat Linux and it's derivatives such as YellowDog:
51
+	Ensure that you have the sendmail-devel RPM installed
51 52
 	Add to /etc/mail/sendmail.mc:
52 53
 	INPUT_MAIL_FILTER(`clamav', `S=local:/var/run/clamav.sock, F=, T=S:4m;R:4m')dnl
53 54
 	define(`confINPUT_MAIL_FILTERS', `clamav')
... ...
@@ -156,6 +157,9 @@ Changes
156 156
 		Fix warning about root usage
157 157
 0.65d	25/11/03 Handle empty hostname or hostaddr
158 158
 		Fix based on a submission by Michael Dankov <misha@btrc.ru>
159
+0.65e	29/11/03 Fix problem of possible confused pointers if large
160
+		number of recipients given.
161
+		Fix by Michael Dankov <misha@btrc.ru>.
159 162
 
160 163
 BUG REPORTS
161 164
 
... ...
@@ -160,9 +160,15 @@
160 160
  *			Fix warning about root usage
161 161
  *	0.65d	25/11/03 Handle empty hostname or hostaddr
162 162
  *			Fix based on a submission by Michael Dankov <misha@btrc.ru>
163
+ *	0.65e	29/11/03 Fix problem of possible confused pointers if large
164
+ *			number of recipients given.
165
+ *			Fix by Michael Dankov <misha@btrc.ru>.
163 166
  *
164 167
  * Change History:
165 168
  * $Log: clamav-milter.c,v $
169
+ * Revision 1.24  2003/11/29 11:51:19  nigelhorne
170
+ * Fix problem of possible confused pointers if large number of recipients given
171
+ *
166 172
  * Revision 1.23  2003/11/25 05:56:43  nigelhorne
167 173
  * Handle empty hostname or hostaddr
168 174
  *
... ...
@@ -217,9 +223,9 @@
217 217
  * Revision 1.6  2003/09/28 16:37:23  nigelhorne
218 218
  * Added -f flag use MaxThreads if --max-children not set
219 219
  */
220
-static	char	const	rcsid[] = "$Id: clamav-milter.c,v 1.23 2003/11/25 05:56:43 nigelhorne Exp $";
220
+static	char	const	rcsid[] = "$Id: clamav-milter.c,v 1.24 2003/11/29 11:51:19 nigelhorne Exp $";
221 221
 
222
-#define	CM_VERSION	"0.65d"
222
+#define	CM_VERSION	"0.65e"
223 223
 
224 224
 /*#define	CONFDIR	"/usr/local/etc"*/
225 225
 
... ...
@@ -1312,6 +1318,7 @@ clamfi_eom(SMFICTX *ctx)
1312 1312
 			if(&ptr[strlen(*to) + 2] >= &err[i]) {
1313 1313
 				i += 1024;
1314 1314
 				err = realloc(err, i);
1315
+				ptr = strchr(err, '\0');
1315 1316
 			}
1316 1317
 			ptr = strrcpy(ptr, " ");
1317 1318
 			ptr = strrcpy(ptr, *to);