From 7bccbc05af779060aec4c74153d8ca09522e4db0 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 8 Nov 2001 22:32:38 +0000 Subject: [PATCH] Update. 2001-11-08 Ulrich Drepper * elf/dl-object.c (_dl_new_object): Various small optimizations. --- ChangeLog | 4 +++ elf/dl-object.c | 96 ++++++++++++++++++++++++++++----------------------------- 2 files changed, 52 insertions(+), 48 deletions(-) diff --git a/ChangeLog b/ChangeLog index 11c8f61..0d56b69 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2001-11-08 Ulrich Drepper + + * elf/dl-object.c (_dl_new_object): Various small optimizations. + 2001-11-07 Ulrich Drepper * sysdeps/generic/dl-cache.c: Optimize SEARCH_CACHE and diff --git a/elf/dl-object.c b/elf/dl-object.c index 9406ea1..ed78b47 100644 --- a/elf/dl-object.c +++ b/elf/dl-object.c @@ -45,13 +45,12 @@ _dl_new_object (char *realname, const char *libname, int type, if (new == NULL) return NULL; - newname = (struct libname_list *) (new + 1); + new->l_libname = newname = (struct libname_list *) (new + 1); newname->name = (char *) memcpy (newname + 1, libname, libname_len); /* newname->next = NULL; We use calloc therefore not necessary. */ newname->dont_free = 1; new->l_name = realname; - new->l_libname = newname; new->l_type = type; new->l_loader = loader; /* new->l_global = 0; We use calloc therefore not necessary. */ @@ -68,7 +67,7 @@ _dl_new_object (char *realname, const char *libname, int type, if (_dl_loaded != NULL) { l = _dl_loaded; - while (l->l_next) + while (l->l_next != NULL) l = l->l_next; new->l_prev = l; /* new->l_next = NULL; Would be necessary but we use calloc. */ @@ -80,82 +79,83 @@ _dl_new_object (char *realname, const char *libname, int type, else _dl_loaded = new; ++_dl_nloaded; - /* This is our local scope. */ - if (loader != NULL) - { - while (loader->l_loader != NULL) - loader = loader->l_loader; - if (idx == 0 || &loader->l_searchlist != new->l_scope[0]) - new->l_scope[idx] = &loader->l_searchlist; - } - else if (idx == 0 || &new->l_searchlist != new->l_scope[0]) - new->l_scope[idx] = &new->l_searchlist; + + /* If we have no loader the new object acts as it. */ + if (loader == NULL) + loader = new; + else + /* Determine the local scope. */ + while (loader->l_loader != NULL) + loader = loader->l_loader; + + /* Insert the scope if it isn't the global scope we already added. */ + if (idx == 0 || &loader->l_searchlist != new->l_scope[0]) + new->l_scope[idx] = &loader->l_searchlist; new->l_local_scope[0] = &new->l_searchlist; /* Don't try to find the origin for the main map which has the name "". */ if (realname[0] != '\0') { + size_t realname_len = strlen (realname) + 1; char *origin; + char *cp; if (realname[0] == '/') { - /* It an absolute path. Use it. But we have to make a copy since - we strip out the trailing slash. */ - size_t len = strlen (realname) + 1; - origin = (char *) malloc (len); + /* It is an absolute path. Use it. But we have to make a + copy since we strip out the trailing slash. */ + cp = origin = (char *) malloc (realname_len); if (origin == NULL) - origin = (char *) -1; - else - memcpy (origin, realname, len); + { + origin = (char *) -1; + goto out; + } } else { - size_t realname_len = strlen (realname) + 1; - size_t len = 128 + realname_len; + size_t len = realname_len; char *result = NULL; /* Get the current directory name. */ - origin = (char *) malloc (len); - - while (origin != NULL - && (result = __getcwd (origin, len - realname_len)) == NULL - && errno == ERANGE) + origin = NULL; + do { len += 128; origin = (char *) realloc (origin, len); } + while (origin != NULL + && (result = __getcwd (origin, len - realname_len)) == NULL + && errno == ERANGE); if (result == NULL) { - /* We were not able to determine the current directory. */ + /* We were not able to determine the current directory. + Note that free(origin) is OK if origin == NULL. */ free (origin); origin = (char *) -1; + goto out; } - else - { - /* Now append the filename. */ - char *cp = strchr (origin, '\0'); - - if (cp [-1] != '/') - *cp++ = '/'; - memcpy (cp, realname, realname_len); - } + /* Find the end of the path and see whether we have to add + a slash. */ + cp = __rawmemchr (origin, '\0'); + if (cp[-1] != '/') + *cp++ = '/'; } - if (origin != (char *) -1) - { - /* Now remove the filename and the slash. Do this even if the - string is something like "/foo" which leaves an empty string. */ - char *last = strrchr (origin, '/'); - - if (last == origin) - origin[1] = '\0'; - else - *last = '\0'; - } + /* Add the real file name. */ + memcpy (cp, realname, realname_len); + + /* Now remove the filename and the slash. Leave the slash if it + the name is something like "/foo". */ + cp = strrchr (origin, '/'); + if (cp == origin) + origin[1] = '\0'; + else + *cp = '\0'; + out: new->l_origin = origin; } -- 2.7.4