quick: Fixing conversion warning found in clang FreeBSD

Fixing a conversion warning for the ZONE_NO_RESET option to ata security options that are passed in. Warning was only seen in FreeBSD 14's clang Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>
Seagate · Apr 29, 2024 · 065e448 · 065e448
1 parent 856f3ff
commit 065e448
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/subprojects/opensea-transport b/subprojects/opensea-transport
diff --git a/utils/C/openSeaChest/openSeaChest_Erase.c b/utils/C/openSeaChest/openSeaChest_Erase.c
@@ -2748,7 +2748,14 @@ int32_t main(int argc, char *argv[])
                 ataPassword.passwordType = ATA_SECURITY_USING_MASTER_PW;
                 memcpy(ataPassword.password, ATA_SECURITY_PASSWORD, ATA_SECURITY_PASSWORD_BYTE_COUNT);//ATA_SECURITY_PASSWORD_BYTE_COUNT shouldn't ever be > 32. Should be caught above.
                 ataPassword.passwordLength = ATA_SECURITY_PASSWORD_BYTE_COUNT;
-                ataPassword.zacSecurityOption = ZONE_NO_RESET;
+                if (ZONE_NO_RESET == goTrue)
+                {
+                    ataPassword.zacSecurityOption = ATA_ZAC_ERASE_FULL_ZONES;
+                }
+                else 
+                {
+                    ataPassword.zacSecurityOption = ATA_ZAC_ERASE_EMPTY_ZONES;
+                }
                 eATASecurityEraseType ataSecureEraseType = ATA_SECURITY_ERASE_STANDARD_ERASE;
                 if (ATA_SECURITY_ERASE_ENHANCED_FLAG)
                 {