summaryrefslogtreecommitdiffstats
path: root/lib/libalpm
diff options
context:
space:
mode:
authorAaron Griffin <aaron@archlinux.org>2007-03-04 10:16:28 +0100
committerAaron Griffin <aaron@archlinux.org>2007-03-04 10:16:28 +0100
commit28b2dbb52c8b7bb0b611b2e86cef69f3e911facb (patch)
tree47a6082d788dbd7bebf0ee1bada33ab2aab67c59 /lib/libalpm
parentcdb46ef3fa6d2bea95ae45b8b807497982b18fd5 (diff)
downloadpacman-28b2dbb52c8b7bb0b611b2e86cef69f3e911facb.tar.gz
pacman-28b2dbb52c8b7bb0b611b2e86cef69f3e911facb.tar.xz
* Fixed the handle realroot stuff
* Added some {}
Diffstat (limited to 'lib/libalpm')
-rw-r--r--lib/libalpm/handle.c10
1 files changed, 6 insertions, 4 deletions
diff --git a/lib/libalpm/handle.c b/lib/libalpm/handle.c
index c17d86a5..b044adcb 100644
--- a/lib/libalpm/handle.c
+++ b/lib/libalpm/handle.c
@@ -155,13 +155,13 @@ void alpm_option_set_root(const char *root)
/* According to the man page, realpath is safe to use IFF the second arg is
* NULL. */
char *realroot = realpath(root, NULL);
- if(!realroot) {
- realroot = root;
+ if(realroot) {
+ root = realroot;
+ } else {
_alpm_log(PM_LOG_ERROR, _("cannot canonicalize specified root path '%s'"), root);
}
- /* check again, in case both are null */
- if(realroot) {
+ if(root) {
/* verify root ends in a '/' */
int rootlen = strlen(realroot);
if(realroot[rootlen-1] != '/') {
@@ -172,6 +172,8 @@ void alpm_option_set_root(const char *root)
handle->root[rootlen-1] = '/';
_alpm_log(PM_LOG_DEBUG, _("option 'root' = %s"), handle->root);
+ }
+ if(realroot) {
free(realroot);
}
}