Fix crash if hash in release notes is greater than 40 characters

russellbanks · Feb 20, 2024 · da7e5e9 · da7e5e9
1 parent b70d38b
commit da7e5e9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/types/release_notes.rs b/src/types/release_notes.rs
@@ -136,9 +136,9 @@ fn remove_sha1(input: &str) -> String {
     let mut buffer = heapless::String::<SHA1_LEN>::new();
 
     for character in input.chars() {
-        if character.is_ascii_hexdigit() {
+        if character.is_ascii_hexdigit() && buffer.len() < SHA1_LEN {
             buffer.push(character).unwrap();
-        } else if buffer.len() == SHA1_LEN {
+        } else if !character.is_ascii_hexdigit() && buffer.len() == SHA1_LEN {
             buffer.clear();
         } else {
             result.push_str(&buffer);