moparisthebest/pacman
1
0
Fork 0
mirror of https://github.com/moparisthebest/pacman synced 2024-12-22 15:58:50 -05:00
Code Issues Releases Wiki Activity

query_fileowner: remove resolve_path function

Browse Source 
resolve_path is a equivalent to calling realpath(path, NULL) except that the
returned string is guaranteed to be PATH_MAX long.  We never append to the
returned string, so this is unnecessary.

Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com>
This commit is contained in:
Andrew Gregory 2012-07-22 13:30:11 -04:00 committed by Dan McGee
parent bad00696f3
commit 9648887481
1 changed files with 3 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/pacman/query.c
View File

@ -38,23 +38,6 @@
#define LOCAL_PREFIX "local/"
static char *resolve_path(const char *file)
{
char *str = NULL;
str = calloc(PATH_MAX, sizeof(char));
if(!str) {
return NULL;
}
if(!realpath(file, str)) {
free(str);
return NULL;
}
return str;
}
/* check if filename exists in PATH */
static int search_path(char **filename, struct stat *bufptr)
{
@ -178,7 +161,7 @@ static int query_fileowner(alpm_list_t *targets)
if(strcmp(dname, "") == 0) {
rpath = NULL;
} else {
rpath = resolve_path(dname);
rpath = realpath(dname, NULL);
if(!rpath) {
pm_printf(ALPM_LOG_ERROR, _("cannot determine real path for '%s': %s\n"),
@ -202,7 +185,7 @@ static int query_fileowner(alpm_list_t *targets)
char *ppath, *pdname;
const char *pkgfile = file->name;
/* avoid the costly resolve_path usage if the basenames don't match */
/* avoid the costly realpath usage if the basenames don't match */
if(strcmp(mbasename(pkgfile), bname) != 0) {
continue;
}
@ -225,7 +208,7 @@ static int query_fileowner(alpm_list_t *targets)
strcpy(path + rootlen, pkgfile);
pdname = mdirname(path);
ppath = resolve_path(pdname);
ppath = realpath(pdname, NULL);
free(pdname);
if(ppath && strcmp(ppath, rpath) == 0) {