Skip to content

Conversation

@julek-wolfssl
Copy link
Member

  • Add filesystem access implementation in HAL (hal/filesystem.c)
  • Introduce new library_fs target and build rules for filesystem-backed partition management
  • Provide example configuration for library_fs
  • Add CLI application (hal/library_fs.c) to query and manage partitions
  • Update documentation with usage instructions and build steps for the new CLI tool
  • Update .gitignore

julek-wolfssl and others added 3 commits October 22, 2025 14:45
- Add filesystem access implementation in HAL (`hal/filesystem.c`)
- Introduce new library_fs target and build rules for filesystem-backed partition management
- Provide example configuration for library_fs
- Add CLI application (`hal/library_fs.c`) to query and manage partitions
- Update documentation with usage instructions and build steps for the new CLI tool
- Update .gitignore
@danielinux
Copy link
Member

danielinux commented Oct 23, 2025

There is a printf in image.c:1277 requiring a mock for wolfBoot_get_version() in unit-image.c unit test.

Consider this fix:
fix-unit-tests-pr615.patch

diff --git a/tools/unit-tests/unit-image.c b/tools/unit-tests/unit-image.c
index 81f5b2d0..69e3a0ef 100644
--- a/tools/unit-tests/unit-image.c
+++ b/tools/unit-tests/unit-image.c
@@ -76,6 +76,12 @@ static const unsigned char pubkey_digest[SHA256_DIGEST_SIZE] = {
 };
 
 
+uint32_t wolfBoot_get_blob_version(uint8_t *blob)
+{
+    return 1;
+}
+
+
 static unsigned char test_img_v200000000_signed_bin[] = {
       0x57, 0x4f, 0x4c, 0x46, 0x13, 0x00, 0x00, 0x00, 0x01, 0x00, 0x04, 0x00,
   0x00, 0xc2, 0xeb, 0x0b, 0xff, 0xff, 0xff, 0xff, 0x02, 0x00, 0x08, 0x00,

Copy link
Contributor

@dgarske dgarske left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this approach will work. I am having trouble getting this cross-compiled for the older Ubuntu on customer hardware, but will work though it.
Can you go ahead and add CI tests for this similar to your script? Thank you

@julek-wolfssl
Copy link
Member Author

@dgarske CI added.

@julek-wolfssl julek-wolfssl removed their assignment Oct 27, 2025
danielinux
danielinux previously approved these changes Oct 28, 2025
@dgarske dgarske self-requested a review October 28, 2025 15:59
@danielinux danielinux merged commit c6776bc into wolfSSL:master Oct 30, 2025
219 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants