Make rustc stop complaining about unused `Box::from_raw`s

[ldk-c-bindings] / lightning-c-bindings / src / c_types / mod.rs

diff --git a/lightning-c-bindings/src/c_types/mod.rs b/lightning-c-bindings/src/c_types/mod.rs
index 471b067ac94fab9d19e1e6f034fdf49d1eea8cc2..71616375aabfc10a5784650328cf0803f45c24f1 100644 (file)
--- a/lightning-c-bindings/src/c_types/mod.rs
+++ b/lightning-c-bindings/src/c_types/mod.rs
@@ -11,6 +11,7 @@ use bitcoin::secp256k1::ecdsa::Signature as SecpSignature;
 use bitcoin::secp256k1::Error as SecpError;
 use bitcoin::secp256k1::ecdsa::RecoveryId;
 use bitcoin::secp256k1::ecdsa::RecoverableSignature as SecpRecoverableSignature;
+use bitcoin::secp256k1::Scalar as SecpScalar;
 use bitcoin::bech32;
 use bitcoin::util::address;
 
@@ -24,6 +25,8 @@ pub(crate) use core2::io::{self, Cursor, Read};
 #[cfg(feature = "no-std")]
 use alloc::{boxed::Box, vec::Vec, string::String};
 
+use core::convert::TryFrom;
+
 #[repr(C)]
 /// A dummy struct of which an instance must never exist.
 /// This corresponds to the Rust type `Infallible`, or, in unstable rust, `!`
@@ -53,10 +56,12 @@ impl Into<bech32::u5> for u5 {
 pub struct WitnessVersion(u8);
 
 impl From<address::WitnessVersion> for WitnessVersion {
-       fn from(o: address::WitnessVersion) -> Self { Self(o.into_num()) }
+       fn from(o: address::WitnessVersion) -> Self { Self(o.to_num()) }
 }
 impl Into<address::WitnessVersion> for WitnessVersion {
-       fn into(self) -> address::WitnessVersion { address::WitnessVersion::from_num(self.0).expect("WitnessVersion objects must be in the range 0..=16") }
+       fn into(self) -> address::WitnessVersion {
+               address::WitnessVersion::try_from(self.0).expect("WitnessVersion objects must be in the range 0..=16")
+       }
 }
 
 #[derive(Clone)]
@@ -145,6 +150,28 @@ impl RecoverableSignature {
        }
 }
 
+#[repr(C)]
+#[derive(Clone)]
+/// Represents a scalar value between zero and the secp256k1 curve order, in big endian.
+pub struct BigEndianScalar {
+       /// The bytes of the scalar value.
+       pub big_endian_bytes: [u8; 32],
+}
+impl BigEndianScalar {
+       pub(crate) fn from_rust(scalar: &SecpScalar) -> Self {
+               Self { big_endian_bytes: scalar.to_be_bytes() }
+       }
+       pub(crate) fn into_rust(&self) -> SecpScalar {
+               SecpScalar::from_be_bytes(self.big_endian_bytes).expect("Scalar greater than the curve order")
+       }
+}
+
+#[no_mangle]
+/// Convenience function for constructing a new BigEndianScalar
+pub extern "C" fn BigEndianScalar_new(big_endian_bytes: ThirtyTwoBytes) -> BigEndianScalar {
+       BigEndianScalar { big_endian_bytes: big_endian_bytes.data }
+}
+
 #[repr(C)]
 #[derive(Copy, Clone)]
 /// Represents an error returned from libsecp256k1 during validation of some secp256k1 data
@@ -288,52 +315,57 @@ pub enum IOError {
        Other,
        UnexpectedEof,
 }
