Merge pull request #2782 from cgwalters/testinst-updates

This commit is contained in:
Joseph Marrero Corchado 2022-11-22 14:22:48 -05:00 committed by GitHub
commit 52b12e71ee
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 8 additions and 9 deletions

View File

@ -11,16 +11,16 @@ name = "ostree-test"
path = "src/insttestmain.rs" path = "src/insttestmain.rs"
[dependencies] [dependencies]
cap-std-ext = "0.25" cap-std-ext = "1.0"
clap = "2.32.0" clap = "2.32.0"
structopt = "0.3" structopt = "0.3"
serde = "1.0.111" serde = "1.0.111"
serde_derive = "1.0.111" serde_derive = "1.0.111"
serde_json = "1.0" serde_json = "1.0"
sh-inline = "0.2.0" sh-inline = "0.4.0"
anyhow = "1.0" anyhow = "1.0"
tempfile = "3.1.0" tempfile = "3.1.0"
ostree-ext = { version = "0.7.0" } ostree-ext = { version = "0.9" }
libtest-mimic = "0.5.0" libtest-mimic = "0.5.0"
twoway = "0.2.1" twoway = "0.2.1"
hyper = { version = "0.14", features = ["runtime", "http1", "http2", "tcp", "server"] } hyper = { version = "0.14", features = ["runtime", "http1", "http2", "tcp", "server"] }
@ -29,12 +29,12 @@ futures = "0.3.4"
http = "0.2.0" http = "0.2.0"
tokio = { version = "1.4.0", features = ["full"] } tokio = { version = "1.4.0", features = ["full"] }
futures-util = "0.3.1" futures-util = "0.3.1"
libc = "0.2.100"
base64 = "0.12.0" base64 = "0.12.0"
procspawn = "0.8" procspawn = "0.8"
rand = "0.8" rand = "0.8"
strum = "0.18.0" strum = "0.18.0"
strum_macros = "0.18.0" strum_macros = "0.18.0"
nix = "0.23.0"
# See discussion in https://github.com/coreos/rpm-ostree/pull/2569#issuecomment-780569188 # See discussion in https://github.com/coreos/rpm-ostree/pull/2569#issuecomment-780569188
rpmostree-client = { git = "https://github.com/coreos/rpm-ostree", tag = "v2021.3" } rpmostree-client = { git = "https://github.com/coreos/rpm-ostree", tag = "v2021.3" }

View File

@ -52,6 +52,6 @@ pub(crate) fn itest_tmpfiles() -> Result<()> {
return Ok(()); return Ok(());
} }
let metadata = Path::new("/run/ostree").metadata()?; let metadata = Path::new("/run/ostree").metadata()?;
assert_eq!(metadata.permissions().mode() & !nix::libc::S_IFMT, 0o755); assert_eq!(metadata.permissions().mode() & !libc::S_IFMT, 0o755);
Ok(()) Ok(())
} }

View File

@ -75,7 +75,6 @@ pub(crate) fn mutate_one_executable_to(
/// Find ELF files in the srcdir, write new copies to dest (only percentage) /// Find ELF files in the srcdir, write new copies to dest (only percentage)
pub(crate) fn mutate_executables_to(src: &Dir, dest: &Dir, percentage: u32) -> Result<u32> { pub(crate) fn mutate_executables_to(src: &Dir, dest: &Dir, percentage: u32) -> Result<u32> {
use nix::sys::stat::Mode as NixMode;
assert!(percentage > 0 && percentage <= 100); assert!(percentage > 0 && percentage <= 100);
let mut mutated = 0; let mut mutated = 0;
for entry in src.entries()? { for entry in src.entries()? {
@ -84,13 +83,13 @@ pub(crate) fn mutate_executables_to(src: &Dir, dest: &Dir, percentage: u32) -> R
continue; continue;
} }
let meta = entry.metadata()?; let meta = entry.metadata()?;
let mode = NixMode::from_bits_truncate(meta.mode()); let mode = meta.mode();
// Must be executable // Must be executable
if !mode.intersects(NixMode::S_IXUSR | NixMode::S_IXGRP | NixMode::S_IXOTH) { if mode & (libc::S_IXUSR | libc::S_IXGRP | libc::S_IXOTH) == 0 {
continue; continue;
} }
// Not suid // Not suid
if mode.intersects(NixMode::S_ISUID | NixMode::S_ISGID) { if mode & (libc::S_ISUID | libc::S_ISGID) == 0 {
continue; continue;
} }
// Greater than 1k in size // Greater than 1k in size