[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 7/8] posix: Use char_array for home_dir in glob
From: |
Adhemerval Zanella |
Subject: |
[PATCH 7/8] posix: Use char_array for home_dir in glob |
Date: |
Tue, 5 Jan 2021 15:58:19 -0300 |
This patch uses dynarray at glob internal home directory ame
manipulation for GLOB_TILDE. It simplifies it and removes all the
boilerplate buffer managements required.
Checked x86_64-linux-gnu.
---
posix/glob.c | 69 ++++++++++++++++++++++++++++++++--------------------
1 file changed, 42 insertions(+), 27 deletions(-)
diff --git a/posix/glob.c b/posix/glob.c
index c126b4501d..b6727ee884 100644
--- a/posix/glob.c
+++ b/posix/glob.c
@@ -606,10 +606,15 @@ __glob (const char *pattern, int flags, int (*errfunc)
(const char *, int),
&& (char_array_pos (&dirname, 2) == '\0'
|| char_array_pos (&dirname, 2) == '/')))
{
+ struct char_array home_dir;
+
+ const char *home_env = getenv ("HOME");
+ home_env = home_env == NULL ? "" : home_env;
+ if (!char_array_init_str (&home_dir, home_env))
+ goto err_nospace;
+
+ if (char_array_is_empty (&home_dir))
/* Look up home directory. */
- char *home_dir = getenv ("HOME");
- int malloc_home_dir = 0;
- if (home_dir == NULL || home_dir[0] == '\0')
{
#ifdef WINDOWS32
/* Windows NT defines HOMEDRIVE and HOMEPATH. But give
@@ -619,16 +624,21 @@ __glob (const char *pattern, int flags, int (*errfunc)
(const char *, int),
if (home_drive != NULL && home_path != NULL)
{
- size_t home_drive_len = strlen (home_drive);
- size_t home_path_len = strlen (home_path);
- char *mem = alloca (home_drive_len + home_path_len + 1);
-
- memcpy (mem, home_drive, home_drive_len);
- memcpy (mem + home_drive_len, home_path, home_path_len + 1);
- home_dir = mem;
+ if (!char_array_set_str (&home_dir, home_drive)
+ || !char_array_append_str (&home_dir, home_path))
+ {
+ char_array_free (&home_dir);
+ goto err_nospace;
+ }
}
else
- home_dir = "c:/users/default"; /* poor default */
+ {
+ if (!char_array_set_str (&home_dir, "c:/users/default"))
+ {
+ char_array_free (&home_dir);
+ goto err_nospace;
+ }
+ }
#else
int err;
struct passwd *p;
@@ -657,44 +667,49 @@ __glob (const char *pattern, int flags, int (*errfunc)
(const char *, int),
if (!scratch_buffer_grow (&s))
goto err_nospace;
}
- if (err == 0)
- {
- home_dir = strdup (p->pw_dir);
- malloc_home_dir = 1;
- }
+ if (err == 0 && !char_array_set_str (&home_dir, p->pw_dir))
+ err = 1;
scratch_buffer_free (&s);
- if (err == 0 && home_dir == NULL)
- goto err_nospace;
+ if (err == 0)
+ goto err_nospace;
#endif /* WINDOWS32 */
}
- if (home_dir == NULL || home_dir[0] == '\0')
+ if (char_array_is_empty (&home_dir))
{
- if (__glibc_unlikely (malloc_home_dir))
- free (home_dir);
if (flags & GLOB_TILDE_CHECK)
{
+ char_array_free (&home_dir);
retval = GLOB_NOMATCH;
goto out;
}
else
{
- home_dir = (char *) "~"; /* No luck. */
- malloc_home_dir = 0;
+ if (!char_array_set_str (&home_dir, "~"))
+ {
+ char_array_free (&home_dir);
+ goto err_nospace;
+ }
}
}
/* Now construct the full directory. */
+ bool e = true;
if (char_array_pos (&dirname, 1) == '\0')
{
- if (!char_array_set_str (&dirname, home_dir))
- goto err_nospace;
+ e = char_array_set_str (&dirname, char_array_str (&home_dir));
dirlen = char_array_size (&dirname) - 1;
}
else
{
/* Replaces '~' by the obtained HOME dir. */
char_array_erase (&dirname, 0);
- if (!char_array_prepend_str (&dirname, home_dir))
- goto err_nospace;
+ e = char_array_prepend_str (&dirname,
+ char_array_str (&home_dir));
+ }
+ if (e == false)
+ {
+ char_array_free (&dirname);
+ char_array_free (&home_dir);
+ goto err_nospace;
}
dirname_modified = true;
}
--
2.25.1
- [PATCH 0/8] Remove alloca usage from glob, Adhemerval Zanella, 2021/01/05
- [PATCH 1/8] malloc: Add specialized dynarray for C strings, Adhemerval Zanella, 2021/01/05
- [PATCH 3/8] posix: Remove alloca usage for GLOB_BRACE on glob, Adhemerval Zanella, 2021/01/05
- [PATCH 4/8] posix: Remove alloca usage on glob dirname, Adhemerval Zanella, 2021/01/05
- [PATCH 2/8] posix: Use char_array for internal glob dirname, Adhemerval Zanella, 2021/01/05
- [PATCH 6/8] posix: Remove alloca usage on glob user_name, Adhemerval Zanella, 2021/01/05
- [PATCH 5/8] posix: Use dynarray for globname in glob, Adhemerval Zanella, 2021/01/05
- [PATCH 7/8] posix: Use char_array for home_dir in glob,
Adhemerval Zanella <=
- [PATCH 8/8] posix: Remove all alloca usage in glob, Adhemerval Zanella, 2021/01/05
- Re: [PATCH 0/8] Remove alloca usage from glob, Paul Eggert, 2021/01/13