Skip to content

Commit 3271fd7

Browse files
TomAFrenchsaleel
authored andcommitted
chore: display warning when installing bb versions < 0.82.0 (#13027)
This PR adds a warning when bbup installs a version of barretenberg prior to #12602 --------- Co-authored-by: saleel <saleel@aztecprotocol.com>
1 parent 4b43a9d commit 3271fd7

File tree

1 file changed

+13
-0
lines changed
  • barretenberg/bbup

1 file changed

+13
-0
lines changed

barretenberg/bbup/bbup

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@ set -e
66
RED='\033[0;31m'
77
GREEN='\033[0;32m'
88
BLUE='\033[0;34m'
9+
YELLOW='\033[1;33m'
910
NC='\033[0m'
11+
BOLD="\033[1m"
1012
SUCCESS=""
1113
ERROR=""
1214

@@ -27,6 +29,10 @@ print_spinner() {
2729
printf " \b\b\b\b"
2830
}
2931

32+
warn() {
33+
echo -e "${BOLD}${YELLOW}WARN: ${1}${NC}"
34+
}
35+
3036
get_bb_version_for_noir() {
3137
local noir_version=$1
3238
local resolved_version=$noir_version
@@ -58,6 +64,13 @@ install_bb() {
5864
local architecture=$(uname -m)
5965
local platform=""
6066

67+
if ! version_gte "$version" "0.82.0"; then
68+
# Unknown when this vulnerability was introduced so we warn on all versions prior to fix.
69+
# See: https://github.com/AztecProtocol/aztec-packages/pull/12602
70+
warn "There is a critical soundness issue in Ultrahonk in this version of Barretenberg. It is recommended to update to v0.82.2 or greater."
71+
warn "Any solidity verifier contracts must be regenerated using a patched version of Barretenberg."
72+
fi
73+
6174
# Convert architecture names
6275
if [ "$architecture" = "arm64" ] || [ "$architecture" = "aarch64" ]; then
6376
if version_gte "$version" "0.77.0"; then

0 commit comments

Comments
 (0)