Skip to content

Commit 0213918

Browse files
committed
use saturating math
1 parent 40c8214 commit 0213918

File tree

1 file changed

+18
-2
lines changed

1 file changed

+18
-2
lines changed

pallets/subtensor/src/utils/identity.rs

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,15 @@ impl<T: Config> Pallet<T> {
167167
.saturating_add(identity.description.len())
168168
.saturating_add(identity.additional.len());
169169

170-
total_length <= 256 + 256 + 256 + 1024 + 256 + 1024 + 1024
170+
let max_length: usize = 256_usize
171+
.saturating_add(256)
172+
.saturating_add(256)
173+
.saturating_add(1024)
174+
.saturating_add(256)
175+
.saturating_add(1024)
176+
.saturating_add(1024);
177+
178+
total_length <= max_length
171179
&& identity.name.len() <= 256
172180
&& identity.url.len() <= 256
173181
&& identity.github_repo.len() <= 256
@@ -197,7 +205,15 @@ impl<T: Config> Pallet<T> {
197205
.saturating_add(identity.github_repo.len())
198206
.saturating_add(identity.subnet_contact.len());
199207

200-
total_length <= 256 + 1024 + 1024 + 1024 + 256 + 1024 + 1024
208+
let max_length: usize = 256_usize
209+
.saturating_add(1024)
210+
.saturating_add(1024)
211+
.saturating_add(1024)
212+
.saturating_add(256)
213+
.saturating_add(1024)
214+
.saturating_add(1024);
215+
216+
total_length <= max_length
201217
&& identity.subnet_name.len() <= 256
202218
&& identity.github_repo.len() <= 1024
203219
&& identity.subnet_contact.len() <= 1024

0 commit comments

Comments
 (0)