Reject UserInfo values that do not satisfy validate_name

Signed-off-by: mulhern <[email protected]>
stratis-storage · Mar 19, 2024 · 8f66253 · 8f66253
1 parent 9ef2fb2
commit 8f66253
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/src/engine/sim_engine/pool.rs b/src/engine/sim_engine/pool.rs
@@ -628,6 +628,7 @@ impl Pool for SimPool {
         uuid: DevUuid,
         user_info: Option<&str>,
     ) -> StratisResult<RenameAction<DevUuid>> {
+        user_info.map(validate_name).transpose()?;
         Ok(self.get_mut_blockdev_internal(uuid).map_or_else(
             || RenameAction::NoSource,
             |(_, b)| {

diff --git a/src/engine/strat_engine/pool.rs b/src/engine/strat_engine/pool.rs
@@ -1133,6 +1133,7 @@ impl Pool for StratPool {
         uuid: DevUuid,
         user_info: Option<&str>,
     ) -> StratisResult<RenameAction<DevUuid>> {
+        user_info.map(validate_name).transpose()?;
         let result = self.backstore.set_blockdev_user_info(uuid, user_info);
         match result {
             Ok(Some(uuid)) => {