diff options
author | Robby Workman <rworkman@slackware.com> | 2021-05-31 11:49:52 -0500 |
---|---|---|
committer | Robby Workman <rworkman@slackware.com> | 2021-06-04 22:23:27 -0500 |
commit | 3c4c175de873a3ac5599da454bb693d2d5719811 (patch) | |
tree | a0e98cfd99c30ed8e20e55ee9a2c44b8a3e7326a | |
parent | c43af13723c0cc2ada7f6542c73449c757799ce3 (diff) | |
download | slackpkg-3c4c175de873a3ac5599da454bb693d2d5719811.tar.xz |
Harden slackpkg with respect to obtaining GPG key
Co-authored-by: CRTS <crts@gmx.net>
Reviewed-by: Piter PUNK <piterpunk@slackware.com>
-rw-r--r-- | files/core-functions.sh | 52 | ||||
-rw-r--r-- | files/slackpkg | 13 |
2 files changed, 46 insertions, 19 deletions
diff --git a/files/core-functions.sh b/files/core-functions.sh index a579f9f..69f8062 100644 --- a/files/core-functions.sh +++ b/files/core-functions.sh @@ -64,9 +64,14 @@ function system_setup() { # Create $WORKDIR just in case mkdir -p "${WORKDIR}" + # Select the command to fetch files and packages from network sources + if [ "$DOWNLOADER" = "curl" ]; then + DOWNLOADER="curl ${CURLFLAGS} -o" + else + DOWNLOADER="wget ${WGETFLAGS} -O" + fi + # Set LOCAL if mirror isn't through network - # If mirror is through network, select the command to fetch - # files and packages from there. # MEDIA=${SOURCE%%:*} if [ "$MEDIA" = "cdrom" ] || [ "$MEDIA" = "file" ] || \ @@ -75,11 +80,6 @@ function system_setup() { LOCAL=1 else LOCAL=0 - if [ "$DOWNLOADER" = "curl" ]; then - DOWNLOADER="curl ${CURLFLAGS} -o" - else - DOWNLOADER="wget ${WGETFLAGS} -O" - fi fi # Set MORECMD, EDITCMD and check BATCH mode @@ -555,6 +555,42 @@ function checkgpg() { gpg --verify ${1}.asc ${1} 2>/dev/null && echo "1" || echo "0" } +function get_gpg_key() { + if ping -c 1 slackware.com &>/dev/null; then + echo -e "\t\t\tGetting key from https://www.slackware.com/infra/keys/GPG-KEY" + $DOWNLOADER $TMPDIR/gpgkey https://www.slackware.com/infra/keys/GPG-KEY &>/dev/null + elif ping -c 1 mirrors.slackware.com &>/dev/null; then + echo -e "\t\t\tGetting key from https://mirrors.slackware.com/slackware/slackware-current/GPG-KEY" + $DOWNLOADER $TMPDIR/gpgkey https://mirrors.slackware.com/slackware/slackware-current/GPG-KEY &>/dev/null + else + echo -e "\ +slackpkg is unable to get the Slackware GPG key from either\n\ +slackware.com or mirrors.slackware.com; if you trust the\n\ +source you have configured in /etc/slackpkg/mirrors, slackpkg\n\ +can import the GPG key from that source.\n\ +The source currently in use is:\n\ +\t ${SOURCE}\n\ +Do you want to import the GPG key from this source? (YES|NO)\n" + read ANSWER + case "$ANSWER" in + YES|Y|yes|y) + getfile ${SOURCE}GPG-KEY $TMPDIR/gpgkey + ;; + *) + echo -e "\t\tslackpkg is unable to get the Slackware GPG key." + cleanup + ;; + esac + fi +} + +function import_gpg_key() { + mkdir -p ~/.gnupg + gpg --yes --batch --delete-key "$SLACKKEY" &>/dev/null + gpg --import $TMPDIR/gpgkey &>/dev/null && \ + echo -e "\t\t\tSlackware Linux Project's GPG key added" +} + # Found packages in repository. # This function selects the package from the higher priority # repository directories. @@ -904,7 +940,7 @@ function getfile() { echo -e "\t\t\tDownloading $1..." $DOWNLOADER $2 $1 fi -} +} # Function to download the correct package and many "checks" # diff --git a/files/slackpkg b/files/slackpkg index 1076ba3..f606574 100644 --- a/files/slackpkg +++ b/files/slackpkg @@ -350,17 +350,8 @@ case "$CMD" in # the key # if [ "$UPARG" = "gpg" ] || [ "$GPGFIRSTTIME" = "0" ]; then - # - # Creates .gnupg directory if doesn't exist - # without this dir, gpg got an error. - # - if ! [ -e ~/.gnupg ]; then - mkdir ~/.gnupg - fi - getfile ${SOURCE}GPG-KEY $TMPDIR/gpgkey - gpg --yes --batch --delete-key "$SLACKKEY" &>/dev/null - gpg --import $TMPDIR/gpgkey &>/dev/null && \ - echo -e "\t\t\tSlackware Linux Project's GPG key added" + get_gpg_key && import_gpg_key + if [ "$UPARG" = "gpg" ]; then cleanup fi |