github/kata-containers
1
0
Fork 1
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-04-29 20:24:31 +00:00
Code Issues Projects Releases Wiki Activity

agent: Remove unwrap from verify_cid()

Browse Source 
Improved the `verify_cid()` function that validates container ID's by
removing the need for an `unwrap()`.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
This commit is contained in:
James O. D. Hunt 2021-11-15 10:59:18 +00:00
parent a7d1c70c4b
commit 351cef7b6a
1 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/agent/src/rpc.rs
View File

@ -111,11 +111,18 @@ pub struct AgentService {
// ^[a-zA-Z0-9][a-zA-Z0-9_.-]+$ // ^[a-zA-Z0-9][a-zA-Z0-9_.-]+$
// //
fn verify_cid(id: &str) -> Result<()> { fn verify_cid(id: &str) -> Result<()> {
let valid = id.len() > 1 let mut chars = id.chars();
&& id.chars().next().unwrap().is_alphanumeric()
&& id let valid = match chars.next() {
.chars() Some(first)
.all(|c| (c.is_alphanumeric() || ['.', '-', '_'].contains(&c))); if first.is_alphanumeric()
&& id.len() > 1
&& chars.all(|c| c.is_alphanumeric() || ['.', '-', '_'].contains(&c)) =>
{
true
}
_ => false,
};
match valid { match valid {
true => Ok(()), true => Ok(()),