Fix memory leaked when GETD2 returns without freeing original_filename.

Cisco-Talos · Aug 18, 2021 · 8fd1c66 · 8fd1c66
1 parent a868a7a
commit 8fd1c66
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/libclamav/sis.c b/libclamav/sis.c
@@ -151,14 +151,12 @@ enum {
             smax       = tmp;                                                \
             if (((size_t)-1) == tmp) {                                       \
                 cli_dbgmsg("SIS: Read failed during GETD2\n");               \
-                FREE(alangs);                                                \
-                FREE(ptrs);                                                  \
-                return CL_CLEAN;                                             \
+                status = CL_CLEAN;                                           \
+                goto done;                                                   \
             } else if ((smax += sleft) < 4) {                                \
                 cli_dbgmsg("SIS: EOF\n");                                    \
-                FREE(alangs);                                                \
-                FREE(ptrs);                                                  \
-                return CL_CLEAN;                                             \
+                status = CL_CLEAN;                                           \
+                goto done;                                                   \
             }                                                                \
             pos += smax - sleft;                                             \
             sleft = smax;                                                    \