Skip to content

Commit 50f8a6b

Browse files
committed
store: Don't use CONSUME flag
libostree currently has a bug here with fsverity support; in the consume case (which right now is always zero-sized files because we don't hardlink them) it doesn't enable verity. This would be an easy bug to fix in libostree. But OTOH there's no real reason to set consume here either. The main idea of consume is when one is operating on potentially large files external to libostree, but most things here are hardlinked, and what isn't is just the zero sized files. Signed-off-by: Colin Walters <[email protected]>
1 parent 9d9a235 commit 50f8a6b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ostree-ext/src/container/store.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1058,7 +1058,7 @@ impl ImageImporter {
10581058
let root_dir = td.open_dir(rootpath)?;
10591059

10601060
let modifier =
1061-
ostree::RepoCommitModifier::new(ostree::RepoCommitModifierFlags::CONSUME, None);
1061+
ostree::RepoCommitModifier::new(ostree::RepoCommitModifierFlags::empty(), None);
10621062
modifier.set_devino_cache(&devino);
10631063
// If we have derived layers, then we need to handle the case where
10641064
// the derived layers include custom policy. Just relabel everything

0 commit comments

Comments
 (0)