diff --git a/clamav-devel/ChangeLog b/clamav-devel/ChangeLog index f776d00f5..c38e55df5 100644 --- a/clamav-devel/ChangeLog +++ b/clamav-devel/ChangeLog @@ -1,3 +1,7 @@ +Thu Dec 28 15:09:37 GMT 2006 (njh) +---------------------------------- + * libclamav/mbox.c: Some HTML.Phishing.Bank-512 being found (bug 33) + Wed Dec 27 23:14:57 GMT 2006 (njh) ---------------------------------- * libclamav/mbox.c: Code tidy diff --git a/clamav-devel/libclamav/mbox.c b/clamav-devel/libclamav/mbox.c index e2dda40ac..fd6f69ce3 100644 --- a/clamav-devel/libclamav/mbox.c +++ b/clamav-devel/libclamav/mbox.c @@ -16,7 +16,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, * MA 02110-1301, USA. */ -static char const rcsid[] = "$Id: mbox.c,v 1.366 2006/12/27 23:18:49 njh Exp $"; +static char const rcsid[] = "$Id: mbox.c,v 1.367 2006/12/28 15:07:55 njh Exp $"; #ifdef _MSC_VER #include /* only needed in CL_EXPERIMENTAL */ @@ -2032,7 +2032,7 @@ parseEmailHeader(message *m, const char *line, const table_t *rfc821) static mbox_status parseEmailBody(message *messageIn, text *textIn, mbox_ctx *mctx, unsigned int recursion_level) { - mbox_status rc = OK; + mbox_status rc; text *aText = textIn; message *mainMessage = messageIn; fileblob *fb; @@ -2064,6 +2064,7 @@ parseEmailBody(message *messageIn, text *textIn, mbox_ctx *mctx, unsigned int re cli_dbgmsg("Parsing mail file\n"); + rc = OK; mimeType = messageGetMimeType(mainMessage); mimeSubtype = messageGetMimeSubtype(mainMessage); @@ -2179,7 +2180,8 @@ parseEmailBody(message *messageIn, text *textIn, mbox_ctx *mctx, unsigned int re infected = TRUE; break; } - } else if(encodingLine(mainMessage) == t_line->t_next) { + } else if(t_line->t_next && + (encodingLine(mainMessage) == t_line->t_next)) { /* * We look for the next line * since later on we'll skip @@ -2189,7 +2191,8 @@ parseEmailBody(message *messageIn, text *textIn, mbox_ctx *mctx, unsigned int re * which it would have been in * an RFC compliant world */ - cli_dbgmsg("Found MIME attachment before the first MIME section\n"); + cli_dbgmsg("Found MIME attachment before the first MIME section \"%s\"\n", + lineGetData(t_line->t_next->t_line)); if(messageGetEncoding(mainMessage) == NOENCODING) break; }