From 4b5130d50af9698a97800816f88614171220fe6f Mon Sep 17 00:00:00 2001
From: Micah Snyder <micasnyd@cisco.com>
Date: Mon, 3 Feb 2025 13:25:31 -0500
Subject: [PATCH] Tests: remove dead code

Remove check for 'srcdir' and 'unrar_disabled' variables.
These were only used by legacy Automake tooling.

Resolves: https://github.com/Cisco-Talos/clamav/issues/1447
---
 unit_tests/check_bytecode.c | 10 +++-------
 unit_tests/check_clamav.c   | 15 ++-------------
 2 files changed, 5 insertions(+), 20 deletions(-)

diff --git a/unit_tests/check_bytecode.c b/unit_tests/check_bytecode.c
index 36bbc5f719..63fdff84e7 100644
--- a/unit_tests/check_bytecode.c
+++ b/unit_tests/check_bytecode.c
@@ -501,17 +501,13 @@ END_TEST
 
 static void runload(const char *dbname, struct cl_engine *engine, unsigned signoexp)
 {
-    const char *srcdir = getenv("srcdir");
     char *str;
     unsigned signo = 0;
     int rc;
-    if (!srcdir) {
-        /* when run from automake srcdir is set, but if run manually then not */
-        srcdir = SRCDIR;
-    }
-    str = malloc(strlen(srcdir) + 1 + strlen(dbname) + 1);
+
+    str = malloc(strlen(SRCDIR) + 1 + strlen(dbname) + 1);
     ck_assert_msg(!!str, "malloc");
-    sprintf(str, "%s" PATHSEP "%s", srcdir, dbname);
+    sprintf(str, "%s" PATHSEP "%s", SRCDIR, dbname);
 
     rc = cl_load(str, engine, &signo, CL_DB_STDOPT);
     ck_assert_msg(rc == CL_SUCCESS, "failed to load %s: %s\n",
diff --git a/unit_tests/check_clamav.c b/unit_tests/check_clamav.c
index 1d55f1763c..d9d300f90a 100644
--- a/unit_tests/check_clamav.c
+++ b/unit_tests/check_clamav.c
@@ -538,11 +538,6 @@ static unsigned skip_files(void)
     unsigned skipped = 0;
 
     /* skip .rar files if unrar is disabled */
-    const char *s = getenv("unrar_disabled");
-    if (s && !strcmp(s, "1")) {
-        skipped += 2;
-    }
-
 #if HAVE_UNRAR
 #else
     skipped += 2;
@@ -1899,17 +1894,11 @@ void errmsg_expected(void)
 int open_testfile(const char *name, int flags)
 {
     int fd;
-    const char *srcdir = getenv("srcdir");
     char *str;
 
-    if (!srcdir) {
-        /* when run from automake srcdir is set, but if run manually then not */
-        srcdir = SRCDIR;
-    }
-
-    str = malloc(strlen(name) + strlen(srcdir) + 2);
+    str = malloc(strlen(name) + strlen(SRCDIR) + 2);
     ck_assert_msg(!!str, "malloc");
-    sprintf(str, "%s" PATHSEP "%s", srcdir, name);
+    sprintf(str, "%s" PATHSEP "%s", SRCDIR, name);
 
     fd = open(str, flags);
     ck_assert_msg(fd >= 0, "open() failed: %s", str);