sshkeys: Add paranoid whitespace check

Newlines, in particular, would mess things up.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/src/sshkeys.rs b/src/sshkeys.rs
index 30b7693dbcd418bb3a70c250284cfefce30483aa..a3268464a6404d4a274e8892677da53688b83ced 100644 (file)
--- a/src/sshkeys.rs
+++ b/src/sshkeys.rs
@@ -119,6 +119,7 @@ mod veneer {
   #[derive(Error,Debug,Clone,Serialize,Deserialize)]
   pub enum KeyError {
     #[error("bad key data: {0}")]                        BadData(String),
+    #[error("whitespace in public key data!")]           Whitespace,
     #[error("failed to save key data, possibly broken")] Dirty,
   }
 
@@ -146,7 +147,11 @@ mod veneer {
         options: None,
         comment: None,
       };
-      (PubData(data.to_string()), Comment(comment.unwrap_or_default()))
+      let data = PubData(data.to_string());
+      if data.0.chars().any(|c| c !=' ' && c.is_whitespace()) {
+        throw!(KeyError::Whitespace);
+      }
+      (data, Comment(comment.unwrap_or_default()))
     }
   }