1 use rustc_hir::def_id::LocalDefId;
2 use rustc_middle::mir::visit::{PlaceContext, Visitor};
3 use rustc_middle::mir::*;
4 use rustc_middle::ty::query::Providers;
5 use rustc_middle::ty::{self, TyCtxt};
6 use rustc_session::lint::builtin::UNALIGNED_REFERENCES;
11 pub(crate) fn provide(providers: &mut Providers) {
12 *providers = Providers { unsafe_derive_on_repr_packed, ..*providers };
15 pub struct CheckPackedRef;
17 impl<'tcx> MirLint<'tcx> for CheckPackedRef {
18 fn run_lint(&self, tcx: TyCtxt<'tcx>, body: &Body<'tcx>) {
19 let param_env = tcx.param_env(body.source.def_id());
20 let source_info = SourceInfo::outermost(body.span);
21 let mut checker = PackedRefChecker { body, tcx, param_env, source_info };
22 checker.visit_body(&body);
26 struct PackedRefChecker<'a, 'tcx> {
29 param_env: ty::ParamEnv<'tcx>,
30 source_info: SourceInfo,
33 fn unsafe_derive_on_repr_packed(tcx: TyCtxt<'_>, def_id: LocalDefId) {
34 let lint_hir_id = tcx.hir().local_def_id_to_hir_id(def_id);
36 tcx.struct_span_lint_hir(UNALIGNED_REFERENCES, lint_hir_id, tcx.def_span(def_id), |lint| {
37 // FIXME: when we make this a hard error, this should have its
39 let message = if tcx.generics_of(def_id).own_requires_monomorphization() {
40 "`#[derive]` can't be used on a `#[repr(packed)]` struct with \
41 type or const parameters (error E0133)"
43 "`#[derive]` can't be used on a `#[repr(packed)]` struct that \
44 does not derive Copy (error E0133)"
46 lint.build(message).emit();
50 impl<'tcx> Visitor<'tcx> for PackedRefChecker<'_, 'tcx> {
51 fn visit_terminator(&mut self, terminator: &Terminator<'tcx>, location: Location) {
52 // Make sure we know where in the MIR we are.
53 self.source_info = terminator.source_info;
54 self.super_terminator(terminator, location);
57 fn visit_statement(&mut self, statement: &Statement<'tcx>, location: Location) {
58 // Make sure we know where in the MIR we are.
59 self.source_info = statement.source_info;
60 self.super_statement(statement, location);
63 fn visit_place(&mut self, place: &Place<'tcx>, context: PlaceContext, _location: Location) {
64 if context.is_borrow() {
65 if util::is_disaligned(self.tcx, self.body, self.param_env, *place) {
66 let def_id = self.body.source.instance.def_id();
67 if let Some(impl_def_id) = self
69 .impl_of_method(def_id)
70 .filter(|&def_id| self.tcx.is_builtin_derive(def_id))
72 // If a method is defined in the local crate,
73 // the impl containing that method should also be.
74 self.tcx.ensure().unsafe_derive_on_repr_packed(impl_def_id.expect_local());
76 let source_info = self.source_info;
77 let lint_root = self.body.source_scopes[source_info.scope]
82 self.tcx.struct_span_lint_hir(
87 lint.build("reference to packed field is unaligned")
89 "fields of packed structs are not properly aligned, and creating \
90 a misaligned reference is undefined behavior (even if that \
91 reference is never dereferenced)",
94 "copy the field contents to a local variable, or replace the \
95 reference with a raw pointer and use `read_unaligned`/`write_unaligned` \
96 (loads and stores via `*p` must be properly aligned even when using raw pointers)"