Skip to content

Commit

Permalink
CFI: Encode Virtual calls as calls through the defining trait
Browse files Browse the repository at this point in the history
For example, if `trait Foo: Bar`, and we try to call a method from `Bar`
on `dyn Foo`, encode the callsite as passing a `dyn Bar`, not a `dyn
Foo`.
  • Loading branch information
maurer committed Mar 26, 2024
1 parent 536606b commit eec30e9
Show file tree
Hide file tree
Showing 2 changed files with 85 additions and 10 deletions.
27 changes: 17 additions & 10 deletions compiler/rustc_symbol_mangling/src/typeid/typeid_itanium_cxx_abi.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1112,8 +1112,10 @@ pub fn typeid_for_instance<'tcx>(
mut instance: Instance<'tcx>,
options: TypeIdOptions,
) -> String {
if matches!(instance.def, ty::InstanceDef::Virtual(..)) {
instance.args = strip_receiver_auto(tcx, instance.args)
if let ty::InstanceDef::Virtual(def_id, _) = instance.def {
let upcast_ty = upcast(tcx, def_id, instance.args);
let stripped_ty = strip_receiver_auto(tcx, upcast_ty);
instance.args = tcx.mk_args_trait(stripped_ty, instance.args.into_iter().skip(1));
}

if let Some(impl_id) = tcx.impl_of_method(instance.def_id())
Expand Down Expand Up @@ -1159,15 +1161,11 @@ pub fn typeid_for_instance<'tcx>(
typeid_for_fnabi(tcx, fn_abi, options)
}

fn strip_receiver_auto<'tcx>(
tcx: TyCtxt<'tcx>,
args: ty::GenericArgsRef<'tcx>,
) -> ty::GenericArgsRef<'tcx> {
let ty = args.type_at(0);
fn strip_receiver_auto<'tcx>(tcx: TyCtxt<'tcx>, ty: Ty<'tcx>) -> Ty<'tcx> {
let ty::Dynamic(preds, lifetime, kind) = ty.kind() else {
bug!("Tried to strip auto traits from non-dynamic type {ty}");
};
let new_rcvr = if preds.principal().is_some() {
if preds.principal().is_some() {
let filtered_preds =
tcx.mk_poly_existential_predicates_from_iter(preds.into_iter().filter(|pred| {
!matches!(pred.skip_binder(), ty::ExistentialPredicate::AutoTrait(..))
Expand All @@ -1178,8 +1176,7 @@ fn strip_receiver_auto<'tcx>(
// about it. This technically discards the knowledge that it was a type that was made
// into a trait object at some point, but that's not a lot.
tcx.types.unit
};
tcx.mk_args_trait(new_rcvr, args.into_iter().skip(1))
}
}

fn trait_object_ty<'tcx>(tcx: TyCtxt<'tcx>, poly_trait_ref: ty::PolyTraitRef<'tcx>) -> Ty<'tcx> {
Expand Down Expand Up @@ -1214,3 +1211,13 @@ fn trait_object_ty<'tcx>(tcx: TyCtxt<'tcx>, poly_trait_ref: ty::PolyTraitRef<'tc
);
Ty::new_dynamic(tcx, preds, tcx.lifetimes.re_erased, ty::Dyn)
}

fn upcast<'tcx>(tcx: TyCtxt<'tcx>, def_id: DefId, args: GenericArgsRef<'tcx>) -> Ty<'tcx> {
let self_ty = args.type_at(0);
let Some(trait_id) = tcx.trait_of_item(def_id) else {
// If it's a virtual call to `drop_in_place`, it won't be on a trait.
return self_ty;
};
let trait_ref = ty::TraitRef::from_method(tcx, trait_id, args);
trait_object_ty(tcx, ty::Binder::dummy(trait_ref))
}
68 changes: 68 additions & 0 deletions tests/ui/sanitizer/cfi-supertraits.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
#![feature(trait_upcasting)]
// Check that super-traits are callable.

//@ needs-sanitizer-cfi
// FIXME(#122848) Remove only-linux once OSX CFI binaries work
//@ only-linux
//@ compile-flags: --crate-type=bin -Cprefer-dynamic=off -Clto -Zsanitizer=cfi
//@ compile-flags: -C target-feature=-crt-static -C codegen-units=1 -C opt-level=0
//@ run-pass

trait Parent1 {
type P1;
fn p1(&self) -> Self::P1;
}

trait Parent2 {
type P2;
fn p2(&self) -> Self::P2;
}

trait Child : Parent1 + Parent2 {
type C;
fn c(&self) -> Self::C;
}

struct Foo;

impl Parent1 for Foo {
type P1 = u16;
fn p1(&self) -> Self::P1 {
println!("p1");
1
}
}

impl Parent2 for Foo {
type P2 = u32;
fn p2(&self) -> Self::P2 {
println!("p2");
2
}
}

impl Child for Foo {
type C = u8;
fn c(&self) -> Self::C {
println!("c");
0
}
}

fn main() {
// Child can access its own methods and super methods.
let x = &Foo as &dyn Child<C=u8,P1=u16,P2=u32>;
x.c();
x.p1();
x.p2();
// Parents can be created and access their methods.
let y = &Foo as &dyn Parent1<P1=u16>;
y.p1();
let z = &Foo as &dyn Parent2<P2=u32>;
z.p2();
// Trait upcasting works
let x1 = x as &dyn Parent1<P1=u16>;
x1.p1();
let x2 = x as &dyn Parent2<P2=u32>;
x2.p2();
}

0 comments on commit eec30e9

Please sign in to comment.