From ba29b61f8cbc784fba511e84a60b1758f6d24f01 Mon Sep 17 00:00:00 2001 From: Brandon Thomas Date: Fri, 30 Oct 2026 15:54:46 -0400 Subject: [PATCH] Keep backslash zip entry names with forward slashes so later lookups split them Co-Authored-By: Claude Opus 5.5 --- crates/pptx/src/opc.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/crates/pptx/src/opc.rs b/crates/pptx/src/opc.rs index adbcf448..20a53bcb 100644 --- a/crates/pptx/src/opc.rs +++ b/crates/pptx/src/opc.rs @@ -93,7 +93,8 @@ impl Package { } } total = total.saturating_add(data.len() as u64); - parts.insert(norm(&name), (name, data)); + // A name kept with `\` would reach code that splits paths on `/`. + parts.insert(norm(&name), (name.replace('\\', "/"), data)); } Ok(Package { parts, ct_cache: std::sync::OnceLock::new() }) } @@ -504,6 +505,7 @@ mod tests { let z = zip_of(&[("PPT\\Slides\\Slide1.xml", 3)]); let p = Package::open(&z).unwrap(); assert!(p.get("ppt/slides/slide1.xml").is_some()); + assert_eq!(p.original_name("ppt/slides/slide1.xml"), Some("PPT/Slides/Slide1.xml")); } #[test]