[LTP] [PATCH v2 1/3] lib/tst_tmpdir: Normalize user defined TMPDIR
Petr Vorel
pvorel@suse.cz
Mon Mar 25 12:50:32 CET 2024
Follow the changes to shell API 273c49793 ("tst_test.sh: Remove possible
double/trailing slashes from TMPDIR") and remove: 1) trailing slash
2) double slashes.
This is needed, because some tests compare file path of files which are
in TMPDIR with strcmp() or and extra slashes break it (e.g. chdir01A,
ioctl_loop0[12], mount0[67]).
Co-developed-by: Cyril Hrubis <chrubis@suse.cz>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
lib/tst_tmpdir.c | 20 ++++++++++++++++++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/lib/tst_tmpdir.c b/lib/tst_tmpdir.c
index b73b5c66f..b6e51ba0a 100644
--- a/lib/tst_tmpdir.c
+++ b/lib/tst_tmpdir.c
@@ -124,16 +124,32 @@ char *tst_get_tmpdir(void)
const char *tst_get_tmpdir_root(void)
{
- const char *env_tmpdir = getenv("TMPDIR");
+ char *env_tmpdir = getenv("TMPDIR");
+ char prev_c = 0;
+ size_t k = 0;
if (!env_tmpdir)
- env_tmpdir = TEMPDIR;
+ env_tmpdir = strdup(TEMPDIR);
if (env_tmpdir[0] != '/') {
tst_brkm(TBROK, NULL, "You must specify an absolute "
"pathname for environment variable TMPDIR");
return NULL;
}
+
+ for (int i = 0; env_tmpdir[i] != '\0'; i++) {
+ if (i)
+ prev_c = env_tmpdir[i-1];
+
+ if (env_tmpdir[i] != '/' || prev_c != '/')
+ env_tmpdir[k++] = env_tmpdir[i];
+ }
+
+ env_tmpdir[k] = '\0';
+
+ if (env_tmpdir[k-1] == '/')
+ env_tmpdir[k-1] = '\0';
+
return env_tmpdir;
}
--
2.43.0
More information about the ltp
mailing list