From ceabd82ac8e796249feacf39c836034ed5e11c6d Mon Sep 17 00:00:00 2001 From: Amlal El Mahrouss Date: Sat, 29 Nov 2025 15:01:34 -0500 Subject: chore: source code review and fixes. Signed-off-by: Amlal El Mahrouss --- tools/chk.hefs.cc | 4 ++-- tools/mkfs.hefs.cc | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'tools') diff --git a/tools/chk.hefs.cc b/tools/chk.hefs.cc index 41c04b0f..a520d231 100644 --- a/tools/chk.hefs.cc +++ b/tools/chk.hefs.cc @@ -57,8 +57,8 @@ int main(int argc, char** argv) { output_device.read(reinterpret_cast(&boot_node), sizeof(boot_node)); - if (strncmp(boot_node.magic, kOpenHeFSMagic, kOpenHeFSMagicLen) != 0 || boot_node.sectorCount < 1 || - boot_node.sectorSize < kMkFsSectorSz) { + if (strncmp(boot_node.magic, kOpenHeFSMagic, kOpenHeFSMagicLen) != 0 || + boot_node.sectorCount < 1 || boot_node.sectorSize < kMkFsSectorSz) { mkfs::console_out() << "hefs: error: Device is not an OpenHeFS disk: " << opt_disk << "\n"; return EXIT_FAILURE; } diff --git a/tools/mkfs.hefs.cc b/tools/mkfs.hefs.cc index 507d9bab..6cf71bd1 100644 --- a/tools/mkfs.hefs.cc +++ b/tools/mkfs.hefs.cc @@ -153,8 +153,8 @@ int main(int argc, char** argv) { "BootNode::magic too small to hold kOpenHeFSMagicLen"); std::memset(boot_node.magic, 0, sizeof(boot_node.magic)); - size_t magic_copy = - (sizeof(boot_node.magic) < kOpenHeFSMagicLen - 1) ? sizeof(boot_node.magic) : (kOpenHeFSMagicLen - 1); + size_t magic_copy = (sizeof(boot_node.magic) < kOpenHeFSMagicLen - 1) ? sizeof(boot_node.magic) + : (kOpenHeFSMagicLen - 1); std::memcpy(boot_node.magic, kOpenHeFSMagic, magic_copy); boot_node.magic[magic_copy] = 0; -- cgit v1.2.3