X-Git-Url: https://git.jtryba.com/browse/friidump.git/blobdiff_plain/6e43cd3fd4f7324a177331a01d653b4c19c4bca7..refs/heads/test/xgd1-security-cache-probe-v1:/libfriidump/dumper.c?ds=sidebyside

diff --git a/libfriidump/dumper.c b/libfriidump/dumper.c
index 36ac682..5279535 100644
--- a/libfriidump/dumper.c
+++ b/libfriidump/dumper.c
@@ -321,6 +321,10 @@ static void xbox_write_redump_metadata (dumper *dmp, xbox_redump_metadata *meta)
 	fprintf (json, "    \"pregame_padding_sector_count\": %u,\n", XBOX_XGD1_GAME_OUTPUT_START_LBA - XBOX_XGD1_VIDEO_L0_SECTORS);
 	fprintf (json, "    \"game_output_start_lba\": %u,\n", XBOX_XGD1_GAME_OUTPUT_START_LBA);
 	fprintf (json, "    \"game_leadin_sector_count\": %u,\n", XBOX_XISO_LEADIN_SECTORS);
+	fprintf (json, "    \"game_leadin_unlocked_source_start_lba\": %u,\n",
+		(meta -> game_source_lba >= XBOX_XISO_LEADIN_SECTORS) ?
+		meta -> game_source_lba - XBOX_XISO_LEADIN_SECTORS : 0);
+	fprintf (json, "    \"game_leadin_source_policy\": \"drive_read10_with_per_sector_zero_fallback\",\n");
 	fprintf (json, "    \"game_unlocked_source_start_lba\": %u,\n", meta -> game_source_lba);
 	fprintf (json, "    \"game_unlocked_source_sector_count\": %u,\n", meta -> game_source_sectors);
 	fprintf (json, "    \"postgame_padding_start_lba\": %u,\n", XBOX_XGD1_GAME_OUTPUT_START_LBA + XBOX_XGD1_UNLOCKED_GAME_VIEW_SECTORS);
@@ -339,8 +343,13 @@ static void xbox_write_redump_metadata (dumper *dmp, xbox_redump_metadata *meta)
 	fprintf (json, "  },\n");
 	fprintf (json, "  \"lead_in_capture\": {\n");
 	fprintf (json, "    \"game_leadin_attempted\": true,\n");
+	fprintf (json, "    \"unlocked_source_start_lba\": %u,\n",
+		(meta -> game_source_lba >= XBOX_XISO_LEADIN_SECTORS) ?
+		meta -> game_source_lba - XBOX_XISO_LEADIN_SECTORS : 0);
+	fprintf (json, "    \"sector_count\": %u,\n", XBOX_XISO_LEADIN_SECTORS);
 	fprintf (json, "    \"readable_sectors\": %u,\n", meta -> game_leadin_read_sectors);
-	fprintf (json, "    \"zero_filled_sectors\": %u\n", meta -> game_leadin_zero_sectors);
+	fprintf (json, "    \"zero_filled_sectors\": %u,\n", meta -> game_leadin_zero_sectors);
+	fprintf (json, "    \"physical_geometry_evidence\": \"friidump_0.5.3.13_cache_aligned_raw_id_probe\"\n");
 	fprintf (json, "  },\n");
 	fprintf (json, "  \"dvd_structures\": {\n");
 	fprintf (json, "    \"pfi\": "); if (pfi_leaf) xbox_json_string (json, pfi_leaf); else fprintf (json, "null"); fprintf (json, ",\n");
@@ -348,7 +357,8 @@ static void xbox_write_redump_metadata (dumper *dmp, xbox_redump_metadata *meta)
 	fprintf (json, "  },\n");
 	fprintf (json, "  \"notes\": [\n");
 	fprintf (json, "    \"PFI and DMI are stored as separate binary READ DVD STRUCTURE captures because they are not READ(10) user-data sectors.\",\n");
