From 03f2e2360ab2950675f9cde32ccc93d1753c6541 Mon Sep 17 00:00:00 2001 From: Andrew Gregory Date: Thu, 26 Jul 2012 01:04:20 -0400 Subject: query_fileowner: remove trailing '/' from targets Trailing '/' in paths causes lstat to dereference symlinks to directories which causes it to break even though the symlink is a valid target. Signed-off-by: Andrew Gregory --- src/pacman/query.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/pacman/query.c b/src/pacman/query.c index 176f91c2..50b52dd7 100644 --- a/src/pacman/query.c +++ b/src/pacman/query.c @@ -58,7 +58,7 @@ static int search_path(char **filename, struct stat *bufptr) /* strip the trailing slash if one exists */ while(path[plen - 1] == '/') { - path[--plen] = '\0'; + path[--plen] = '\0'; } fullname = malloc(plen + flen + 2); @@ -133,10 +133,17 @@ static int query_fileowner(alpm_list_t *targets) const char *bname; struct stat buf; alpm_list_t *i; + size_t len; int found = 0; filename = strdup(t->data); + /* trailing '/' causes lstat to dereference directory symlinks */ + len = strlen(filename) - 1; + while(len > 0 && filename[len] == '/') { + filename[len--] = '\0'; + } + if(lstat(filename, &buf) == -1) { /* if it is not a path but a program name, then check in PATH */ if(strchr(filename, '/') == NULL) { -- cgit v1.2.3-70-g09d2