Add error reporting for null pointers

bindings/native/src/client.rs

@@ -12,7 +12,7 @@ use iota_sdk_bindings_core::{
     ClientMethod,
 };
 
-use crate::error::{set_last_error, Result};
+use crate::error::{set_last_error, Result, Error};
 
 pub struct Client {
     pub client: RustClient,
@@ -52,7 +52,7 @@ unsafe fn internal_destroy_client(client_ptr: *mut Client) -> Result<()> {
 
     if client_ptr.is_null() {
         log::error!("[Rust] Client pointer was null!");
-        return Ok(());
+        return Err(Error::from("pointer is null"));
     }
 
     let _ = Box::from_raw(client_ptr);

bindings/native/src/lib.rs

@@ -34,10 +34,10 @@ pub(crate) fn block_on<C: futures::Future>(cb: C) -> C::Output {
 
 unsafe fn internal_destroy_string(ptr: *mut c_char) -> Result<()> {
     if ptr.is_null() {
-        return Ok(());
+        log::error!("[Rust] String pointer was null!");
+        return Err(Error::from("pointer is null"));
     }
 
-
     let mut str = CString::from_raw(ptr);
     str.zeroize();
 

bindings/native/src/secret_manager.rs

@@ -14,7 +14,7 @@ use iota_sdk_bindings_core::{
 };
 use tokio::sync::RwLock;
 
-use crate::error::{set_last_error, Result};
+use crate::error::{set_last_error, Result, Error};
 
 pub struct SecretManager {
     pub secret_manager: Arc<RwLock<RustSecretManager>>,
@@ -51,7 +51,7 @@ unsafe fn internal_destroy_secret_manager(secret_manager_ptr: *mut SecretManager
 
     if secret_manager_ptr.is_null() {
         log::error!("[Rust] Secret Manager pointer was null!");
-        return Ok(());
+        return Err(Error::from("pointer is null"));
     }
 
     let _ = Box::from_raw(secret_manager_ptr);