aboutsummaryrefslogtreecommitdiff
path: root/elf
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2000-12-09 00:02:12 +0000
committerUlrich Drepper <drepper@redhat.com>2000-12-09 00:02:12 +0000
commit7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35 (patch)
tree2eb4a9c98ac52d02390eccfa7baca3168c986008 /elf
parent8b80182964bde1fa84333674e454d41e1fa0c8ca (diff)
downloadglibc-7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35.zip
glibc-7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35.tar.gz
glibc-7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35.tar.bz2
Update.
2000-12-08 Jakub Jelinek <jakub@redhat.com> * elf/dl-open.c (_dl_open): If objname points right after errstring, allocate it together with errstring using alloca. * elf/dl-error.c (_dl_signal_error): If malloc failed, set objname to "", because it might point to local stack.
Diffstat (limited to 'elf')
-rw-r--r--elf/dl-error.c2
-rw-r--r--elf/dl-open.c16
2 files changed, 16 insertions, 2 deletions
diff --git a/elf/dl-error.c b/elf/dl-error.c
index 1c6de64..83d984f 100644
--- a/elf/dl-error.c
+++ b/elf/dl-error.c
@@ -88,7 +88,7 @@ _dl_signal_error (int errcode, const char *objname, const char *errstring)
else
{
/* This is better than nothing. */
- lcatch->objname = objname;
+ lcatch->objname = "";
lcatch->errstring = _dl_out_of_memory;
}
longjmp (lcatch->env, errcode ?: -1);
diff --git a/elf/dl-open.c b/elf/dl-open.c
index 48bb9f8..9035579 100644
--- a/elf/dl-open.c
+++ b/elf/dl-open.c
@@ -391,6 +391,7 @@ _dl_open (const char *file, int mode, const void *caller)
{
/* Some error occurred during loading. */
char *local_errstring;
+ size_t len_errstring;
/* Remove the object from memory. It may be in an inconsistent
state if relocation failed, for example. */
@@ -399,7 +400,20 @@ _dl_open (const char *file, int mode, const void *caller)
/* Make a local copy of the error string so that we can release the
memory allocated for it. */
- local_errstring = strdupa (errstring);
+ len_errstring = strlen (errstring) + 1;
+ if (objname == errstring + len_errstring)
+ {
+ len_errstring += strlen (objname) + 1;
+ local_errstring = alloca (len_errstring);
+ memcpy (local_errstring, errstring, len_errstring);
+ objname = local_errstring + len_errstring;
+ }
+ else
+ {
+ local_errstring = alloca (len_errstring);
+ memcpy (local_errstring, errstring, len_errstring);
+ }
+
if (errstring != _dl_out_of_memory)
free ((char *) errstring);