ci: GitHub actions add windows-latest test build

avast · Jan 9, 2024 · 5afb621 · 5afb621
1 parent d1ff982
commit 5afb621
Show file tree

Hide file tree

Showing 5 changed files with 86 additions and 14 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -20,15 +20,19 @@ jobs:
     - name: Install YARA deps
       run: sudo apt-get install -y automake libtool make gcc pkg-config libssl-dev libjansson-dev libmagic-dev
     - name: Build YARA
-      run: cd yari-sys/yara && ./bootstrap.sh && CFLAGS="-fPIC" ./configure --enable-debug --disable-shared --enable-static --enable-cuckoo --enable-magic --enable-dotnet --with-crypto && make
+      run: |
+        cd yari-sys/yara
+        ./bootstrap.sh
+        CFLAGS="-fPIC" ./configure --enable-debug --disable-shared --enable-static --enable-cuckoo --enable-magic --enable-dotnet --with-crypto
+        make
     - name: Build
-      run: cargo build --verbose
+      run: cargo build
     - name: Run Rust tests
-      run: cargo test --verbose
+      run: cargo test
     - name: Run tests with yari using bundled bindings
       run: |
         cargo clean
-        YARI_USE_BUNDLED_BINDINGS=1 cargo test --verbose
+        YARI_USE_BUNDLED_BINDINGS=1 cargo test
     - uses: actions/setup-python@v4
       with:
         python-version: '3.10'
@@ -48,19 +52,69 @@ jobs:
       uses: actions/checkout@v3
       with:
         submodules: 'true'
-    - name: Install (Yara) Dependencies
+    - name: Install YARA deps
       run: brew install autoconf automake libtool pkg-config jansson libmagic openssl [email protected] [email protected] [email protected]
     - name: Build Yara
-      run: cd yari-sys/yara && ./bootstrap.sh && CFLAGS="-fPIC -I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" ./configure --enable-debug --disable-shared --enable-static --enable-cuckoo --enable-magic --enable-dotnet --with-crypto && make
+      run: |
+        cd yari-sys/yara
+        ./bootstrap.sh
+        CFLAGS="-fPIC -I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" ./configure --enable-debug --disable-shared --enable-static --enable-cuckoo --enable-magic --enable-dotnet --with-crypto
+        make
     - name: Build Yari (yari-sys)
       run: cd yari-sys && CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" cargo build --release
     - name: Build Yari (yari-cli)
       run: cd yari-cli && CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" cargo build --release
-    - name: Build Yari (yari-py)
-      run: cd yari-py && python3.12 -mvenv .venv && .venv/bin/pip install maturin pytest && CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" .venv/bin/maturin build --release -i python3.10 -i python3.11 -i python3.12
     - name: Run Rust tests
-      run: CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" cargo test --verbose
+      run: CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" cargo test
     - name: Run Rust tests with bundled bindings
-      run: cargo clean && YARI_USE_BUNDLED_BINDINGS=1 CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" cargo test --verbose
+      run: |
+        cargo clean
+        YARI_USE_BUNDLED_BINDINGS=1 CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" cargo test
     - name: Run Python tests
