diff options
author | Ulrich Drepper <drepper@redhat.com> | 2000-12-09 00:02:12 +0000 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 2000-12-09 00:02:12 +0000 |
commit | 7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35 (patch) | |
tree | 2eb4a9c98ac52d02390eccfa7baca3168c986008 /elf/dl-open.c | |
parent | 8b80182964bde1fa84333674e454d41e1fa0c8ca (diff) | |
download | glibc-7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35.tar.gz glibc-7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35.tar.xz glibc-7b70fef6ae9e858f9d6b8a838dda1ee943dd6c35.zip |
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/dl-open.c')
-rw-r--r-- | elf/dl-open.c | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/elf/dl-open.c b/elf/dl-open.c index 48bb9f82ed..90355796ba 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); |