Avoid unwrapping max_fragment_size

AlfioEmanueleFresta · AlfioEmanueleFresta · commit 97e93c409745 · 2025-03-22T21:58:20.000Z
diff --git a/libwebauthn/src/transport/ble/btleplug/connection.rs b/libwebauthn/src/transport/ble/btleplug/connection.rs
@@ -48,7 +48,9 @@ impl Connection {
         }
 
         let mut cursor = IOCursor::new(max_fragment_length);
-        let max_fragment_size = cursor.read_u16::<BigEndian>().unwrap() as usize;
+        let max_fragment_size = cursor
+            .read_u16::<BigEndian>()
+            .map_err(|_| Error::OperationFailed)? as usize;
         Ok(max_fragment_size)
     }
 

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,9 @@ impl Connection {`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`let mut cursor = IOCursor::new(max_fragment_length);`
`51`		`- let max_fragment_size = cursor.read_u16::<BigEndian>().unwrap() as usize;`
	`51`	`+ let max_fragment_size = cursor`
	`52`	`+ .read_u16::<BigEndian>()`
	`53`	`+ .map_err(\|_\| Error::OperationFailed)? as usize;`
`52`	`54`	`Ok(max_fragment_size)`
`53`	`55`	`}`
`54`	`56`