-      run: cd yari-py && .venv/bin/maturin develop && .venv/bin/pytest
+      run: |
+        cd yari-py
+        python3.12 -m venv .venv 
+        .venv/bin/pip install maturin pytest 
+        CFLAGS="-I$(brew --prefix)/include" LDFLAGS="-L$(brew --prefix)/lib $LDFLAGS" LIBRARY_PATH="$(brew --prefix)/lib:$LIBRARY_PATH" .venv/bin/maturin build -i python3.10 -i python3.11 -i python3.12
+        .venv/bin/maturin build 
+        .venv/bin/maturin develop 
+        .venv/bin/pytest
+  build-windows:
+    runs-on: windows-latest
+    defaults:
+      run:
+        shell: bash
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          submodules: 'recursive'
+      - uses: actions/setup-python@v3
+        with:
+          python-version: '3.10'
+      - name: Setup Nuget.exe
+        uses: nuget/setup-nuget@v1
+        with:
+          nuget-version: 'latest'
+      - name: Setup msbuild
+        uses: microsoft/[email protected]
+      - name: Install YARA Dependencies
+        run: nuget.exe restore windows/vs2017/yara.sln
+        working-directory: ./yari-sys/yara
+      - name: Build YARA
+        run: msbuild /m /p:Platform=x64 /p:Configuration=Release /p:PlatformToolset="v142" /p:WindowsTargetPlatformVersion="10.0.19041.0" windows/vs2017/yara.sln
+        working-directory: ./yari-sys/yara
+        shell: cmd
+      - name: Build yari-sys
+        run: |
+          cargo build
+          RUST_LOG=debug cargo test -- --nocapture --show-output --test-threads 1
+        working-directory: ./yari-sys
+      - name: Build Python wheels
+        run: |
+          pip install --upgrade pip
+          python -m venv env
+          . env/Scripts/activate
+          pip install maturin pytest
+          maturin develop
+          pytest tests
+        working-directory: ./yari-py
diff --git a/yari-sys/build.rs b/yari-sys/build.rs
@@ -32,6 +32,8 @@ fn link_lib(name: &str) {
 #[cfg(target_os = "windows")]
 fn link_windows() {
     link_lib("libyara64");
+    link_lib("crypt32");
+    link_lib("user32");
 }
 
 #[cfg(any(target_os = "linux", target_os = "macos"))]

diff --git a/yari-sys/src/lib.rs b/yari-sys/src/lib.rs
@@ -738,7 +738,11 @@ impl Context {
         debug!("Importing module {:?}", module);
 
         let module_name = CString::new(module.as_ref()).expect("Invalid string");
-        let _res = unsafe { yr_modules_load(module_name.as_ptr(), &mut **self.context) };
+        let res = unsafe { yr_modules_load(module_name.as_ptr(), &mut **self.context) };
+        debug!("Loaded module {:?} with result {}", module, res);
+        if res != ERROR_SUCCESS as i32 {
+            return Err(YariError::UnknownModule(module.to_string()));
+        }
 
         let new_module: *mut YR_OBJECT_STRUCTURE = unsafe {
             yr_hash_table_lookup(
@@ -748,9 +752,9 @@ impl Context {
             )
         }
         .cast();
-
+        debug!("Module {:?} has pointer {:?}", module, new_module);
         if new_module.is_null() {
-            return Err(YariError::SymbolNotFound(module.to_string()));
+            return Err(YariError::UnknownModule(module.to_string()));
         }
 
         self.modules.insert(module, new_module);
@@ -1336,6 +1340,7 @@ impl Context {
     }
 
     pub fn dump_module(&mut self, module: Module) -> Result<(), YariError> {
+        debug!("Dumping module {:?}", module);
         self.import_module(module)?;
         match self.modules.get(&module) {
             Some(module) => {

diff --git a/yari-sys/tests/common.rs b/yari-sys/tests/common.rs
@@ -4,10 +4,12 @@ use yari_sys::Module;
 use yari_sys::{Context, ContextBuilder};
 
 pub fn context() -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     ContextBuilder::default().build().unwrap()
 }
 
 pub fn context_with_cuckoo() -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     let test_root = Path::new(env!("CARGO_MANIFEST_DIR"));
     let cuckoo_report = test_root.join("tests/assets/cuckoo.json");
 
@@ -18,6 +20,7 @@ pub fn context_with_cuckoo() -> Context {
 }
 
 fn context_with_sample(path: &str, rule: Option<&str>) -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     let test_root = Path::new(env!("CARGO_MANIFEST_DIR"));
     let file = test_root.join(path);
 
@@ -29,14 +32,17 @@ fn context_with_sample(path: &str, rule: Option<&str>) -> Context {
 }
 
 pub fn context_with_elf_sample() -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     context_with_sample("tests/assets/elf_hello_world", None)
 }
 
 pub fn context_with_pe_signed_sample() -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     context_with_sample("tests/assets/pe_signed", None)
 }
 
 pub fn context_with_pe_sample_and_rule() -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     context_with_sample(
         "tests/assets/pe_hello_world",
         Some(
@@ -58,6 +64,7 @@ rule r {
 }
 
 pub fn context_with_elf_sample_and_rule() -> Context {
+    let _ = env_logger::builder().is_test(true).try_init();
     context_with_sample(
         "tests/assets/elf_hello_world",
         Some(

diff --git a/yari-sys/tests/tests_dump_module.rs b/yari-sys/tests/tests_dump_module.rs
@@ -4,6 +4,7 @@ mod common;
 
 #[test]
 fn test_function_dump_plain() {
+    println!("test_function_dump_plain");
     let mut context = common::context();
 
     for module in MODULES {
@@ -13,6 +14,7 @@ fn test_function_dump_plain() {
 
 #[test]
 fn test_function_dump_cuckoo() {
+    println!("test_function_dump_cuckoo");
     let mut context = common::context_with_cuckoo();
 
     for module in MODULES {
@@ -22,6 +24,7 @@ fn test_function_dump_cuckoo() {
 
 #[test]
 fn test_function_dump_pe() {
+    println!("test_function_dump_pe");
     let mut context = common::context_with_pe_sample_and_rule();
 
     for module in MODULES {
@@ -31,6 +34,7 @@ fn test_function_dump_pe() {
 
 #[test]
 fn test_function_dump_elf() {
+    println!("test_function_dump_elf");
     let mut context = common::context_with_elf_sample();
 
     for module in MODULES {