+1998-09-01 17:53 Ulrich Drepper <drepper@cygnus.com>
+
+ * elf/dl-load.c (add_name_to_object): Change return type to void and
+ make NAME parameter const. Allocate room for NAME in same memory
+ block used for l_libname entry.
+ (_dl_map_object_from_fd): Don't free NAME on failure.
+ (map_segment): Pass SONAME to add_name_to_object, not a copy.
+ (_dl_map_object): Don't create copy of NAME. Pass NAME to
+ _dl_map_object_from_fd.
+ * elf/dl-object.c (dl_new_object): Allocate room for NAME in same
+ memory block used for l_libname entry.
+ * elf/dl-close.c: Adjust free()ing for this change.
+
1998-09-01 15:36 Ulrich Drepper <drepper@cygnus.com>
* malloc/Makefile: Include Makeconfig before testing config-sysdirs.
`name' is expected to have been allocated with malloc and will
be freed if the shared object already has this name.
Returns false if the object already had this name. */
-static int
+static void
internal_function
-add_name_to_object (struct link_map *l, char *name)
+add_name_to_object (struct link_map *l, const char *name)
{
struct libname_list *lnp, *lastp;
struct libname_list *newname;
-
- if (name == NULL)
- {
- /* No more memory. */
- _dl_signal_error (ENOMEM, NULL, "could not allocate name string");
- return 0;
- }
+ size_t name_len;
lastp = NULL;
for (lnp = l->l_libname; lnp != NULL; lastp = lnp, lnp = lnp->next)
if (strcmp (name, lnp->name) == 0)
- {
- free (name);
- return 0;
- }
+ return;
- newname = malloc (sizeof *newname);
+ name_len = strlen (name) + 1;
+ newname = malloc (sizeof *newname + name_len);
if (newname == NULL)
{
/* No more memory. */
_dl_signal_error (ENOMEM, name, "cannot allocate name record");
- free (name);
- return 0;
+ return;
}
/* The object should have a libname set from _dl_new_object. */
assert (lastp != NULL);
- newname->name = name;
+ newname->name = memcpy (newname + 1, name, name_len);
newname->next = NULL;
lastp->next = newname;
- return 1;
}
/* All known directories in sorted order. */
}
free (realname);
_dl_signal_error (code, name, msg);
- free (name); /* Hmmm. Can this leak memory? Better
- than a segfault, anyway. */
}
inline caddr_t map_segment (ElfW(Addr) mapstart, size_t len,
continue;
/* We have a match on a new name -- cache it. */
- add_name_to_object (l, local_strdup (soname));
+ add_name_to_object (l, soname);
}
/* We have a match -- bump the reference count and return it. */
}
}
- if (fd != -1)
- {
- name_copy = local_strdup (name);
- if (name_copy == NULL)
- {
- __close (fd);
- fd = -1;
- }
- }
-
if (fd == -1)
{
if (trace_mode)
_dl_signal_error (errno, name, "cannot open shared object file");
}
- return _dl_map_object_from_fd (name_copy, fd, realname, loader, type);
+ return _dl_map_object_from_fd (name, fd, realname, loader, type);
}
internal_function
_dl_new_object (char *realname, const char *libname, int type, int find_origin)
{
- struct link_map *new = malloc (sizeof *new);
- struct libname_list *newname = malloc (sizeof *newname);
+ size_t libname_len = strlen (libname) + 1;
+ struct link_map *new = calloc (sizeof *new, 1);
+ struct libname_list *newname = malloc (sizeof *newname + libname_len);
if (! new || ! newname)
return NULL;
- memset (new, 0, sizeof *new);
new->l_name = realname;
- newname->name = libname;
+ newname->name = memcpy (newname + 1, libname, libname_len);
newname->next = NULL;
new->l_libname = newname;
new->l_type = type;