diff options
author | Noah Goldstein <goldstein.w.n@gmail.com> | 2022-08-08 11:26:22 +0800 |
---|---|---|
committer | Noah Goldstein <goldstein.w.n@gmail.com> | 2022-08-09 17:00:03 +0800 |
commit | 483cfe1a6a33d6335b1901581b41040d2d412511 (patch) | |
tree | 09d8edb50a9a14e45263b7d9eda26fefdd4a6eb6 | |
parent | 8bc3f94a062776abfaf14201fba37bea5328bf92 (diff) | |
download | glibc-483cfe1a6a33d6335b1901581b41040d2d412511.tar glibc-483cfe1a6a33d6335b1901581b41040d2d412511.tar.gz glibc-483cfe1a6a33d6335b1901581b41040d2d412511.tar.bz2 glibc-483cfe1a6a33d6335b1901581b41040d2d412511.zip |
elf: Replace `strcpy` call with `memcpy` [BZ #29454]
GCC normally does this optimization for us in
strlen_pass::handle_builtin_strcpy but only for optimized
build. To avoid needing to include strcpy.S in the rtld build to
support the debug build, just do the optimization by hand.
-rw-r--r-- | elf/dl-cache.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/elf/dl-cache.c b/elf/dl-cache.c index 8bbf110d02..b97c17b3a9 100644 --- a/elf/dl-cache.c +++ b/elf/dl-cache.c @@ -509,8 +509,9 @@ _dl_load_cache_lookup (const char *name) we are accessing. Therefore we must make the copy of the mapping data without using malloc. */ char *temp; - temp = alloca (strlen (best) + 1); - strcpy (temp, best); + size_t best_len = strlen (best) + 1; + temp = alloca (best_len); + memcpy (temp, best, best_len); return __strdup (temp); } |