Fix broken serde::Deserialize and FromStr impl of keyPair

Fixes #491

Author: elsirion

src/key.rs

@@ -1053,14 +1053,12 @@ impl<'a> From<&'a KeyPair> for PublicKey {
     }
 }
 
+#[cfg(feature = "global-context")]
 impl str::FromStr for KeyPair {
     type Err = Error;
 
     fn from_str(s: &str) -> Result<Self, Self::Err> {
-        let ctx = unsafe {
-            Secp256k1::from_raw_all(ffi::secp256k1_context_no_precomp as *mut ffi::Context)
-        };
-        KeyPair::from_seckey_str(&ctx, s)
+        KeyPair::from_seckey_str(SECP256K1, s)
     }
 }
 
@@ -1082,7 +1080,7 @@ impl serde::Serialize for KeyPair {
     }
 }
 
-#[cfg(feature = "serde")]
+#[cfg(all(feature = "serde", feature = "global-context"))]
 #[cfg_attr(docsrs, doc(cfg(feature = "serde")))]
 impl<'de> serde::Deserialize<'de> for KeyPair {
     fn deserialize<D: serde::Deserializer<'de>>(d: D) -> Result<Self, D::Error> {
@@ -1093,9 +1091,8 @@ impl<'de> serde::Deserialize<'de> for KeyPair {
         } else {
             let visitor = super::serde_util::Tuple32Visitor::new(
                 "raw 32 bytes KeyPair",
-                |data| unsafe {
-                    let ctx = Secp256k1::from_raw_all(ffi::secp256k1_context_no_precomp as *mut ffi::Context);
-                    KeyPair::from_seckey_slice(&ctx, data)
+                |data| {
+                    KeyPair::from_seckey_slice(SECP256K1, data)
                 }
             );
             d.deserialize_tuple(constants::SECRET_KEY_SIZE, visitor)
@@ -1630,6 +1627,7 @@ pub mod serde_keypair {
 #[cfg(test)]
 #[allow(unused_imports)]
 mod test {
+    use bitcoin_hashes::hex::ToHex;
     use super::*;
 
     use core::str::FromStr;
@@ -2431,6 +2429,16 @@ mod test {
         assert_tokens(&pk.readable(), &[Token::Str(PK_STR)]);
         assert_tokens(&pk.readable(), &[Token::String(PK_STR)]);
     }
+
+    #[test]
+    #[cfg(feature = "global-context")]
+    fn test_keypair_from_str() {
+        let ctx = crate::Secp256k1::new();
+        let keypair = KeyPair::new(&ctx, &mut thread_rng());
+        let msg = keypair.secret_key().secret_bytes().to_hex();
+        let parsed_key: KeyPair = msg.parse().unwrap();
+        assert_eq!(parsed_key, keypair);
+    }
 }
 
 #[cfg(bench)]