2001-09-08  Ulrich Drepper  <drepper@redhat.com>

	* elf/dl-object.c: Avoid allocating extra memory block for name.
	* elf/dl-close.c (_dl_close): Don't free l_libname if it is no
	allocated separately.
	* elf/dl-load.c (_dl_map_object_from_fd): Likewise.
This commit is contained in:
Ulrich Drepper 2001-09-08 20:03:14 +00:00
parent f6233fe003
commit 1181062126
4 changed files with 18 additions and 7 deletions

View File

@ -1,3 +1,10 @@
2001-09-08 Ulrich Drepper <drepper@redhat.com>
* elf/dl-object.c: Avoid allocating extra memory block for name.
* elf/dl-close.c (_dl_close): Don't free l_libname if it is no
allocated separately.
* elf/dl-load.c (_dl_map_object_from_fd): Likewise.
2001-09-08 H.J. Lu <hjl@gnu.org>
* po/zh_TW.po: Fix a typo.

View File

@ -254,7 +254,8 @@ _dl_close (void *_map)
{
struct libname_list *this = lnp;
lnp = lnp->next;
free (this);
if (!this->dont_free)
free (this);
}
while (lnp != NULL);

View File

@ -1099,7 +1099,8 @@ _dl_map_object_from_fd (const char *name, int fd, struct filebuf *fbp,
/* We are not supposed to load this object. Free all resources. */
__munmap ((void *) l->l_map_start, l->l_map_end - l->l_map_start);
free (l->l_libname);
if (!l->l_libname->dont_free)
free (l->l_libname);
if (l->l_phdr_allocated)
free ((void *) l->l_phdr);

View File

@ -40,15 +40,17 @@ _dl_new_object (char *realname, const char *libname, int type,
struct link_map *new;
struct libname_list *newname;
new = (struct link_map *) calloc (sizeof *new, 1);
newname = (struct libname_list *) malloc (sizeof *newname + libname_len);
if (new == NULL || newname == NULL)
new = (struct link_map *) calloc (sizeof (*new) + sizeof (*newname)
+ libname_len, 1);
if (new == NULL)
return NULL;
new->l_name = realname;
newname = (struct libname_list *) (new + 1);
newname->name = (char *) memcpy (newname + 1, libname, libname_len);
newname->next = NULL;
newname->dont_free = 0;
newname->dont_free = 1;
new->l_name = realname;
new->l_libname = newname;
new->l_type = type;
new->l_loader = loader;