-#[cfg(feature = "std")]
 impl IOError {
-       pub(crate) fn from_rust(err: std::io::Error) -> Self {
-               match err.kind() {
-                       std::io::ErrorKind::NotFound => IOError::NotFound,
-                       std::io::ErrorKind::PermissionDenied => IOError::PermissionDenied,
-                       std::io::ErrorKind::ConnectionRefused => IOError::ConnectionRefused,
-                       std::io::ErrorKind::ConnectionReset => IOError::ConnectionReset,
-                       std::io::ErrorKind::ConnectionAborted => IOError::ConnectionAborted,
-                       std::io::ErrorKind::NotConnected => IOError::NotConnected,
-                       std::io::ErrorKind::AddrInUse => IOError::AddrInUse,
-                       std::io::ErrorKind::AddrNotAvailable => IOError::AddrNotAvailable,
-                       std::io::ErrorKind::BrokenPipe => IOError::BrokenPipe,
-                       std::io::ErrorKind::AlreadyExists => IOError::AlreadyExists,
-                       std::io::ErrorKind::WouldBlock => IOError::WouldBlock,
-                       std::io::ErrorKind::InvalidInput => IOError::InvalidInput,
-                       std::io::ErrorKind::InvalidData => IOError::InvalidData,
-                       std::io::ErrorKind::TimedOut => IOError::TimedOut,
-                       std::io::ErrorKind::WriteZero => IOError::WriteZero,
-                       std::io::ErrorKind::Interrupted => IOError::Interrupted,
-                       std::io::ErrorKind::Other => IOError::Other,
-                       std::io::ErrorKind::UnexpectedEof => IOError::UnexpectedEof,
+       pub(crate) fn from_rust_kind(err: io::ErrorKind) -> Self {
+               match err {
+                       io::ErrorKind::NotFound => IOError::NotFound,
+                       io::ErrorKind::PermissionDenied => IOError::PermissionDenied,
+                       io::ErrorKind::ConnectionRefused => IOError::ConnectionRefused,
+                       io::ErrorKind::ConnectionReset => IOError::ConnectionReset,
+                       io::ErrorKind::ConnectionAborted => IOError::ConnectionAborted,
+                       io::ErrorKind::NotConnected => IOError::NotConnected,
+                       io::ErrorKind::AddrInUse => IOError::AddrInUse,
+                       io::ErrorKind::AddrNotAvailable => IOError::AddrNotAvailable,
+                       io::ErrorKind::BrokenPipe => IOError::BrokenPipe,
+                       io::ErrorKind::AlreadyExists => IOError::AlreadyExists,
+                       io::ErrorKind::WouldBlock => IOError::WouldBlock,
+                       io::ErrorKind::InvalidInput => IOError::InvalidInput,
+                       io::ErrorKind::InvalidData => IOError::InvalidData,
+                       io::ErrorKind::TimedOut => IOError::TimedOut,
+                       io::ErrorKind::WriteZero => IOError::WriteZero,
+                       io::ErrorKind::Interrupted => IOError::Interrupted,
+                       io::ErrorKind::Other => IOError::Other,
+                       io::ErrorKind::UnexpectedEof => IOError::UnexpectedEof,
                        _ => IOError::Other,
                }
        }
-       pub(crate) fn to_rust(&self) -> std::io::Error {
-               std::io::Error::new(match self {
-                       IOError::NotFound => std::io::ErrorKind::NotFound,
-                       IOError::PermissionDenied => std::io::ErrorKind::PermissionDenied,
-                       IOError::ConnectionRefused => std::io::ErrorKind::ConnectionRefused,
-                       IOError::ConnectionReset => std::io::ErrorKind::ConnectionReset,
-                       IOError::ConnectionAborted => std::io::ErrorKind::ConnectionAborted,
-                       IOError::NotConnected => std::io::ErrorKind::NotConnected,
-                       IOError::AddrInUse => std::io::ErrorKind::AddrInUse,
-                       IOError::AddrNotAvailable => std::io::ErrorKind::AddrNotAvailable,
-                       IOError::BrokenPipe => std::io::ErrorKind::BrokenPipe,
-                       IOError::AlreadyExists => std::io::ErrorKind::AlreadyExists,
-                       IOError::WouldBlock => std::io::ErrorKind::WouldBlock,
-                       IOError::InvalidInput => std::io::ErrorKind::InvalidInput,
-                       IOError::InvalidData => std::io::ErrorKind::InvalidData,
-                       IOError::TimedOut => std::io::ErrorKind::TimedOut,
-                       IOError::WriteZero => std::io::ErrorKind::WriteZero,
-                       IOError::Interrupted => std::io::ErrorKind::Interrupted,
-                       IOError::Other => std::io::ErrorKind::Other,
-                       IOError::UnexpectedEof => std::io::ErrorKind::UnexpectedEof,
-               }, "")
+       pub(crate) fn from_rust(err: io::Error) -> Self {
+               Self::from_rust_kind(err.kind())
+       }
+       pub(crate) fn to_rust_kind(&self) -> io::ErrorKind {
+               match self {
+                       IOError::NotFound => io::ErrorKind::NotFound,
+                       IOError::PermissionDenied => io::ErrorKind::PermissionDenied,
+                       IOError::ConnectionRefused => io::ErrorKind::ConnectionRefused,
+                       IOError::ConnectionReset => io::ErrorKind::ConnectionReset,
+                       IOError::ConnectionAborted => io::ErrorKind::ConnectionAborted,
+                       IOError::NotConnected => io::ErrorKind::NotConnected,
+                       IOError::AddrInUse => io::ErrorKind::AddrInUse,
+                       IOError::AddrNotAvailable => io::ErrorKind::AddrNotAvailable,
+                       IOError::BrokenPipe => io::ErrorKind::BrokenPipe,
+                       IOError::AlreadyExists => io::ErrorKind::AlreadyExists,
+                       IOError::WouldBlock => io::ErrorKind::WouldBlock,
+                       IOError::InvalidInput => io::ErrorKind::InvalidInput,
+                       IOError::InvalidData => io::ErrorKind::InvalidData,
+                       IOError::TimedOut => io::ErrorKind::TimedOut,
+                       IOError::WriteZero => io::ErrorKind::WriteZero,
+                       IOError::Interrupted => io::ErrorKind::Interrupted,
+                       IOError::Other => io::ErrorKind::Other,
+                       IOError::UnexpectedEof => io::ErrorKind::UnexpectedEof,
+               }
+       }
+       pub(crate) fn to_rust(&self) -> io::Error {
+               io::Error::new(self.to_rust_kind(), "")
        }
 }
 
@@ -644,10 +676,10 @@ impl<O, E> Drop for CResultTempl<O, E> {
        fn drop(&mut self) {
                if self.result_ok {
                        if unsafe { !self.contents.result.is_null() } {
-                               unsafe { Box::from_raw(self.contents.result) };
+                               let _ = unsafe { Box::from_raw(self.contents.result) };
                        }
                } else if unsafe { !self.contents.err.is_null() } {
-                       unsafe { Box::from_raw(self.contents.err) };
+                       let _ = unsafe { Box::from_raw(self.contents.err) };
                }
        }
 }
@@ -748,7 +780,7 @@ impl<T> SmartPtr<T> {
 impl<T> Drop for SmartPtr<T> {
        fn drop(&mut self) {
                if self.ptr != core::ptr::null_mut() {
-                       unsafe { Box::from_raw(self.ptr); }
+                       let _ = unsafe { Box::from_raw(self.ptr) };
                }
        }
 }