From: Victor Julien <vjulien@oisf.net>
Date: Wed, 23 Nov 2022 13:42:21 +0000 (+0100)
Subject: smtp/files: don't modify prev file on open failure
X-Git-Tag: suricata-6.0.9~14
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2f35376482770e880d0cc0d3eb1d6e19e2536b86;p=thirdparty%2Fsuricata.git

smtp/files: don't modify prev file on open failure

(cherry picked from commit 9f4dd4fc56a8885bdff0714352dffa5b897f5979)
---

diff --git a/src/app-layer-smtp.c b/src/app-layer-smtp.c
index b20659c198..414a2c7d30 100644
--- a/src/app-layer-smtp.c
+++ b/src/app-layer-smtp.c
@@ -446,8 +446,9 @@ int SMTPProcessDataChunk(const uint8_t *chunk, uint32_t len,
                         (uint8_t *) chunk, len, flags) != 0) {
                 ret = MIME_DEC_ERR_DATA;
                 SCLogDebug("FileOpenFile() failed");
+            } else {
+                SMTPNewFile(smtp_state->curr_tx, files->tail);
             }
-            SMTPNewFile(smtp_state->curr_tx, files->tail);
 
             /* If close in the same chunk, then pass in empty bytes */
             if (state->body_end) {