diff options
author | Samuel Dobson <dobsonsa68@gmail.com> | 2021-11-23 12:17:34 +1300 |
---|---|---|
committer | Samuel Dobson <dobsonsa68@gmail.com> | 2021-11-23 15:48:59 +1300 |
commit | c8b9a224e70f70ccc638b2c4200a505cdf024efd (patch) | |
tree | 158d0c3d3b9342ca4e50a1cb9f95f3bfe20cd440 /src/bech32.cpp | |
parent | 92f0cafdca11a9463b6f04229c1c47805c97c1b5 (diff) |
Report encoding type in bech32 error message
Diffstat (limited to 'src/bech32.cpp')
-rw-r--r-- | src/bech32.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/bech32.cpp b/src/bech32.cpp index ec69965d07..7585ee944c 100644 --- a/src/bech32.cpp +++ b/src/bech32.cpp @@ -7,6 +7,7 @@ #include <util/vector.h> #include <assert.h> +#include <optional> namespace bech32 { @@ -539,6 +540,7 @@ std::string LocateErrors(const std::string& str, std::vector<int>& error_locatio // We attempt error detection with both bech32 and bech32m, and choose the one with the fewest errors // We can't simply use the segwit version, because that may be one of the errors + std::optional<Encoding> error_encoding; for (Encoding encoding : {Encoding::BECH32, Encoding::BECH32M}) { std::vector<int> possible_errors; // Recall that (ExpandHRP(hrp) ++ values) is interpreted as a list of coefficients of a polynomial @@ -657,9 +659,13 @@ std::string LocateErrors(const std::string& str, std::vector<int>& error_locatio if (error_locations.empty() || (!possible_errors.empty() && possible_errors.size() < error_locations.size())) { error_locations = std::move(possible_errors); + if (!error_locations.empty()) error_encoding = encoding; } } - return "Invalid checksum"; + return error_encoding == Encoding::BECH32M ? "Invalid Bech32m checksum" + : error_encoding == Encoding::BECH32 ? "Invalid Bech32 checksum" + : "Invalid checksum"; + } } // namespace bech32 |