-	fprintf (json, "    \"Pregame and postgame padding are zero-filled unless a future workflow proves readable source sectors for those ranges.\"\n");
+	fprintf (json, "    \"The 32-sector game lead-in is captured from unlocked source LBA 0..31; only unreadable lead-in sectors use zero-fill fallback.\",\n");
+	fprintf (json, "    \"Pregame and postgame padding retain zero-filled placeholder content; their physical locations are resolved but their inaccessible bytes are not.\"\n");
 	fprintf (json, "  ]\n");
 	fprintf (json, "}\n");
 	fclose (json);
@@ -1064,21 +1074,19 @@ static int dumper_dump_xbox_redump_iso (dumper *dmp, u_int32_t *current_sector)
 	if (game_source_lba != XBOX_XISO_STANDARD_GAME_LBA && disc_is_xbox_challenge_drive (dmp -> dsk))
 		warning ("Xbox XDVDFS was detected at LBA %u, not the expected GDR-8050L game-view LBA 32", game_source_lba);
 
-	if (disc_is_xbox_challenge_drive (dmp -> dsk)) {
-		/* Match the original GDR-8050L dumper's option-1 redump path: the
-		 * 32-sector XISO/game lead-in area is synthetic zero-fill, and the real
-		 * XDVDFS data begins at unlocked source LBA 32. */
-		xbox_ref_log_fprintf (stderr, "[XBOX] Writing 32-sector game lead-in as zero-fill for GDR-8050L redump flow.\n");
-		meta.game_leadin_zero_sectors += XBOX_XISO_LEADIN_SECTORS;
-		if (current_sector) *current_sector = XBOX_XGD1_GAME_OUTPUT_START_LBA;
-		if (!xbox_write_zero_iso_range (dmp, XBOX_XISO_LEADIN_SECTORS, XBOX_XGD1_GAME_OUTPUT_START_LBA, meta.output_sectors, "GAME-XISO-LEADIN"))
-			goto cleanup;
-	} else {
-		if (!xbox_write_leadin_iso_range (dmp,
-				(game_source_lba >= XBOX_XISO_LEADIN_SECTORS) ? game_source_lba - XBOX_XISO_LEADIN_SECTORS : 0,
-				XBOX_XGD1_GAME_OUTPUT_START_LBA, meta.output_sectors, &meta, "redump game", true, current_sector))
-			goto cleanup;
-	}
+	/* FriiDump 0.5.3.13 cache-aligned raw-ID validation proved that the
+	 * unlocked GDR-8050L view exposes source LBA 0..31 as the physical
+	 * game-region lead-in immediately preceding the XDVDFS header at LBA 32.
+	 * Read those sectors from the drive; retain the existing per-sector
+	 * zero-fill fallback only for genuinely unreadable sectors. */
+	xbox_ref_log_fprintf (stderr,
+		"[XBOX] Capturing 32-sector game lead-in from unlocked source LBA %u..%u.\n",
+		(game_source_lba >= XBOX_XISO_LEADIN_SECTORS) ? game_source_lba - XBOX_XISO_LEADIN_SECTORS : 0,
+		(game_source_lba >= XBOX_XISO_LEADIN_SECTORS) ? game_source_lba - 1 : XBOX_XISO_LEADIN_SECTORS - 1);
+	if (!xbox_write_leadin_iso_range (dmp,
+			(game_source_lba >= XBOX_XISO_LEADIN_SECTORS) ? game_source_lba - XBOX_XISO_LEADIN_SECTORS : 0,
+			XBOX_XGD1_GAME_OUTPUT_START_LBA, meta.output_sectors, &meta, "redump game", true, current_sector))
+		goto cleanup;
 
 	if (!xbox_dump_read10_iso_range (dmp, game_source_lba, XBOX_XGD1_GAME_SOURCE_SECTORS,
 			XBOX_XGD1_GAME_OUTPUT_START_LBA + XBOX_XISO_LEADIN_SECTORS, meta.output_sectors, "GAME-XDVDFS", true, current_sector))