]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
files: append data on closing even with FILE_NOSTORE 12719/head 12725/head
authorPhilippe Antoine <pantoine@oisf.net>
Tue, 4 Mar 2025 15:30:19 +0000 (16:30 +0100)
committerPhilippe Antoine <pantoine@oisf.net>
Thu, 6 Mar 2025 06:27:41 +0000 (07:27 +0100)
Ticket: 7577

When HTTP1 post multipart handles a small file, it will call
HTPFileClose with some data
This data needs to be appended to the streaming buffer for usage
by file.data keyword even if we do not end up storing the file

(cherry picked from commit f68e2f5537c95388858129aa1b076e976354584a)

src/util-file.c

index 89ef50c1d4ebb633412678e0c1d857942cf0f3a3..47b169ce72b97e7fc8f2c227f2f289b62ad973f3 100644 (file)
@@ -1018,11 +1018,10 @@ int FileCloseFilePtr(File *ff, const StreamingBufferConfig *sbcfg, const uint8_t
                 SCLogDebug("file %p data %p data_len %u", ff, data, data_len);
                 SCSha256Update(ff->sha256_ctx, data, data_len);
             }
-        } else {
-            if (AppendData(sbcfg, ff, data, data_len) != 0) {
-                ff->state = FILE_STATE_ERROR;
-                SCReturnInt(-1);
-            }
+        }
+        if (AppendData(sbcfg, ff, data, data_len) != 0) {
+            ff->state = FILE_STATE_ERROR;
+            SCReturnInt(-1);
         }
     }