pq/libfreemkv
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix UDF file data reads: use partition_start not metadata_start

Browse Source 
File data extents are in the physical partition (partition_start + lba),
not the metadata partition. ICBs (directory/file entries) are in metadata,
but the data they point to is in physical. This one-line fix makes all
MPLS playlists readable — previously only 1/18 had correct MPLS magic.

Tested on Civil War UHD: 18 playlists, main movie at 00400.mpls (57min, 6 clips).
This commit is contained in:
MattJackson
2026-04-07 11:44:23 -07:00
parent cc58ab1754
commit 721308cb8e
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/udf.rs
+3 -1
View File
@@ -87,9 +87,11 @@ impl UdfFs {
let (data_lba, data_len) = self.read_icb_extent(session, entry.meta_lba)?; let (data_lba, data_len) = self.read_icb_extent(session, entry.meta_lba)?;
// Read file data sector by sector // Read file data sector by sector
// File DATA is in the physical partition (partition_start + lba),
// NOT the metadata partition. ICBs are in metadata, data is in physical.
let sector_count = ((data_len as u64 + 2047) / 2048) as u32; let sector_count = ((data_len as u64 + 2047) / 2048) as u32;
let mut data = vec![0u8; (sector_count as usize) * 2048]; let mut data = vec![0u8; (sector_count as usize) * 2048];
let abs_start = self.meta_to_abs(data_lba); let abs_start = self.partition_start + data_lba;
for i in 0..sector_count { for i in 0..sector_count {
let offset = (i as usize) * 2048; let offset = (i as usize) * 2048;