Update depreciated code

This commit is contained in:
Kevin Butler 2015-02-20 21:04:01 +00:00
parent 00e4941a75
commit cb0e1688c8
7 changed files with 25 additions and 25 deletions

View File

@ -1,5 +1,5 @@
use libc::{c_int, c_ulong, c_void}; use libc::{c_int, c_ulong, c_void};
use std::ffi::{CString, c_str_to_bytes}; use std::ffi::{CStr, CString};
use std::cmp::Ordering; use std::cmp::Ordering;
use std::{fmt, ptr}; use std::{fmt, ptr};
@ -88,7 +88,7 @@ impl BigNum {
pub fn from_dec_str(s: &str) -> Result<BigNum, SslError> { pub fn from_dec_str(s: &str) -> Result<BigNum, SslError> {
BigNum::new().and_then(|v| unsafe { BigNum::new().and_then(|v| unsafe {
let c_str = CString::from_slice(s.as_bytes()); let c_str = CString::new(s.as_bytes()).unwrap();
try_ssl!(ffi::BN_dec2bn(v.raw_ptr(), c_str.as_ptr())); try_ssl!(ffi::BN_dec2bn(v.raw_ptr(), c_str.as_ptr()));
Ok(v) Ok(v)
}) })
@ -96,7 +96,7 @@ impl BigNum {
pub fn from_hex_str(s: &str) -> Result<BigNum, SslError> { pub fn from_hex_str(s: &str) -> Result<BigNum, SslError> {
BigNum::new().and_then(|v| unsafe { BigNum::new().and_then(|v| unsafe {
let c_str = CString::from_slice(s.as_bytes()); let c_str = CString::new(s.as_bytes()).unwrap();
try_ssl!(ffi::BN_hex2bn(v.raw_ptr(), c_str.as_ptr())); try_ssl!(ffi::BN_hex2bn(v.raw_ptr(), c_str.as_ptr()));
Ok(v) Ok(v)
}) })
@ -421,7 +421,7 @@ impl BigNum {
unsafe { unsafe {
let buf = ffi::BN_bn2dec(self.raw()); let buf = ffi::BN_bn2dec(self.raw());
assert!(!buf.is_null()); assert!(!buf.is_null());
let str = String::from_utf8(c_str_to_bytes(&buf).to_vec()).unwrap(); let str = String::from_utf8(CStr::from_ptr(buf).to_bytes().to_vec()).unwrap();
ffi::CRYPTO_free(buf as *mut c_void); ffi::CRYPTO_free(buf as *mut c_void);
str str
} }
@ -431,7 +431,7 @@ impl BigNum {
unsafe { unsafe {
let buf = ffi::BN_bn2hex(self.raw()); let buf = ffi::BN_bn2hex(self.raw());
assert!(!buf.is_null()); assert!(!buf.is_null());
let str = String::from_utf8(c_str_to_bytes(&buf).to_vec()).unwrap(); let str = String::from_utf8(CStr::from_ptr(buf).to_bytes().to_vec()).unwrap();
ffi::CRYPTO_free(buf as *mut c_void); ffi::CRYPTO_free(buf as *mut c_void);
str str
} }

View File

@ -253,7 +253,7 @@ mod tests {
]; ];
let mut h = HMAC::new(MD5, &*tests[0].0); let mut h = HMAC::new(MD5, &*tests[0].0);
for i in 0..100us { for i in 0..100usize {
let test = &tests[i % 2]; let test = &tests[i % 2];
test_hmac_recycle(&mut h, test); test_hmac_recycle(&mut h, test);
} }
@ -351,7 +351,7 @@ mod tests {
]; ];
let mut h = HMAC::new(SHA1, &*tests[0].0); let mut h = HMAC::new(SHA1, &*tests[0].0);
for i in 0..100us { for i in 0..100usize {
let test = &tests[i % 2]; let test = &tests[i % 2];
test_hmac_recycle(&mut h, test); test_hmac_recycle(&mut h, test);
} }
@ -382,7 +382,7 @@ mod tests {
// recycle test // recycle test
let mut h = HMAC::new(ty, &*tests[5].0); let mut h = HMAC::new(ty, &*tests[5].0);
for i in 0..100us { for i in 0..100usize {
let test = &tests[4 + i % 2]; let test = &tests[4 + i % 2];
let tup = (test.0.clone(), test.1.clone(), results[4 + i % 2].clone()); let tup = (test.0.clone(), test.1.clone(), results[4 + i % 2].clone());
test_hmac_recycle(&mut h, &tup); test_hmac_recycle(&mut h, &tup);

View File

@ -1,4 +1,4 @@
#![feature(unsafe_destructor, core, old_io, std_misc, libc, hash, old_path)] #![feature(unsafe_destructor, core, old_io, std_misc, libc, old_path)]
#![crate_name="openssl"] #![crate_name="openssl"]
#![crate_type="rlib"] #![crate_type="rlib"]
#![crate_type="dylib"] #![crate_type="dylib"]

View File

@ -4,7 +4,7 @@ pub use self::OpensslError::*;
use libc::c_ulong; use libc::c_ulong;
use std::error; use std::error;
use std::fmt; use std::fmt;
use std::ffi::c_str_to_bytes; use std::ffi::CStr;
use std::old_io::IoError; use std::old_io::IoError;
use ffi; use ffi;
@ -75,21 +75,21 @@ pub enum OpensslError {
fn get_lib(err: c_ulong) -> String { fn get_lib(err: c_ulong) -> String {
unsafe { unsafe {
let bytes = c_str_to_bytes(&ffi::ERR_lib_error_string(err)).to_vec(); let bytes = CStr::from_ptr(ffi::ERR_lib_error_string(err)).to_bytes().to_vec();
String::from_utf8(bytes).unwrap() String::from_utf8(bytes).unwrap()
} }
} }
fn get_func(err: c_ulong) -> String { fn get_func(err: c_ulong) -> String {
unsafe { unsafe {
let bytes = c_str_to_bytes(&ffi::ERR_func_error_string(err)).to_vec(); let bytes = CStr::from_ptr(ffi::ERR_func_error_string(err)).to_bytes().to_vec();
String::from_utf8(bytes).unwrap() String::from_utf8(bytes).unwrap()
} }
} }
fn get_reason(err: c_ulong) -> String { fn get_reason(err: c_ulong) -> String {
unsafe { unsafe {
let bytes = c_str_to_bytes(&ffi::ERR_reason_error_string(err)).to_vec(); let bytes = CStr::from_ptr(ffi::ERR_reason_error_string(err)).to_bytes().to_vec();
String::from_utf8(bytes).unwrap() String::from_utf8(bytes).unwrap()
} }
} }

View File

@ -1,5 +1,5 @@
use libc::{c_int, c_void, c_long}; use libc::{c_int, c_void, c_long};
use std::ffi::{CString, c_str_to_bytes}; use std::ffi::{CStr, CString};
use std::old_io::{IoResult, IoError, EndOfFile, OtherIoError, Stream, Reader, Writer}; use std::old_io::{IoResult, IoError, EndOfFile, OtherIoError, Stream, Reader, Writer};
use std::mem; use std::mem;
use std::fmt; use std::fmt;
@ -247,7 +247,7 @@ impl SslContext {
pub fn set_CA_file(&mut self, file: &Path) -> Option<SslError> { pub fn set_CA_file(&mut self, file: &Path) -> Option<SslError> {
wrap_ssl_result( wrap_ssl_result(
unsafe { unsafe {
let file = CString::from_slice(file.as_vec()); let file = CString::new(file.as_vec()).unwrap();
ffi::SSL_CTX_load_verify_locations(*self.ctx, file.as_ptr(), ptr::null()) ffi::SSL_CTX_load_verify_locations(*self.ctx, file.as_ptr(), ptr::null())
}) })
} }
@ -257,7 +257,7 @@ impl SslContext {
file_type: X509FileType) -> Option<SslError> { file_type: X509FileType) -> Option<SslError> {
wrap_ssl_result( wrap_ssl_result(
unsafe { unsafe {
let file = CString::from_slice(file.as_vec()); let file = CString::new(file.as_vec()).unwrap();
ffi::SSL_CTX_use_certificate_file(*self.ctx, file.as_ptr(), file_type as c_int) ffi::SSL_CTX_use_certificate_file(*self.ctx, file.as_ptr(), file_type as c_int)
}) })
} }
@ -267,7 +267,7 @@ impl SslContext {
file_type: X509FileType) -> Option<SslError> { file_type: X509FileType) -> Option<SslError> {
wrap_ssl_result( wrap_ssl_result(
unsafe { unsafe {
let file = CString::from_slice(file.as_vec()); let file = CString::new(file.as_vec()).unwrap();
ffi::SSL_CTX_use_PrivateKey_file(*self.ctx, file.as_ptr(), file_type as c_int) ffi::SSL_CTX_use_PrivateKey_file(*self.ctx, file.as_ptr(), file_type as c_int)
}) })
} }
@ -275,7 +275,7 @@ impl SslContext {
pub fn set_cipher_list(&mut self, cipher_list: &str) -> Option<SslError> { pub fn set_cipher_list(&mut self, cipher_list: &str) -> Option<SslError> {
wrap_ssl_result( wrap_ssl_result(
unsafe { unsafe {
let cipher_list = CString::from_slice(cipher_list.as_bytes()); let cipher_list = CString::new(cipher_list.as_bytes()).unwrap();
ffi::SSL_CTX_set_cipher_list(*self.ctx, cipher_list.as_ptr()) ffi::SSL_CTX_set_cipher_list(*self.ctx, cipher_list.as_ptr())
}) })
} }
@ -378,7 +378,7 @@ impl Ssl {
// #define SSL_set_tlsext_host_name(s,name) \ // #define SSL_set_tlsext_host_name(s,name) \
// SSL_ctrl(s,SSL_CTRL_SET_TLSEXT_HOSTNAME,TLSEXT_NAMETYPE_host_name,(char *)name) // SSL_ctrl(s,SSL_CTRL_SET_TLSEXT_HOSTNAME,TLSEXT_NAMETYPE_host_name,(char *)name)
let hostname = CString::from_slice(hostname.as_bytes()); let hostname = CString::new(hostname.as_bytes()).unwrap();
ffi::SSL_ctrl(*self.ssl, ffi::SSL_CTRL_SET_TLSEXT_HOSTNAME, ffi::SSL_ctrl(*self.ssl, ffi::SSL_CTRL_SET_TLSEXT_HOSTNAME,
ffi::TLSEXT_NAMETYPE_host_name, ffi::TLSEXT_NAMETYPE_host_name,
hostname.as_ptr() as *mut c_void) hostname.as_ptr() as *mut c_void)
@ -542,7 +542,7 @@ impl<S: Stream> SslStream<S> {
let meth = unsafe { ffi::SSL_COMP_get_name(ptr) }; let meth = unsafe { ffi::SSL_COMP_get_name(ptr) };
let s = unsafe { let s = unsafe {
String::from_utf8(c_str_to_bytes(&meth).to_vec()).unwrap() String::from_utf8(CStr::from_ptr(meth).to_bytes().to_vec()).unwrap()
}; };
Some(s) Some(s)

View File

@ -1,7 +1,7 @@
use serialize::hex::FromHex; use serialize::hex::FromHex;
use std::old_io::net::tcp::TcpStream; use std::old_io::net::tcp::TcpStream;
use std::old_io::{Writer}; use std::old_io::{Writer};
use std::thread::Thread; use std::thread;
use crypto::hash::Type::{SHA256}; use crypto::hash::Type::{SHA256};
use ssl::SslMethod::Sslv23; use ssl::SslMethod::Sslv23;
@ -199,7 +199,7 @@ fn test_clone() {
let stream = TcpStream::connect("127.0.0.1:15418").unwrap(); let stream = TcpStream::connect("127.0.0.1:15418").unwrap();
let mut stream = SslStream::new(&SslContext::new(Sslv23).unwrap(), stream).unwrap(); let mut stream = SslStream::new(&SslContext::new(Sslv23).unwrap(), stream).unwrap();
let mut stream2 = stream.clone(); let mut stream2 = stream.clone();
let _t = Thread::spawn(move || { let _t = thread::spawn(move || {
stream2.write_all("GET /\r\n\r\n".as_bytes()).unwrap(); stream2.write_all("GET /\r\n\r\n".as_bytes()).unwrap();
stream2.flush().unwrap(); stream2.flush().unwrap();
}); });

View File

@ -247,7 +247,7 @@ impl X509Generator {
let mut ctx: ffi::X509V3_CTX = mem::zeroed(); let mut ctx: ffi::X509V3_CTX = mem::zeroed();
ffi::X509V3_set_ctx(&mut ctx, x509, x509, ffi::X509V3_set_ctx(&mut ctx, x509, x509,
ptr::null_mut(), ptr::null_mut(), 0); ptr::null_mut(), ptr::null_mut(), 0);
let value = CString::from_slice(value.as_bytes()); let value = CString::new(value.as_bytes()).unwrap();
let ext = ffi::X509V3_EXT_conf_nid(ptr::null_mut(), let ext = ffi::X509V3_EXT_conf_nid(ptr::null_mut(),
mem::transmute(&ctx), mem::transmute(&ctx),
extension, extension,
@ -265,8 +265,8 @@ impl X509Generator {
fn add_name(name: *mut ffi::X509_NAME, key: &str, value: &str) -> Result<(), SslError> { fn add_name(name: *mut ffi::X509_NAME, key: &str, value: &str) -> Result<(), SslError> {
let value_len = value.len() as c_int; let value_len = value.len() as c_int;
lift_ssl!(unsafe { lift_ssl!(unsafe {
let key = CString::from_slice(key.as_bytes()); let key = CString::new(key.as_bytes()).unwrap();
let value = CString::from_slice(value.as_bytes()); let value = CString::new(value.as_bytes()).unwrap();
ffi::X509_NAME_add_entry_by_txt(name, key.as_ptr(), ffi::MBSTRING_UTF8, ffi::X509_NAME_add_entry_by_txt(name, key.as_ptr(), ffi::MBSTRING_UTF8,
value.as_ptr(), value_len, -1, 0) value.as_ptr(), value_len, -1, 0)
}) })