feat: Création de la crate fsv, couche de haut niveau pour l'usage des librairies FSV

Cargo.lock

@@ -1523,6 +1523,15 @@ dependencies = [
  "libc",
 ]
 
+[[package]]
+name = "fsv"
+version = "0.1.0"
+dependencies = [
+ "fsv-sys",
+ "thiserror",
+ "utils",
+]
+
 [[package]]
 name = "fsv-sys"
 version = "0.1.0"

Cargo.toml

@@ -4,6 +4,7 @@ members = [
     "crates/backend",
     "crates/desktop",
     "crates/sesam-vitale",
+    "crates/fsv",
     "crates/fsv-sys",
     "crates/utils",
 ]

crates/fsv-sys/PROGRESS.md

@@ -0,0 +1,32 @@
+# État d'avancement de l'implémentation des bindings FSV
+
+| Module      | Progression                        |
+|-------------|------------------------------------|
+| [SSV](#ssv) | ![](https://geps.dev/progress/5)   |
+| [SGD](#sgd) | ![](https://geps.dev/progress/0)   |
+| [SRT](#srt) | ![](https://geps.dev/progress/0)   |
+| [STS](#sts) | ![](https://geps.dev/progress/0)   |
+
+## SSV
+
+| Fonctions implémentées |
+|------------------------|
+| SSV_InitLIB2           |
+| SSV_LireConfig         |
+| SSV_LireCartePS        |
+
+## SGD
+
+| Fonctions implémentées |
+|------------------------|
+
+
+## SRT
+
+| Fonctions implémentées |
+|------------------------|
+
+## STS
+
+| Fonctions implémentées |
+|------------------------|

crates/fsv-sys/README.md

@@ -1,5 +1,25 @@
 # FSV-sys, bindings Rust pour le package FSV SESAM-Vitale
 
+## Librairies FSV
+
+### Versions supportées
+
+| Version FSV |
+|-------------|
+| 1.40.14     |
+| 1.40.13     |
+
+### État d'avancement de l'implémentation des bindings FSV
+
+Les détails de l'avancement de l'implémentation des bindings FSV sont donnés dans le fichier [PROGRESS.md](PROGRESS.md)
+
+| Module      | Progression                        |
+|-------------|------------------------------------|
+| [SSV](#ssv) | ![](https://geps.dev/progress/5)   |
+| [SGD](#sgd) | ![](https://geps.dev/progress/0)   |
+| [SRT](#srt) | ![](https://geps.dev/progress/0)   |
+| [STS](#sts) | ![](https://geps.dev/progress/0)   |
+
 ## Utilisation
 
 ### Pré-requis

crates/fsv-sys/src/lib.rs

@@ -2,7 +2,9 @@
 #![allow(non_camel_case_types)]
 #![allow(non_snake_case)]
 
-#[derive(Debug)]
+use std::marker::PhantomData;
+
+#[derive(Debug, Clone)]
 pub enum SUPPORTED_FSV_VERSIONS {
     V1_40_14, // 1.40.14
     V1_40_13, // 1.40.13
@@ -17,30 +19,12 @@ impl SUPPORTED_FSV_VERSIONS {
     }
 }
 
-pub mod BINDINGS {
-    include!(concat!(env!("OUT_DIR"), "/bindings_1.40.13.rs"));
+mod BINDINGS_V1_40_14 {
+    include!(concat!(env!("OUT_DIR"), "/bindings_1.40.14.rs"));
 }
 
-// We need to keep the this use statement to get `ssv_function` macro working well
-use BINDINGS::*;
-
-/// Macro to generate a function that implements a call to an external function in BINDINGS
-macro_rules! ssv_function {
-    ($binding:ty, $func_name:ident, {$($arg_name:ident: $arg_type:ty),*}) => {
-        /// # Safety
-        /// This function is unsafe because it calls an external function through FFI.
-        /// The caller must ensure that the provided arguments are valid and that the
-        /// external function is safe to call.
-        pub unsafe fn $func_name(&self, $($arg_name: $arg_type),*) -> Result<u16, Error> {
-            let func_struct: libloading::Symbol<'_, $binding> =
-                unsafe { self.library.get(stringify!($binding).as_bytes())? };
-            let func = match *func_struct {
-                Some(func) => func,
-                None => return Err(Error::SymbolMissing(stringify!($binding))),
-            };
-            Ok(func($($arg_name),*))
-        }
-    };
+mod BINDINGS_V1_40_13 {
+    include!(concat!(env!("OUT_DIR"), "/bindings_1.40.13.rs"));
 }
 
 #[derive(thiserror::Error, Debug)]
@@ -51,15 +35,121 @@ pub enum Error {
     SymbolMissing(&'static str),
 }
 
+/// Macro to generate a function that implements a call to an external function in BINDINGS
+macro_rules! ssv_function {
+    ($binding:ty, $func_name:ident, {$($arg_name:ident: $arg_type:ty),*}) => {
+        /// # Safety
+        /// This function is unsafe because it calls an external function through FFI.
+        /// The caller must ensure that the provided arguments are valid and that the
+        /// external function is safe to call.
+        pub unsafe fn $func_name(&self, $($arg_name: $arg_type),*) -> Result<u16, Error> {
+            let symbol_name = match stringify!($binding)
+                .split(&[' ', ':'])
+                .last() {
+                    Some(name) => name,
+                    None => return Err(Error::SymbolMissing(stringify!($binding))),
+                };
+            let func_struct: libloading::Symbol<'_, $binding> =
+                unsafe { self.library.get(symbol_name.as_bytes())? };
+            let func = match *func_struct {
+                Some(func) => func,
+                None => return Err(Error::SymbolMissing(stringify!($binding))),
+            };
+            Ok(func($($arg_name),*))
+        }
+    };
+}
+
+/// `sealed::Sealed` trait is used to prevent external crates from implementing the LibVersion trait.
+mod sealed { pub trait Sealed {}}
+
 /// Wrapper around the SESAM-VITALE library
 /// This struct is responsible for loading the library and providing an interface to call its functions.
 /// The library is loaded at creation and kept in memory until the struct is dropped.
-#[derive(Debug)]
-pub struct SSVLibrary {
-    version: SUPPORTED_FSV_VERSIONS,
+pub trait SSVLibrary_Common {
+    fn new(path: &str) -> Result<Self, Error> where Self: Sized;
+}
+
+pub trait SSVLibrary_Version: sealed::Sealed {}
+
+pub struct V1_40_13 {}
+impl sealed::Sealed for V1_40_13 {}
+impl SSVLibrary_Version for V1_40_13 {}
+
+pub struct V1_40_14 {}
+impl sealed::Sealed for V1_40_14 {}
+impl SSVLibrary_Version for V1_40_14 {}
+
+pub struct SSVLibrary<Version: SSVLibrary_Version> {
+    _version: PhantomData<Version>,
     library: libloading::Library,
 }
 
+impl<Version: SSVLibrary_Version> SSVLibrary_Common for SSVLibrary<Version> {
+    fn new(path: &str) -> Result<Self, Error> {
+        let library = unsafe { libloading::Library::new(path)?};
+        Ok(Self {
+            _version: PhantomData,
+            library
+        })
+    }
+}
+
+impl SSVLibrary<V1_40_14> {
+    pub fn library(&self) -> &libloading::Library {
+        &self.library
+    }
+
+    ssv_function!(BINDINGS_V1_40_14::SSV_InitLIB2, ssv_init_lib2, {
+        pcFichierSesam: *const i8
+    });
+
+    ssv_function!(BINDINGS_V1_40_14::SSV_LireConfig, ssv_lire_config, {
+        pZDataOut: *mut *mut libc::c_void,
+        psTailleDataOut: *mut usize
+    });
+
+    ssv_function!(BINDINGS_V1_40_14::SSV_LireCartePS, ssv_lire_carte_ps, {
+        NomRessourcePS: *const i8,
+        NomRessourceLecteur: *const i8,
+        CodePorteurPS: *const i8,
+        pZDataOut: *mut *mut libc::c_void,
+        pTailleZone: *mut usize
+    });
+}
+
+impl SSVLibrary<V1_40_13> {
+    ssv_function!(BINDINGS_V1_40_13::SSV_InitLIB2, ssv_init_lib2, {
+        pcFichierSesam: *const i8
+    });
+
+    ssv_function!(BINDINGS_V1_40_13::SSV_LireConfig, ssv_lire_config, {
+        pZDataOut: *mut *mut libc::c_void,
+        psTailleDataOut: *mut usize
+    });
+
+    ssv_function!(BINDINGS_V1_40_13::SSV_LireCartePS, ssv_lire_carte_ps, {
+        NomRessourcePS: *const i8,
+        NomRessourceLecteur: *const i8,
+        CodePorteurPS: *const i8,
+        pZDataOut: *mut *mut libc::c_void,
+        pTailleZone: *mut usize
+    });
+}
+
+pub fn get_library_path(version: &SUPPORTED_FSV_VERSIONS) -> String {
+    let root_path = get_library_root_path();
+    let library_name = get_library_name();
+    let version = version.as_str();
+    format!("{root_path}/{version}/lib/{library_name}")
+}
+
+pub fn sesam_ini_path(version: &SUPPORTED_FSV_VERSIONS) -> String {
+    let root_path = get_sesam_ini_root_path();
+    let version = version.as_str();
+    format!("{root_path}/{version}/conf/sesam.ini")
+}
+
 fn get_library_name() -> &'static str {
     // TODO : Use libloading::library_filename to get platform-specific filename ?
     "libssvlux64.so"
@@ -73,48 +163,6 @@ fn get_sesam_ini_root_path() -> &'static str {
     "/etc/opt/santesocial/fsv"
 }
 
-impl SSVLibrary {
-    pub fn new(version: SUPPORTED_FSV_VERSIONS) -> Result<Self, Error> {
-        let library_path = Self::get_library_path(&version);
-        let library = unsafe { libloading::Library::new(library_path)? };
-        Ok(SSVLibrary { version, library })
-    }
-
-    pub fn library(&self) -> &libloading::Library {
-        &self.library
-    }
-
-    pub fn sesam_ini_path(&self) -> String {
-        let root_path = get_sesam_ini_root_path();
-        let version = self.version.as_str();
-        format!("{root_path}/{version}/conf/sesam.ini")
-    }
-
-    pub fn get_library_path(version: &SUPPORTED_FSV_VERSIONS) -> String {
-        let root_path = get_library_root_path();
-        let library_name = get_library_name();
-        let version = version.as_str();
-        format!("{root_path}/{version}/lib/{library_name}")
-    }
-
-    ssv_function!(SSV_InitLIB2, ssv_init_lib2, {
-        pcFichierSesam: *const i8
-    });
-
-    ssv_function!(SSV_LireConfig, ssv_lire_config, {
-        pZDataOut: *mut *mut libc::c_void,
-        psTailleDataOut: *mut usize
-    });
-
-    ssv_function!(SSV_LireCartePS, ssv_lire_carte_ps, {
-        NomRessourcePS: *const i8,
-        NomRessourceLecteur: *const i8,
-        CodePorteurPS: *const i8,
-        pZDataOut: *mut *mut libc::c_void,
-        pTailleZone: *mut usize
-    });
-}
-
 #[cfg(test)]
 mod test {
     use std::{ffi::CString, ptr};
@@ -123,22 +171,22 @@ mod test {
 
     #[test]
     fn test_initlib2() {
-        let ssv_library =
-            SSVLibrary::new(SUPPORTED_FSV_VERSIONS::V1_40_13).expect("SSVLibrary::new failed");
+        let lib_path = &get_library_path(&SUPPORTED_FSV_VERSIONS::V1_40_13);
+        let ssv_library = SSVLibrary::<V1_40_13>::new(lib_path).expect("SSVLibrary::new failed");
 
         let sesam_ini_str =
-            CString::new(ssv_library.sesam_ini_path()).expect("CString::new failed");
+            CString::new(sesam_ini_path(&SUPPORTED_FSV_VERSIONS::V1_40_13)).expect("CString::new failed");
         let result = unsafe { ssv_library.ssv_init_lib2(sesam_ini_str.as_ptr()) }.unwrap();
         assert_eq!(result, 0);
     }
 
     #[test]
     fn test_lire_config_and_carte_ps() {
-        let ssv_library =
-            SSVLibrary::new(SUPPORTED_FSV_VERSIONS::V1_40_13).expect("SSVLibrary::new failed");
+        let lib_path = &get_library_path(&SUPPORTED_FSV_VERSIONS::V1_40_13);
+        let ssv_library = SSVLibrary::<V1_40_13>::new(lib_path).expect("SSVLibrary::new failed");
 
         let sesam_ini_str =
-            CString::new(ssv_library.sesam_ini_path()).expect("CString::new failed");
+            CString::new(sesam_ini_path(&SUPPORTED_FSV_VERSIONS::V1_40_13)).expect("CString::new failed");
         let result = unsafe { ssv_library.ssv_init_lib2(sesam_ini_str.as_ptr()) }.unwrap();
         assert_eq!(result, 0);
 
@@ -146,6 +194,7 @@ mod test {
         let mut size: libc::size_t = 0;
         let result = unsafe { ssv_library.ssv_lire_config(&mut buffer_ptr, &mut size) }.unwrap();
         assert_eq!(result, 0);
+        unsafe { libc::free(buffer_ptr) };
 
         let nom_ressource_ps =
             CString::new("Gemalto PC Twin Reader (645D94C3) 00 00").expect("CString::new failed");
@@ -165,5 +214,6 @@ mod test {
         }
         .unwrap();
         assert_eq!(result, 0);
+        unsafe { libc::free(buffer_ptr) };
     }
 }

crates/fsv/Cargo.toml

@@ -0,0 +1,10 @@
+[package]
+name = "fsv"
+version = "0.1.0"
+edition = "2021"
+
+[dependencies]
+thiserror = "1.0.64"
+
+fsv-sys = { path = "../fsv-sys" }
+utils = { path = "../utils" }

crates/fsv/src/lib.rs

@@ -0,0 +1 @@
+mod ssv;

crates/fsv/src/ssv.rs

@@ -0,0 +1,83 @@
+use thiserror::Error;
+
+use fsv_sys::{get_library_path, Error as FsvError, SSVLibrary, SSVLibrary_Common, SUPPORTED_FSV_VERSIONS, V1_40_13, V1_40_14};
+
+#[derive(Error, Debug)]
+pub enum Error {
+    #[error(transparent)]
+    SysLibrary(#[from] FsvError)
+}
+
+/// Enum to hold the different versions of the SSV library
+enum SsvLibraryVersion {
+    V1_40_13(SSVLibrary<V1_40_13>),
+    V1_40_14(SSVLibrary<V1_40_14>),
+}
+
+/// Struct to hold the SSV library and access its functions
+pub struct SSV {
+    library: SsvLibraryVersion,
+}
+
+impl SSV {
+    fn new(version: SUPPORTED_FSV_VERSIONS) -> Result<Self, Error> {
+        let library = match version {
+            SUPPORTED_FSV_VERSIONS::V1_40_13 => {
+                let lib_path = get_library_path(&version);
+                let library = SSVLibrary::<V1_40_13>::new(&lib_path)?;
+                SsvLibraryVersion::V1_40_13(library)
+            },
+            SUPPORTED_FSV_VERSIONS::V1_40_14 => {
+                let lib_path = get_library_path(&version);
+                let library = SSVLibrary::<V1_40_14>::new(&lib_path)?;
+                SsvLibraryVersion::V1_40_14(library)
+            },
+        };
+        Ok(Self { 
+            library,
+        })
+    }
+
+    /// # Initialize the SSV library
+    /// Implement: SSV_InitLIB2
+    pub fn init_library(&self, sesam_ini_path: &str) -> Result<(), Error> {
+        let sesam_ini_path = std::ffi::CString::new(sesam_ini_path).expect("CString::new failed");
+        match &self.library {
+            SsvLibraryVersion::V1_40_13(library) => {
+                let result = unsafe { library.ssv_init_lib2(sesam_ini_path.as_ptr()) }?;
+                println!("SSV_InitLIB2 result: {}", result);
+            },
+            SsvLibraryVersion::V1_40_14(library) => {
+                let result = unsafe { library.ssv_init_lib2(sesam_ini_path.as_ptr()) }?;
+                println!("SSV_InitLIB2 result: {}", result);
+            },
+        }
+        Ok(())
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use std::env;
+
+    use utils::config::load_config;
+
+    use super::*;
+
+    fn init() -> SSV {
+        load_config().unwrap();
+        SSV::new(SUPPORTED_FSV_VERSIONS::V1_40_13).expect("SSV::new failed")
+    }
+
+    #[test]
+    fn test_init_library() {
+        let lib = init();
+        let sesam_ini_path = env::var("SESAM_INI_PATH").expect("SESAM_INI_PATH must be set");
+        lib.init_library(&sesam_ini_path).expect("init_library failed");
+    }
+
+    #[test]
+    fn test_ssv() {
+        assert_eq!("ssv", "ssv");
+    }
+}