Check gpg version before setting --weak-digest
This commit is contained in:
parent
72fb5158b1
commit
f20e664f40
2 changed files with 16 additions and 4 deletions
|
@ -19,8 +19,20 @@ else
|
||||||
# an attacker could construct a pull-req that results in a commit object that
|
# an attacker could construct a pull-req that results in a commit object that
|
||||||
# they've created a collision for. Not the most likely attack, but preventing
|
# they've created a collision for. Not the most likely attack, but preventing
|
||||||
# it is pretty easy so we do so as a "belt-and-suspenders" measure.
|
# it is pretty easy so we do so as a "belt-and-suspenders" measure.
|
||||||
|
GPG_RES=""
|
||||||
GPG_RES="$(echo "$INPUT" | gpg --trust-model always --weak-digest sha1 "$@" 2>/dev/null)"
|
for LINE in "$(gpg --version)"; do
|
||||||
|
case "$LINE" in
|
||||||
|
"gpg (GnuPG) 1.4.1"*|"gpg (GnuPG) 2.0."*)
|
||||||
|
echo "Please upgrade to at least gpg 2.1.10 to check for weak signatures" > /dev/stderr
|
||||||
|
GPG_RES="$(echo "$INPUT" | gpg --trust-model always "$@" 2>/dev/null)"
|
||||||
|
;;
|
||||||
|
# We assume if you're running 2.1+, you're probably running 2.1.10+
|
||||||
|
# gpg will fail otherwise
|
||||||
|
# We assume if you're running 1.X, it is either 1.4.1X or 1.4.20+
|
||||||
|
# gpg will fail otherwise
|
||||||
|
esac
|
||||||
|
done
|
||||||
|
[ "$GPG_RES" = "" ] && GPG_RES="$(echo "$INPUT" | gpg --trust-model always --weak-digest sha1 "$@" 2>/dev/null)"
|
||||||
fi
|
fi
|
||||||
for LINE in $(echo "$GPG_RES"); do
|
for LINE in $(echo "$GPG_RES"); do
|
||||||
case "$LINE" in
|
case "$LINE" in
|
||||||
|
@ -40,7 +52,7 @@ if ! $VALID; then
|
||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
if $VALID && $REVSIG; then
|
if $VALID && $REVSIG; then
|
||||||
echo "$INPUT" | gpg --trust-model always "$@" | grep "\[GNUPG:\] \(NEWSIG\|SIG_ID\|VALIDSIG\)" 2>/dev/null
|
echo "$INPUT" | gpg --trust-model always "$@" 2>/dev/null | grep "\[GNUPG:\] \(NEWSIG\|SIG_ID\|VALIDSIG\)"
|
||||||
echo "$GOODREVSIG"
|
echo "$GOODREVSIG"
|
||||||
else
|
else
|
||||||
echo "$INPUT" | gpg --trust-model always "$@" 2>/dev/null
|
echo "$INPUT" | gpg --trust-model always "$@" 2>/dev/null
|
||||||
|
|
|
@ -43,7 +43,7 @@ IS_SIGNED () {
|
||||||
export BITCOIN_VERIFY_COMMITS_ALLOW_REVSIG=0
|
export BITCOIN_VERIFY_COMMITS_ALLOW_REVSIG=0
|
||||||
fi
|
fi
|
||||||
|
|
||||||
if ! git -c "gpg.program=${DIR}/gpg.sh" verify-commit $1 > /dev/null 2>&1; then
|
if ! git -c "gpg.program=${DIR}/gpg.sh" verify-commit $1 > /dev/null; then
|
||||||
return 1;
|
return 1;
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue