-
Notifications
You must be signed in to change notification settings - Fork 197
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
WIP: Add support for wrapping binaries (rpm)
We need to be friendlier to people who are transitioning from "traditional" yum managed systems. This patchset starts to lay out the groundwork for supporting "intercepting" binaries that are in the tree. To start with for example, we wrap `/usr/bin/rpm` and cause it to drop privileges. This way it can't corrupt anything; we're not just relying on the read-only bind mount. For example nothing will accidentally get written to `/var/lib/rpm`. Now a tricky thing with this one is we *do* want it to write if we're in an unlocked state. There are tons of other examples of binaries we want to intercept, among them: - `grubby` -> `rpm-ostree kargs` - `dracut` -> `rpm-ostree initramfs` - `yum` -> well...we'll talk about that later
- Loading branch information
Showing
14 changed files
with
378 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,6 +4,7 @@ version = "0.1.0" | |
authors = ["Colin Walters <[email protected]>", "Jonathan Lebon <[email protected]>"] | ||
|
||
[dependencies] | ||
nix = "0.13.0" | ||
failure = "0.1.3" | ||
serde = "1.0.78" | ||
serde_derive = "1.0.78" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,93 @@ | ||
/* | ||
* Copyright (C) 2019 Red Hat, Inc. | ||
* | ||
* This program is free software; you can redistribute it and/or modify | ||
* it under the terms of the GNU General Public License as published by | ||
* the Free Software Foundation; either version 2 of the License, or | ||
* (at your option) any later version. | ||
* | ||
* This program is distributed in the hope that it will be useful, | ||
* but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
* GNU General Public License for more details. | ||
* | ||
* You should have received a copy of the GNU General Public License | ||
* along with this program; if not, write to the Free Software | ||
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA | ||
*/ | ||
|
||
use std::io::prelude::*; | ||
use std::{io, path}; | ||
use failure::Fallible; | ||
|
||
use rayon::prelude::*; | ||
use openat_utils::OpenatDirExt; | ||
mod cliutil; | ||
mod clirpm; | ||
use self::clirpm::cli_rpm; | ||
|
||
pub(crate) static WRAP_DESTDIR : &str = "usr/libexec/rpm-ostree/wrapped"; | ||
|
||
static WRAPPED_BINARIES : &[&str] = &["usr/bin/rpm"]; | ||
|
||
fn cliwrap_main(args: &Vec<String>) -> Fallible<()> { | ||
// We'll panic here if the vector is empty, but that is intentional; | ||
// the outer code should always pass us at least one arg. | ||
let name = args[0].as_str(); | ||
let args : Vec<&str> = args.iter().map(|v| v.as_str()).collect(); | ||
|
||
// If we're not booted into ostree, just run the child directly. | ||
if !cliutil::is_ostree_booted() { | ||
cliutil::exec_real_wrapped(name, &args) | ||
} else { | ||
match name { | ||
"rpm" => cli_rpm(&args), | ||
_ => bail!("Unknown wrapped binary: {}", name), | ||
} | ||
} | ||
} | ||
|
||
// Move the real binaries to a subdir, and replace them with | ||
// a shell script that calls our wrapping code. | ||
fn write_wrappers(rootfs_dfd: &openat::Dir) -> Fallible<()> { | ||
rootfs_dfd.create_dir(WRAP_DESTDIR, 0o755)?; | ||
WRAPPED_BINARIES.par_iter().try_for_each(|&bin| { | ||
let binpath = path::Path::new(bin); | ||
|
||
if !rootfs_dfd.exists(binpath)? { | ||
return Ok(()); | ||
} | ||
|
||
let name = binpath.file_name().unwrap().to_str().unwrap(); | ||
let destpath = format!("{}/{}", WRAP_DESTDIR, name); | ||
rootfs_dfd.local_rename(bin, destpath.as_str())?; | ||
|
||
let f = rootfs_dfd.write_file(binpath, 0o755)?; | ||
let mut f = io::BufWriter::new(f); | ||
f.write(b"#!/bin/sh\nexec /usr/bin/rpm-ostree cliwrap $0\n")?; | ||
f.flush()?; | ||
Ok(()) | ||
}) | ||
} | ||
|
||
mod ffi { | ||
use super::*; | ||
use ffiutil::*; | ||
use glib; | ||
use libc; | ||
use failure::ResultExt; | ||
|
||
#[no_mangle] | ||
pub extern "C" fn ror_cliwrap_write_wrappers(rootfs_dfd: libc::c_int, gerror: *mut *mut glib_sys::GError) -> libc::c_int { | ||
let rootfs_dfd = ffi_view_openat_dir(rootfs_dfd); | ||
int_glib_error(write_wrappers(&rootfs_dfd).with_context(|e| format!("During cli wrapper replacement: {}", e)), gerror) | ||
} | ||
|
||
#[no_mangle] | ||
pub extern "C" fn ror_cliwrap_entrypoint(argv: *mut *mut libc::c_char, | ||
gerror: *mut *mut glib_sys::GError) -> libc::c_int { | ||
let v: Vec<String> = unsafe { glib::translate::FromGlibPtrContainer::from_glib_none(argv) }; | ||
int_glib_error(cliwrap_main(&v), gerror) | ||
} | ||
} | ||
pub use self::ffi::*; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,61 @@ | ||
use clap::{App, Arg}; | ||
use failure::Fallible; | ||
|
||
use crate::cliwrap::cliutil; | ||
|
||
fn new_rpm_app<'r>() -> App<'r, 'static> { | ||
let name = "cli-ostree-wrapper-rpm"; | ||
App::new(name) | ||
.bin_name(name) | ||
.version("0.1") | ||
.about("Wrapper for rpm") | ||
.arg(Arg::with_name("all").short("a")) | ||
.arg(Arg::with_name("file").short("f")) | ||
.arg(Arg::with_name("package").short("p")) | ||
.arg(Arg::with_name("query").short("q")) | ||
.arg(Arg::with_name("verify").short("V")) | ||
.arg(Arg::with_name("version")) | ||
} | ||
|
||
pub fn cli_rpm(argv: &[&str]) -> Fallible<()> { | ||
if cliutil::is_unlocked()? { | ||
// For now if we're unlocked, just directly exec rpm. In the future we | ||
// may choose to take over installing a package live. | ||
cliutil::exec_real_wrapped("rpm", argv) | ||
} else { | ||
let mut app = new_rpm_app(); | ||
let mut with_warning = true; | ||
if let Ok(matches) = app.get_matches_from_safe_borrow(argv) { | ||
// Implement custom option handling here | ||
if matches.is_present("verify") { | ||
println!("rpm --verify is not necessary for ostree-based systems. | ||
All binaries in /usr are underneath a read-only bind mount. | ||
If you wish to verify integrity, use `ostree fsck`."); | ||
return Ok(()) | ||
} | ||
with_warning = false; | ||
} | ||
cliutil::run_unprivileged(with_warning, "rpm", argv) | ||
} | ||
} | ||
|
||
#[cfg(test)] | ||
mod tests { | ||
use super::*; | ||
#[test] | ||
fn test_qa() { | ||
let app = new_rpm_app(); | ||
let argv = vec!["rpm", "-qa"]; | ||
let matches = app.get_matches_from_safe(&argv); | ||
assert!(matches.is_ok()); | ||
} | ||
|
||
#[test] | ||
fn test_unknown() { | ||
let app = new_rpm_app(); | ||
let argv = vec!["rpm", "--not-a-valid-arg"]; | ||
let matches = app.get_matches_from_safe(&argv); | ||
assert!(matches.is_err()); | ||
} | ||
|
||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,104 @@ | ||
use failure::Fallible; | ||
use libc; | ||
use std::{thread, time, path}; | ||
use std::ffi::CString; | ||
use nix::sys::statvfs; | ||
use nix::unistd; | ||
|
||
use cliwrap; | ||
|
||
// Returns true if the current process is booted via ostree. | ||
pub fn is_ostree_booted() -> bool { | ||
path::Path::new("/run/ostree-booted").exists() | ||
} | ||
|
||
// Returns true if the current process is running as root. | ||
pub fn is_unlocked() -> Fallible<bool> { | ||
Ok(!statvfs::statvfs("/usr")?.flags().contains(statvfs::FsFlags::ST_RDONLY)) | ||
} | ||
|
||
// Returns true if the current process is running as root. | ||
pub fn am_privileged() -> bool { | ||
unsafe { libc::getuid() == 0 } | ||
} | ||
|
||
// Wrapper for execv which accepts strings | ||
fn execvp_strs(argv0: &str, argv: &[&str]) -> Fallible<()> { | ||
let argv0 = CString::new(argv0).unwrap(); | ||
let argv : Vec<CString> = argv.iter().map(|&v| CString::new(v).unwrap()).collect(); | ||
unistd::execvp(&argv0, &argv)?; | ||
Ok(()) | ||
} | ||
|
||
// Return the absolute path to the underlying wrapped binary | ||
fn get_real_bin(bin_name: &str) -> String { | ||
format!("/{}/{}", cliwrap::WRAP_DESTDIR, bin_name) | ||
} | ||
|
||
// Wrapper for execv which accepts strings | ||
pub fn exec_real_wrapped<T: AsRef<str> + std::fmt::Display>(bin_name: T, argv: &[T]) -> Fallible<()> { | ||
let bin_name = bin_name.as_ref(); | ||
let real_bin = get_real_bin(bin_name); | ||
let argv : Vec<&str> = std::iter::once(bin_name).chain(argv.iter().map(|v| v.as_ref())).collect(); | ||
execvp_strs(real_bin.as_str(), &argv) | ||
} | ||
|
||
/// Run a subprocess synchronously as user `bin` (dropping all capabilities). | ||
pub fn run_unprivileged<T: AsRef<str>>( | ||
with_warning: bool, | ||
target_bin: &str, | ||
argv: &[T], | ||
) -> Fallible<()> { | ||
// `setpriv` is in util-linux; we could do this internally, but this is easier. | ||
let setpriv_argv = &[ | ||
"setpriv", | ||
"--no-new-privs", | ||
"--reuid=bin", | ||
"--regid=bin", | ||
"--init-groups", | ||
"--bounding-set", | ||
"-all", | ||
"--", | ||
]; | ||
|
||
let argv: Vec<&str> = argv.into_iter().map(AsRef::as_ref).collect(); | ||
let drop_privileges = am_privileged (); | ||
let app_name = "rpm-ostree"; | ||
if with_warning { | ||
let delay_s = 5; | ||
eprintln!( | ||
"{name}: NOTE: This system is ostree based.", | ||
name = app_name | ||
); | ||
if drop_privileges { | ||
eprintln!(r#"{name}: Dropping privileges as `{bin}` was executed with not "known safe" arguments."#, | ||
name=app_name, bin = target_bin); | ||
} else { | ||
eprintln!( | ||
r#"{name}: Wrapped binary "{bin}" was executed with not "known safe" arguments."#, | ||
name = app_name, | ||
bin = target_bin | ||
); | ||
} | ||
eprintln!( | ||
r##"{name}: You may invoke the real `{bin}` binary in `/{wrap_destdir}/{bin}`. | ||
{name}: Continuing execution in {delay} seconds. | ||
"##, | ||
name = app_name, | ||
wrap_destdir = cliwrap::WRAP_DESTDIR, | ||
bin = target_bin, | ||
delay = delay_s, | ||
); | ||
thread::sleep(time::Duration::from_secs(delay_s)); | ||
} | ||
|
||
if drop_privileges { | ||
let real_bin = get_real_bin(target_bin); | ||
let real_argv : Vec<&str> = setpriv_argv.iter().map(|&v| v) | ||
.chain(std::iter::once(real_bin.as_str())) | ||
.chain(argv).collect(); | ||
execvp_strs("setpriv", &real_argv) | ||
} else { | ||
exec_real_wrapped(target_bin, &argv) | ||
} | ||
} |
Oops, something went wrong.