check bounds before normalizing (keybase#24292)

* check bounds before normalizing * fix fmt error * -_-
lynsei · Aug 17, 2020 · 5125de0 · 5125de0
1 parent 1c032f6
commit 5125de0
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 6 deletions.
diff --git a/go/engine/cryptocurrency.go b/go/engine/cryptocurrency.go
@@ -50,6 +50,9 @@ func (e *CryptocurrencyEngine) SubConsumers() []libkb.UIConsumer {
 }
 
 func normalizeAddress(address string) string {
+	if len(address) <= 3 {
+		return address
+	}
 	switch strings.ToLower(address)[0:3] {
 	case "bc1", "zs1":
 		// bech32 addresses require that cases not be mixed

diff --git a/go/kbfs/data/data_mocks_test.go b/go/kbfs/data/data_mocks_test.go
diff --git a/go/kbfs/libkbfs/data_mocks_test.go b/go/kbfs/libkbfs/data_mocks_test.go
diff --git a/go/kbfs/libkbfs/libkbfs_mocks_test.go b/go/kbfs/libkbfs/libkbfs_mocks_test.go
diff --git a/go/kbfs/libkbfs/libkey_mocks_test.go b/go/kbfs/libkbfs/libkey_mocks_test.go