Skip to content

Commit 9006deb

Browse files
committed
CFI: Fix drop and drop_in_place
Fix drop and drop_in_place by transforming self of drop and drop_in_place methods into Drop trait objects.
1 parent c98ea0d commit 9006deb

File tree

2 files changed

+58
-2
lines changed

2 files changed

+58
-2
lines changed

compiler/rustc_symbol_mangling/src/typeid/typeid_itanium_cxx_abi.rs

Lines changed: 38 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -745,6 +745,19 @@ fn encode_ty<'tcx>(
745745
typeid
746746
}
747747

748+
/// Creates a trait object.
749+
#[inline]
750+
fn new_dynamic_trait<'tcx>(
751+
tcx: TyCtxt<'tcx>,
752+
def_id: DefId,
753+
args: GenericArgsRef<'tcx>,
754+
) -> Ty<'tcx> {
755+
let predicate =
756+
ty::ExistentialPredicate::Trait(ty::ExistentialTraitRef { def_id: def_id, args: args });
757+
let predicates = tcx.mk_poly_existential_predicates(&[ty::Binder::dummy(predicate)]);
758+
Ty::new_dynamic(tcx, predicates, tcx.lifetimes.re_erased, ty::Dyn)
759+
}
760+
748761
/// Transforms predicates for being encoded and used in the substitution dictionary.
749762
fn transform_predicates<'tcx>(
750763
tcx: TyCtxt<'tcx>,
@@ -1112,8 +1125,31 @@ pub fn typeid_for_instance<'tcx>(
11121125
mut instance: Instance<'tcx>,
11131126
options: TypeIdOptions,
11141127
) -> String {
1115-
if matches!(instance.def, ty::InstanceDef::Virtual(..)) {
1116-
instance.args = strip_receiver_auto(tcx, instance.args)
1128+
if (matches!(instance.def, ty::InstanceDef::Virtual(..))
1129+
&& instance.def_id() == tcx.lang_items().drop_in_place_fn().unwrap())
1130+
|| matches!(instance.def, ty::InstanceDef::DropGlue(..))
1131+
{
1132+
// Adjust the type ids of DropGlues
1133+
//
1134+
// DropGlues may have indirect calls to one or more given types drop function. Rust allows
1135+
// for types to be erased to any trait object and retains the drop function for the original
1136+
// type, which means at the indirect call sites in DropGlues, when typeid_for_fnabi is
1137+
// called a second time, it only has information after type erasure and it could be a call
1138+
// on any arbitrary trait object. Normalize them to a synthesized Drop trait object, both on
1139+
// declaration/definition, and during code generation at call sites so they have the same
1140+
// type id and match.
1141+
//
1142+
// FIXME(rcvalle): This allows a drop call on any trait object to call the drop function of
1143+
// any other type.
1144+
//
1145+
let self_ty = Ty::new_mut_ref(
1146+
tcx,
1147+
tcx.lifetimes.re_erased,
1148+
new_dynamic_trait(tcx, tcx.lang_items().drop_trait().unwrap(), List::empty()),
1149+
);
1150+
instance.args = tcx.mk_args_trait(self_ty, List::empty());
1151+
} else if matches!(instance.def, ty::InstanceDef::Virtual(..)) {
1152+
instance.args = strip_receiver_auto(tcx, instance.args);
11171153
}
11181154

11191155
if let Some(impl_id) = tcx.impl_of_method(instance.def_id())
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
// Verifies that drops can be called on arbitrary trait objects.
2+
//
3+
// FIXME(#122848): Remove only-linux when fixed.
4+
//@ only-linux
5+
//@ needs-sanitizer-cfi
6+
//@ compile-flags: -Clto -Copt-level=0 -Cprefer-dynamic=off -Ctarget-feature=-crt-static -Zsanitizer=cfi
7+
//@ run-pass
8+
9+
struct EmptyDrop;
10+
11+
struct NonEmptyDrop;
12+
13+
impl Drop for NonEmptyDrop {
14+
fn drop(&mut self) {}
15+
}
16+
17+
fn main() {
18+
let _ = Box::new(EmptyDrop) as Box<dyn Send>;
19+
let _ = Box::new(NonEmptyDrop) as Box<dyn Send>;
20+
}

0 commit comments

Comments
 (0)