diff --git a/Cargo.toml b/Cargo.toml index 4af5bac9e1c6f4b9753b0dcac95ec96d40c550a6..4c61dd7e770f49547574e442a3dafc71eec3a900 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -3,7 +3,6 @@ name = "openssl" version = "0.0.0" authors = ["Steven Fackler BigNum { BigNum::new_from(0).unwrap() } + fn is_zero(&self) -> bool { unsafe { - ffi::BN_is_zero(self.raw()) == 1 + // It is raw contents of BN_is_zero macro + (*self.raw()).top == 0 } } } diff --git a/src/ffi.rs b/src/ffi.rs index 93a252741f958089b3fa44d83f4f0ae2965b1d6a..0801577eac2f2e3a9ef7a5886f5a3df273fa6c67 100755 --- a/src/ffi.rs +++ b/src/ffi.rs @@ -3,6 +3,8 @@ use libc::{c_void, c_int, c_char, c_ulong, c_long, c_uint, c_uchar, size_t}; use std::ptr; +pub use bn::BIGNUM; + pub type ASN1_INTEGER = c_void; pub type ASN1_STRING = c_void; pub type ASN1_TIME = c_void; @@ -28,16 +30,6 @@ pub type X509_NAME = c_void; pub type X509_REQ = c_void; pub type X509_STORE_CTX = c_void; -#[allow(dead_code)] -#[repr(C)] -pub struct BIGNUM { - d: *mut c_void, - top: c_int, - dmax: c_int, - pub neg: c_int, - flags: c_int, -} - #[repr(C)] pub struct EVP_MD_CTX { digest: *mut EVP_MD, @@ -189,15 +181,6 @@ extern {} #[link(name="wsock32")] extern { } -/* Since the openssl BN_is_zero is sometimes a macro, this wrapper is necessary. */ -pub unsafe fn BN_is_zero(a: *mut BIGNUM) -> c_int { bn_is_zero(a) } - -/* Special import from native/bn_is_zero.c */ -#[link(name = "wrapped", kind = "static")] -extern "C" { - pub fn bn_is_zero(a: *mut BIGNUM) -> c_int; -} - // Functions converted from macros pub unsafe fn BIO_eof(b: *mut BIO) -> bool { BIO_ctrl(b, BIO_CTRL_EOF, 0, ptr::null_mut()) == 1 diff --git a/src/ssl/tests.rs b/src/ssl/tests.rs index f98352be6e7da8c86b576fa547ee4bdbc20bc362..a68d496746caf54e394a2925002a296f85a74cfb 100644 --- a/src/ssl/tests.rs +++ b/src/ssl/tests.rs @@ -237,16 +237,3 @@ fn test_cert_gen() { // FIXME: check data in result to be correct, needs implementation // of X509 getters } - -#[test] -fn test_bn_is_zero() { - use ffi; - use std::ptr; - - unsafe { - let bn = ffi::BN_new(); - assert!(bn != ptr::null_mut()); - // Just make sure it is linked and resolved correctly - ffi::BN_is_zero(bn); - } -}