4 Commits

4 changed files with 18 additions and 12 deletions

View File

@@ -277,7 +277,7 @@ if(USE_SLOG)
target_compile_definitions(aaruformat PRIVATE ENABLE_TRACE ENABLE_FATAL USE_SLOG) target_compile_definitions(aaruformat PRIVATE ENABLE_TRACE ENABLE_FATAL USE_SLOG)
# Link slog # Link slog
target_link_libraries(aaruformat PRIVATE slog) target_link_libraries(aaruformat slog)
message(STATUS "slog logging enabled") message(STATUS "slog logging enabled")
else() else()

View File

@@ -2,7 +2,7 @@
<package xmlns="http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd"> <package xmlns="http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd">
<metadata> <metadata>
<id>libaaruformat</id> <id>libaaruformat</id>
<version>1.0.0-alpha.9</version> <version>1.0.0-alpha.10</version>
<description>Library for management of AaruFormat images.</description> <description>Library for management of AaruFormat images.</description>
<authors>claunia</authors> <authors>claunia</authors>
<projectUrl>https://github.com/aaru-dps/libaaruformat</projectUrl> <projectUrl>https://github.com/aaru-dps/libaaruformat</projectUrl>

View File

@@ -1062,10 +1062,12 @@ bool set_ddt_single_level_v2(aaruformat_context *ctx, uint64_t sector_address, c
TRACE("Exiting set_ddt_single_level_v2() = false"); TRACE("Exiting set_ddt_single_level_v2() = false");
return false; return false;
} }
*ddt_entry |= (uint64_t)sector_status << 60;
} }
// Sector status can be different from previous deduplicated sector
*ddt_entry &= 0x0FFFFFFFFFFFFFFF;
*ddt_entry |= (uint64_t)sector_status << 60;
TRACE("Setting big single-level DDT entry %d to %ull", sector_address, (uint64_t)*ddt_entry); TRACE("Setting big single-level DDT entry %d to %ull", sector_address, (uint64_t)*ddt_entry);
ctx->user_data_ddt2[sector_address] = *ddt_entry; ctx->user_data_ddt2[sector_address] = *ddt_entry;
@@ -1154,10 +1156,12 @@ bool set_ddt_multi_level_v2(aaruformat_context *ctx, uint64_t sector_address, bo
TRACE("Exiting set_ddt_multi_level_v2() = false"); TRACE("Exiting set_ddt_multi_level_v2() = false");
return false; return false;
} }
*ddt_entry |= (uint64_t)sector_status << 60;
} }
// Sector status can be different from previous deduplicated sector
*ddt_entry &= 0x0FFFFFFFFFFFFFFF;
*ddt_entry |= (uint64_t)sector_status << 60;
TRACE("Setting small secondary DDT entry %d to %ull", sector_address % items_per_ddt_entry, TRACE("Setting small secondary DDT entry %d to %ull", sector_address % items_per_ddt_entry,
(uint64_t)*ddt_entry); (uint64_t)*ddt_entry);
ctx->cached_secondary_ddt2[sector_address % items_per_ddt_entry] = *ddt_entry; ctx->cached_secondary_ddt2[sector_address % items_per_ddt_entry] = *ddt_entry;
@@ -1637,10 +1641,12 @@ bool set_ddt_multi_level_v2(aaruformat_context *ctx, uint64_t sector_address, bo
TRACE("Exiting set_ddt_multi_level_v2() = false"); TRACE("Exiting set_ddt_multi_level_v2() = false");
return false; return false;
} }
*ddt_entry |= (uint64_t)sector_status << 60;
} }
// Sector status can be different from previous deduplicated sector
*ddt_entry &= 0x0FFFFFFFFFFFFFFF;
*ddt_entry |= (uint64_t)sector_status << 60;
TRACE("Setting big secondary DDT entry %d to %ull", sector_address % items_per_ddt_entry, (uint64_t)*ddt_entry); TRACE("Setting big secondary DDT entry %d to %ull", sector_address % items_per_ddt_entry, (uint64_t)*ddt_entry);
ctx->cached_secondary_ddt2[sector_address % items_per_ddt_entry] = *ddt_entry; ctx->cached_secondary_ddt2[sector_address % items_per_ddt_entry] = *ddt_entry;

View File

@@ -704,7 +704,7 @@ AARU_EXPORT int32_t AARU_CALL aaruf_read_track_sector(void *context, uint8_t *da
if(ctx->image_info.MetadataMediaType != OpticalDisc) if(ctx->image_info.MetadataMediaType != OpticalDisc)
{ {
FATAL("Incorrect media type %d, expected OpticalDisc", ctx->imageInfo.XmlMediaType); FATAL("Incorrect media type %d, expected OpticalDisc", ctx->image_info.MetadataMediaType);
TRACE("Exiting aaruf_read_track_sector() = AARUF_ERROR_INCORRECT_MEDIA_TYPE"); TRACE("Exiting aaruf_read_track_sector() = AARUF_ERROR_INCORRECT_MEDIA_TYPE");
return AARUF_ERROR_INCORRECT_MEDIA_TYPE; return AARUF_ERROR_INCORRECT_MEDIA_TYPE;
@@ -1282,7 +1282,7 @@ AARU_EXPORT int32_t AARU_CALL aaruf_read_sector_long(void *context, const uint64
tag_length = 24; tag_length = 24;
break; break;
default: default:
FATAL("Unsupported media type %d", ctx->imageInfo.MediaType); FATAL("Unsupported media type %d", ctx->image_info.MediaType);
TRACE("Exiting aaruf_read_sector_long() = AARUF_ERROR_INCORRECT_MEDIA_TYPE"); TRACE("Exiting aaruf_read_sector_long() = AARUF_ERROR_INCORRECT_MEDIA_TYPE");
return AARUF_ERROR_INCORRECT_MEDIA_TYPE; return AARUF_ERROR_INCORRECT_MEDIA_TYPE;
@@ -1339,13 +1339,13 @@ AARU_EXPORT int32_t AARU_CALL aaruf_read_sector_long(void *context, const uint64
TRACE("Exiting aaruf_read_sector_long() = AARUF_STATUS_OK"); TRACE("Exiting aaruf_read_sector_long() = AARUF_STATUS_OK");
return AARUF_STATUS_OK; return AARUF_STATUS_OK;
default: default:
FATAL("Incorrect media type %d for long sector reading", ctx->imageInfo.MediaType); FATAL("Incorrect media type %d for long sector reading", ctx->image_info.MediaType);
TRACE("Exiting aaruf_read_sector_long() = AARUF_ERROR_INCORRECT_MEDIA_TYPE"); TRACE("Exiting aaruf_read_sector_long() = AARUF_ERROR_INCORRECT_MEDIA_TYPE");
return AARUF_ERROR_INCORRECT_MEDIA_TYPE; return AARUF_ERROR_INCORRECT_MEDIA_TYPE;
} }
default: default:
FATAL("Incorrect media type %d for long sector reading", ctx->imageInfo.MediaType); FATAL("Incorrect media type %d for long sector reading", ctx->image_info.MediaType);
TRACE("Exiting aaruf_read_sector_long() = AARUF_ERROR_INCORRECT_MEDIA_TYPE"); TRACE("Exiting aaruf_read_sector_long() = AARUF_ERROR_INCORRECT_MEDIA_TYPE");
return AARUF_ERROR_INCORRECT_MEDIA_TYPE; return AARUF_ERROR_INCORRECT_MEDIA_TYPE;