diff options
| author | Eli Schwartz <eschwartz@archlinux.org> | 2018-02-12 19:41:29 -0500 |
|---|---|---|
| committer | Pierre Schmitz <mail@pierre-schmitz.com> | 2018-02-15 16:41:37 +0100 |
| commit | a7b497e8347fc964f8738d4dfc3c3ef23806fdc7 (patch) | |
| tree | e5ed728cbdaf3fdc7efb9fef30d586baaeb88abb | |
| parent | 5afac1ed83479c5ff7aab134b54245ec4f5b59fe (diff) | |
ARCHES is an array, do not attempt to call it as a string
This meant only the first array element was checked in
check_repo_permission(). Although arguably this should never cause real
issues as something else would have to be broken if multiple
architectures have different permissions, we should catch this now
anyway.
| -rw-r--r-- | db-functions | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/db-functions b/db-functions index f8e058a..6d6beea 100644 --- a/db-functions +++ b/db-functions @@ -410,7 +410,7 @@ check_repo_permission() { [ -w "$FTP_BASE/${PKGPOOL}" ] || return 1 local arch - for arch in ${ARCHES}; do + for arch in ${ARCHES[@]}; do local dir="${FTP_BASE}/${repo}/os/${arch}/" [ -w "${dir}" ] || return 1 [ -f "${dir}"${repo}${DBEXT} -a ! -w "${dir}"${repo}${DBEXT} ] && return 1 |