]> git.lizzy.rs Git - rust.git/blob - src/librustc_privacy/lib.rs
Rollup merge of #57357 - frewsxcv:frewsxcv-partial-eq, r=QuietMisdreavus
[rust.git] / src / librustc_privacy / lib.rs
1 #![doc(html_logo_url = "https://www.rust-lang.org/logos/rust-logo-128x128-blk-v2.png",
2        html_favicon_url = "https://doc.rust-lang.org/favicon.ico",
3        html_root_url = "https://doc.rust-lang.org/nightly/")]
4
5 #![feature(nll)]
6 #![feature(rustc_diagnostic_macros)]
7
8 #![recursion_limit="256"]
9
10 #[macro_use] extern crate rustc;
11 #[macro_use] extern crate syntax;
12 extern crate rustc_typeck;
13 extern crate syntax_pos;
14 extern crate rustc_data_structures;
15
16 use rustc::hir::{self, Node, PatKind};
17 use rustc::hir::def::Def;
18 use rustc::hir::def_id::{CRATE_DEF_INDEX, LOCAL_CRATE, CrateNum, DefId};
19 use rustc::hir::intravisit::{self, Visitor, NestedVisitorMap};
20 use rustc::hir::itemlikevisit::DeepVisitor;
21 use rustc::lint;
22 use rustc::middle::privacy::{AccessLevel, AccessLevels};
23 use rustc::ty::{self, TyCtxt, Ty, TraitRef, TypeFoldable, GenericParamDefKind};
24 use rustc::ty::fold::TypeVisitor;
25 use rustc::ty::query::Providers;
26 use rustc::ty::subst::Substs;
27 use rustc::util::nodemap::NodeSet;
28 use rustc_data_structures::fx::FxHashSet;
29 use rustc_data_structures::sync::Lrc;
30 use syntax::ast::{self, CRATE_NODE_ID, Ident};
31 use syntax::attr;
32 use syntax::symbol::keywords;
33 use syntax_pos::Span;
34
35 use std::{cmp, fmt, mem};
36 use std::marker::PhantomData;
37
38 mod diagnostics;
39
40 ////////////////////////////////////////////////////////////////////////////////
41 /// Generic infrastructure used to implement specific visitors below.
42 ////////////////////////////////////////////////////////////////////////////////
43
44 /// Implemented to visit all `DefId`s in a type.
45 /// Visiting `DefId`s is useful because visibilities and reachabilities are attached to them.
46 /// The idea is to visit "all components of a type", as documented in
47 /// https://github.com/rust-lang/rfcs/blob/master/text/2145-type-privacy.md#how-to-determine-visibility-of-a-type
48 /// Default type visitor (`TypeVisitor`) does most of the job, but it has some shortcomings.
49 /// First, it doesn't have overridable `fn visit_trait_ref`, so we have to catch trait def-ids
50 /// manually. Second, it doesn't visit some type components like signatures of fn types, or traits
51 /// in `impl Trait`, see individual comments in `DefIdVisitorSkeleton::visit_ty`.
52 trait DefIdVisitor<'a, 'tcx: 'a> {
53     fn tcx(&self) -> TyCtxt<'a, 'tcx, 'tcx>;
54     fn shallow(&self) -> bool { false }
55     fn skip_assoc_tys(&self) -> bool { false }
56     fn visit_def_id(&mut self, def_id: DefId, kind: &str, descr: &dyn fmt::Display) -> bool;
57
58     /// Not overridden, but used to actually visit types and traits.
59     fn skeleton(&mut self) -> DefIdVisitorSkeleton<'_, 'a, 'tcx, Self> {
60         DefIdVisitorSkeleton {
61             def_id_visitor: self,
62             visited_opaque_tys: Default::default(),
63             dummy: Default::default(),
64         }
65     }
66     fn visit(&mut self, ty_fragment: impl TypeFoldable<'tcx>) -> bool {
67         ty_fragment.visit_with(&mut self.skeleton())
68     }
69     fn visit_trait(&mut self, trait_ref: TraitRef<'tcx>) -> bool {
70         self.skeleton().visit_trait(trait_ref)
71     }
72     fn visit_predicates(&mut self, predicates: Lrc<ty::GenericPredicates<'tcx>>) -> bool {
73         self.skeleton().visit_predicates(predicates)
74     }
75 }
76
77 struct DefIdVisitorSkeleton<'v, 'a, 'tcx, V>
78     where V: DefIdVisitor<'a, 'tcx> + ?Sized
79 {
80     def_id_visitor: &'v mut V,
81     visited_opaque_tys: FxHashSet<DefId>,
82     dummy: PhantomData<TyCtxt<'a, 'tcx, 'tcx>>,
83 }
84
85 impl<'a, 'tcx, V> DefIdVisitorSkeleton<'_, 'a, 'tcx, V>
86     where V: DefIdVisitor<'a, 'tcx> + ?Sized
87 {
88     fn visit_trait(&mut self, trait_ref: TraitRef<'tcx>) -> bool {
89         let TraitRef { def_id, substs } = trait_ref;
90         self.def_id_visitor.visit_def_id(def_id, "trait", &trait_ref) ||
91         (!self.def_id_visitor.shallow() && substs.visit_with(self))
92     }
93
94     fn visit_predicates(&mut self, predicates: Lrc<ty::GenericPredicates<'tcx>>) -> bool {
95         let ty::GenericPredicates { parent: _, predicates } = &*predicates;
96         for (predicate, _span) in predicates {
97             match predicate {
98                 ty::Predicate::Trait(poly_predicate) => {
99                     let ty::TraitPredicate { trait_ref } = *poly_predicate.skip_binder();
100                     if self.visit_trait(trait_ref) {
101                         return true;
102                     }
103                 }
104                 ty::Predicate::Projection(poly_predicate) => {
105                     let ty::ProjectionPredicate { projection_ty, ty } =
106                         *poly_predicate.skip_binder();
107                     if ty.visit_with(self) {
108                         return true;
109                     }
110                     if self.visit_trait(projection_ty.trait_ref(self.def_id_visitor.tcx())) {
111                         return true;
112                     }
113                 }
114                 ty::Predicate::TypeOutlives(poly_predicate) => {
115                     let ty::OutlivesPredicate(ty, _region) = *poly_predicate.skip_binder();
116                     if ty.visit_with(self) {
117                         return true;
118                     }
119                 }
120                 ty::Predicate::RegionOutlives(..) => {},
121                 _ => bug!("unexpected predicate: {:?}", predicate),
122             }
123         }
124         false
125     }
126 }
127
128 impl<'a, 'tcx, V> TypeVisitor<'tcx> for DefIdVisitorSkeleton<'_, 'a, 'tcx, V>
129     where V: DefIdVisitor<'a, 'tcx> + ?Sized
130 {
131     fn visit_ty(&mut self, ty: Ty<'tcx>) -> bool {
132         let tcx = self.def_id_visitor.tcx();
133         // Substs are not visited here because they are visited below in `super_visit_with`.
134         match ty.sty {
135             ty::Adt(&ty::AdtDef { did: def_id, .. }, ..) |
136             ty::Foreign(def_id) |
137             ty::FnDef(def_id, ..) |
138             ty::Closure(def_id, ..) |
139             ty::Generator(def_id, ..) => {
140                 if self.def_id_visitor.visit_def_id(def_id, "type", ty) {
141                     return true;
142                 }
143                 if self.def_id_visitor.shallow() {
144                     return false;
145                 }
146                 // Default type visitor doesn't visit signatures of fn types.
147                 // Something like `fn() -> Priv {my_func}` is considered a private type even if
148                 // `my_func` is public, so we need to visit signatures.
149                 if let ty::FnDef(..) = ty.sty {
150                     if tcx.fn_sig(def_id).visit_with(self) {
151                         return true;
152                     }
153                 }
154                 // Inherent static methods don't have self type in substs.
155                 // Something like `fn() {my_method}` type of the method
156                 // `impl Pub<Priv> { pub fn my_method() {} }` is considered a private type,
157                 // so we need to visit the self type additionally.
158                 if let Some(assoc_item) = tcx.opt_associated_item(def_id) {
159                     if let ty::ImplContainer(impl_def_id) = assoc_item.container {
160                         if tcx.type_of(impl_def_id).visit_with(self) {
161                             return true;
162                         }
163                     }
164                 }
165             }
166             ty::Projection(proj) | ty::UnnormalizedProjection(proj) => {
167                 if self.def_id_visitor.skip_assoc_tys() {
168                     // Visitors searching for minimal visibility/reachability want to
169                     // conservatively approximate associated types like `<Type as Trait>::Alias`
170                     // as visible/reachable even if both `Type` and `Trait` are private.
171                     // Ideally, associated types should be substituted in the same way as
172                     // free type aliases, but this isn't done yet.
173                     return false;
174                 }
175                 // This will also visit substs if necessary, so we don't need to recurse.
176                 return self.visit_trait(proj.trait_ref(tcx));
177             }
178             ty::Dynamic(predicates, ..) => {
179                 // All traits in the list are considered the "primary" part of the type
180                 // and are visited by shallow visitors.
181                 for predicate in *predicates.skip_binder() {
182                     let trait_ref = match *predicate {
183                         ty::ExistentialPredicate::Trait(trait_ref) => trait_ref,
184                         ty::ExistentialPredicate::Projection(proj) => proj.trait_ref(tcx),
185                         ty::ExistentialPredicate::AutoTrait(def_id) =>
186                             ty::ExistentialTraitRef { def_id, substs: Substs::empty() },
187                     };
188                     let ty::ExistentialTraitRef { def_id, substs: _ } = trait_ref;
189                     if self.def_id_visitor.visit_def_id(def_id, "trait", &trait_ref) {
190                         return true;
191                     }
192                 }
193             }
194             ty::Opaque(def_id, ..) => {
195                 // Skip repeated `Opaque`s to avoid infinite recursion.
196                 if self.visited_opaque_tys.insert(def_id) {
197                     // The intent is to treat `impl Trait1 + Trait2` identically to
198                     // `dyn Trait1 + Trait2`. Therefore we ignore def-id of the opaque type itself
199                     // (it either has no visibility, or its visibility is insignificant, like
200                     // visibilities of type aliases) and recurse into predicates instead to go
201                     // through the trait list (default type visitor doesn't visit those traits).
202                     // All traits in the list are considered the "primary" part of the type
203                     // and are visited by shallow visitors.
204                     if self.visit_predicates(tcx.predicates_of(def_id)) {
205                         return true;
206                     }
207                 }
208             }
209             // These types don't have their own def-ids (but may have subcomponents
210             // with def-ids that should be visited recursively).
211             ty::Bool | ty::Char | ty::Int(..) | ty::Uint(..) |
212             ty::Float(..) | ty::Str | ty::Never |
213             ty::Array(..) | ty::Slice(..) | ty::Tuple(..) |
214             ty::RawPtr(..) | ty::Ref(..) | ty::FnPtr(..) |
215             ty::Param(..) | ty::Error | ty::GeneratorWitness(..) => {}
216             ty::Bound(..) | ty::Placeholder(..) | ty::Infer(..) =>
217                 bug!("unexpected type: {:?}", ty),
218         }
219
220         !self.def_id_visitor.shallow() && ty.super_visit_with(self)
221     }
222 }
223
224 fn def_id_visibility<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>, def_id: DefId)
225                                -> (ty::Visibility, Span, &'static str) {
226     match tcx.hir().as_local_node_id(def_id) {
227         Some(node_id) => {
228             let vis = match tcx.hir().get(node_id) {
229                 Node::Item(item) => &item.vis,
230                 Node::ForeignItem(foreign_item) => &foreign_item.vis,
231                 Node::TraitItem(..) | Node::Variant(..) => {
232                     return def_id_visibility(tcx, tcx.hir().get_parent_did(node_id));
233                 }
234                 Node::ImplItem(impl_item) => {
235                     match tcx.hir().get(tcx.hir().get_parent(node_id)) {
236                         Node::Item(item) => match &item.node {
237                             hir::ItemKind::Impl(.., None, _, _) => &impl_item.vis,
238                             hir::ItemKind::Impl(.., Some(trait_ref), _, _)
239                                 => return def_id_visibility(tcx, trait_ref.path.def.def_id()),
240                             kind => bug!("unexpected item kind: {:?}", kind),
241                         }
242                         node => bug!("unexpected node kind: {:?}", node),
243                     }
244                 }
245                 Node::StructCtor(vdata) => {
246                     let struct_node_id = tcx.hir().get_parent(node_id);
247                     let item = match tcx.hir().get(struct_node_id) {
248                         Node::Item(item) => item,
249                         node => bug!("unexpected node kind: {:?}", node),
250                     };
251                     let (mut ctor_vis, mut span, mut descr) =
252                         (ty::Visibility::from_hir(&item.vis, struct_node_id, tcx),
253                          item.vis.span, item.vis.node.descr());
254                     for field in vdata.fields() {
255                         let field_vis = ty::Visibility::from_hir(&field.vis, node_id, tcx);
256                         if ctor_vis.is_at_least(field_vis, tcx) {
257                             ctor_vis = field_vis;
258                             span = field.vis.span;
259                             descr = field.vis.node.descr();
260                         }
261                     }
262
263                     // If the structure is marked as non_exhaustive then lower the
264                     // visibility to within the crate.
265                     if ctor_vis == ty::Visibility::Public {
266                         let adt_def = tcx.adt_def(tcx.hir().get_parent_did(node_id));
267                         if adt_def.non_enum_variant().is_field_list_non_exhaustive() {
268                             ctor_vis = ty::Visibility::Restricted(DefId::local(CRATE_DEF_INDEX));
269                             span = attr::find_by_name(&item.attrs, "non_exhaustive").unwrap().span;
270                             descr = "crate-visible";
271                         }
272                     }
273
274                     return (ctor_vis, span, descr);
275                 }
276                 Node::Expr(expr) => {
277                     return (ty::Visibility::Restricted(tcx.hir().get_module_parent(expr.id)),
278                             expr.span, "private")
279                 }
280                 node => bug!("unexpected node kind: {:?}", node)
281             };
282             (ty::Visibility::from_hir(vis, node_id, tcx), vis.span, vis.node.descr())
283         }
284         None => {
285             let vis = tcx.visibility(def_id);
286             let descr = if vis == ty::Visibility::Public { "public" } else { "private" };
287             (vis, tcx.def_span(def_id), descr)
288         }
289     }
290 }
291
292 // Set the correct `TypeckTables` for the given `item_id` (or an empty table if
293 // there is no `TypeckTables` for the item).
294 fn item_tables<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>,
295                          node_id: ast::NodeId,
296                          empty_tables: &'a ty::TypeckTables<'tcx>)
297                          -> &'a ty::TypeckTables<'tcx> {
298     let def_id = tcx.hir().local_def_id(node_id);
299     if tcx.has_typeck_tables(def_id) { tcx.typeck_tables_of(def_id) } else { empty_tables }
300 }
301
302 fn min<'a, 'tcx>(vis1: ty::Visibility, vis2: ty::Visibility, tcx: TyCtxt<'a, 'tcx, 'tcx>)
303                  -> ty::Visibility {
304     if vis1.is_at_least(vis2, tcx) { vis2 } else { vis1 }
305 }
306
307 ////////////////////////////////////////////////////////////////////////////////
308 /// Visitor used to determine if pub(restricted) is used anywhere in the crate.
309 ///
310 /// This is done so that `private_in_public` warnings can be turned into hard errors
311 /// in crates that have been updated to use pub(restricted).
312 ////////////////////////////////////////////////////////////////////////////////
313 struct PubRestrictedVisitor<'a, 'tcx: 'a> {
314     tcx:  TyCtxt<'a, 'tcx, 'tcx>,
315     has_pub_restricted: bool,
316 }
317
318 impl<'a, 'tcx> Visitor<'tcx> for PubRestrictedVisitor<'a, 'tcx> {
319     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
320         NestedVisitorMap::All(&self.tcx.hir())
321     }
322     fn visit_vis(&mut self, vis: &'tcx hir::Visibility) {
323         self.has_pub_restricted = self.has_pub_restricted || vis.node.is_pub_restricted();
324     }
325 }
326
327 ////////////////////////////////////////////////////////////////////////////////
328 /// Visitor used to determine impl visibility and reachability.
329 ////////////////////////////////////////////////////////////////////////////////
330
331 struct FindMin<'a, 'tcx, VL: VisibilityLike> {
332     tcx: TyCtxt<'a, 'tcx, 'tcx>,
333     access_levels: &'a AccessLevels,
334     min: VL,
335 }
336
337 impl<'a, 'tcx, VL: VisibilityLike> DefIdVisitor<'a, 'tcx> for FindMin<'a, 'tcx, VL> {
338     fn tcx(&self) -> TyCtxt<'a, 'tcx, 'tcx> { self.tcx }
339     fn shallow(&self) -> bool { VL::SHALLOW }
340     fn skip_assoc_tys(&self) -> bool { true }
341     fn visit_def_id(&mut self, def_id: DefId, _kind: &str, _descr: &dyn fmt::Display) -> bool {
342         self.min = VL::new_min(self, def_id);
343         false
344     }
345 }
346
347 trait VisibilityLike: Sized {
348     const MAX: Self;
349     const SHALLOW: bool = false;
350     fn new_min<'a, 'tcx>(find: &FindMin<'a, 'tcx, Self>, def_id: DefId) -> Self;
351
352     // Returns an over-approximation (`skip_assoc_tys` = true) of visibility due to
353     // associated types for which we can't determine visibility precisely.
354     fn of_impl<'a, 'tcx>(node_id: ast::NodeId, tcx: TyCtxt<'a, 'tcx, 'tcx>,
355                          access_levels: &'a AccessLevels) -> Self {
356         let mut find = FindMin { tcx, access_levels, min: Self::MAX };
357         let def_id = tcx.hir().local_def_id(node_id);
358         find.visit(tcx.type_of(def_id));
359         if let Some(trait_ref) = tcx.impl_trait_ref(def_id) {
360             find.visit_trait(trait_ref);
361         }
362         find.min
363     }
364 }
365 impl VisibilityLike for ty::Visibility {
366     const MAX: Self = ty::Visibility::Public;
367     fn new_min<'a, 'tcx>(find: &FindMin<'a, 'tcx, Self>, def_id: DefId) -> Self {
368         min(def_id_visibility(find.tcx, def_id).0, find.min, find.tcx)
369     }
370 }
371 impl VisibilityLike for Option<AccessLevel> {
372     const MAX: Self = Some(AccessLevel::Public);
373     // Type inference is very smart sometimes.
374     // It can make an impl reachable even some components of its type or trait are unreachable.
375     // E.g. methods of `impl ReachableTrait<UnreachableTy> for ReachableTy<UnreachableTy> { ... }`
376     // can be usable from other crates (#57264). So we skip substs when calculating reachability
377     // and consider an impl reachable if its "shallow" type and trait are reachable.
378     //
379     // The assumption we make here is that type-inference won't let you use an impl without knowing
380     // both "shallow" version of its self type and "shallow" version of its trait if it exists
381     // (which require reaching the `DefId`s in them).
382     const SHALLOW: bool = true;
383     fn new_min<'a, 'tcx>(find: &FindMin<'a, 'tcx, Self>, def_id: DefId) -> Self {
384         cmp::min(if let Some(node_id) = find.tcx.hir().as_local_node_id(def_id) {
385             find.access_levels.map.get(&node_id).cloned()
386         } else {
387             Self::MAX
388         }, find.min)
389     }
390 }
391
392 ////////////////////////////////////////////////////////////////////////////////
393 /// The embargo visitor, used to determine the exports of the ast
394 ////////////////////////////////////////////////////////////////////////////////
395
396 struct EmbargoVisitor<'a, 'tcx: 'a> {
397     tcx: TyCtxt<'a, 'tcx, 'tcx>,
398
399     // Accessibility levels for reachable nodes.
400     access_levels: AccessLevels,
401     // Previous accessibility level; `None` means unreachable.
402     prev_level: Option<AccessLevel>,
403     // Has something changed in the level map?
404     changed: bool,
405 }
406
407 struct ReachEverythingInTheInterfaceVisitor<'b, 'a: 'b, 'tcx: 'a> {
408     access_level: Option<AccessLevel>,
409     item_def_id: DefId,
410     ev: &'b mut EmbargoVisitor<'a, 'tcx>,
411 }
412
413 impl<'a, 'tcx> EmbargoVisitor<'a, 'tcx> {
414     fn get(&self, id: ast::NodeId) -> Option<AccessLevel> {
415         self.access_levels.map.get(&id).cloned()
416     }
417
418     // Updates node level and returns the updated level.
419     fn update(&mut self, id: ast::NodeId, level: Option<AccessLevel>) -> Option<AccessLevel> {
420         let old_level = self.get(id);
421         // Accessibility levels can only grow.
422         if level > old_level {
423             self.access_levels.map.insert(id, level.unwrap());
424             self.changed = true;
425             level
426         } else {
427             old_level
428         }
429     }
430
431     fn reach(&mut self, item_id: ast::NodeId, access_level: Option<AccessLevel>)
432              -> ReachEverythingInTheInterfaceVisitor<'_, 'a, 'tcx> {
433         ReachEverythingInTheInterfaceVisitor {
434             access_level: cmp::min(access_level, Some(AccessLevel::Reachable)),
435             item_def_id: self.tcx.hir().local_def_id(item_id),
436             ev: self,
437         }
438     }
439 }
440
441 impl<'a, 'tcx> Visitor<'tcx> for EmbargoVisitor<'a, 'tcx> {
442     /// We want to visit items in the context of their containing
443     /// module and so forth, so supply a crate for doing a deep walk.
444     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
445         NestedVisitorMap::All(&self.tcx.hir())
446     }
447
448     fn visit_item(&mut self, item: &'tcx hir::Item) {
449         let inherited_item_level = match item.node {
450             hir::ItemKind::Impl(..) =>
451                 Option::<AccessLevel>::of_impl(item.id, self.tcx, &self.access_levels),
452             // Foreign modules inherit level from parents.
453             hir::ItemKind::ForeignMod(..) => self.prev_level,
454             // Other `pub` items inherit levels from parents.
455             hir::ItemKind::Const(..) | hir::ItemKind::Enum(..) | hir::ItemKind::ExternCrate(..) |
456             hir::ItemKind::GlobalAsm(..) | hir::ItemKind::Fn(..) | hir::ItemKind::Mod(..) |
457             hir::ItemKind::Static(..) | hir::ItemKind::Struct(..) |
458             hir::ItemKind::Trait(..) | hir::ItemKind::TraitAlias(..) |
459             hir::ItemKind::Existential(..) |
460             hir::ItemKind::Ty(..) | hir::ItemKind::Union(..) | hir::ItemKind::Use(..) => {
461                 if item.vis.node.is_pub() { self.prev_level } else { None }
462             }
463         };
464
465         // Update level of the item itself.
466         let item_level = self.update(item.id, inherited_item_level);
467
468         // Update levels of nested things.
469         match item.node {
470             hir::ItemKind::Enum(ref def, _) => {
471                 for variant in &def.variants {
472                     let variant_level = self.update(variant.node.data.id(), item_level);
473                     for field in variant.node.data.fields() {
474                         self.update(field.id, variant_level);
475                     }
476                 }
477             }
478             hir::ItemKind::Impl(.., ref trait_ref, _, ref impl_item_refs) => {
479                 for impl_item_ref in impl_item_refs {
480                     if trait_ref.is_some() || impl_item_ref.vis.node.is_pub() {
481                         self.update(impl_item_ref.id.node_id, item_level);
482                     }
483                 }
484             }
485             hir::ItemKind::Trait(.., ref trait_item_refs) => {
486                 for trait_item_ref in trait_item_refs {
487                     self.update(trait_item_ref.id.node_id, item_level);
488                 }
489             }
490             hir::ItemKind::Struct(ref def, _) | hir::ItemKind::Union(ref def, _) => {
491                 if !def.is_struct() {
492                     self.update(def.id(), item_level);
493                 }
494                 for field in def.fields() {
495                     if field.vis.node.is_pub() {
496                         self.update(field.id, item_level);
497                     }
498                 }
499             }
500             hir::ItemKind::ForeignMod(ref foreign_mod) => {
501                 for foreign_item in &foreign_mod.items {
502                     if foreign_item.vis.node.is_pub() {
503                         self.update(foreign_item.id, item_level);
504                     }
505                 }
506             }
507             hir::ItemKind::Existential(..) |
508             hir::ItemKind::Use(..) |
509             hir::ItemKind::Static(..) |
510             hir::ItemKind::Const(..) |
511             hir::ItemKind::GlobalAsm(..) |
512             hir::ItemKind::Ty(..) |
513             hir::ItemKind::Mod(..) |
514             hir::ItemKind::TraitAlias(..) |
515             hir::ItemKind::Fn(..) |
516             hir::ItemKind::ExternCrate(..) => {}
517         }
518
519         // Mark all items in interfaces of reachable items as reachable.
520         match item.node {
521             // The interface is empty.
522             hir::ItemKind::ExternCrate(..) => {}
523             // All nested items are checked by `visit_item`.
524             hir::ItemKind::Mod(..) => {}
525             // Re-exports are handled in `visit_mod`.
526             hir::ItemKind::Use(..) => {}
527             // The interface is empty.
528             hir::ItemKind::GlobalAsm(..) => {}
529             hir::ItemKind::Existential(..) => {
530                 // FIXME: This is some serious pessimization intended to workaround deficiencies
531                 // in the reachability pass (`middle/reachable.rs`). Types are marked as link-time
532                 // reachable if they are returned via `impl Trait`, even from private functions.
533                 let exist_level = cmp::max(item_level, Some(AccessLevel::ReachableFromImplTrait));
534                 self.reach(item.id, exist_level).generics().predicates().ty();
535             }
536             // Visit everything.
537             hir::ItemKind::Const(..) | hir::ItemKind::Static(..) |
538             hir::ItemKind::Fn(..) | hir::ItemKind::Ty(..) => {
539                 if item_level.is_some() {
540                     self.reach(item.id, item_level).generics().predicates().ty();
541                 }
542             }
543             hir::ItemKind::Trait(.., ref trait_item_refs) => {
544                 if item_level.is_some() {
545                     self.reach(item.id, item_level).generics().predicates();
546
547                     for trait_item_ref in trait_item_refs {
548                         let mut reach = self.reach(trait_item_ref.id.node_id, item_level);
549                         reach.generics().predicates();
550
551                         if trait_item_ref.kind == hir::AssociatedItemKind::Type &&
552                            !trait_item_ref.defaultness.has_value() {
553                             // No type to visit.
554                         } else {
555                             reach.ty();
556                         }
557                     }
558                 }
559             }
560             hir::ItemKind::TraitAlias(..) => {
561                 if item_level.is_some() {
562                     self.reach(item.id, item_level).generics().predicates();
563                 }
564             }
565             // Visit everything except for private impl items.
566             hir::ItemKind::Impl(.., ref impl_item_refs) => {
567                 if item_level.is_some() {
568                     self.reach(item.id, item_level).generics().predicates().ty().trait_ref();
569
570                     for impl_item_ref in impl_item_refs {
571                         let impl_item_level = self.get(impl_item_ref.id.node_id);
572                         if impl_item_level.is_some() {
573                             self.reach(impl_item_ref.id.node_id, impl_item_level)
574                                 .generics().predicates().ty();
575                         }
576                     }
577                 }
578             }
579
580             // Visit everything, but enum variants have their own levels.
581             hir::ItemKind::Enum(ref def, _) => {
582                 if item_level.is_some() {
583                     self.reach(item.id, item_level).generics().predicates();
584                 }
585                 for variant in &def.variants {
586                     let variant_level = self.get(variant.node.data.id());
587                     if variant_level.is_some() {
588                         for field in variant.node.data.fields() {
589                             self.reach(field.id, variant_level).ty();
590                         }
591                         // Corner case: if the variant is reachable, but its
592                         // enum is not, make the enum reachable as well.
593                         self.update(item.id, variant_level);
594                     }
595                 }
596             }
597             // Visit everything, but foreign items have their own levels.
598             hir::ItemKind::ForeignMod(ref foreign_mod) => {
599                 for foreign_item in &foreign_mod.items {
600                     let foreign_item_level = self.get(foreign_item.id);
601                     if foreign_item_level.is_some() {
602                         self.reach(foreign_item.id, foreign_item_level)
603                             .generics().predicates().ty();
604                     }
605                 }
606             }
607             // Visit everything except for private fields.
608             hir::ItemKind::Struct(ref struct_def, _) |
609             hir::ItemKind::Union(ref struct_def, _) => {
610                 if item_level.is_some() {
611                     self.reach(item.id, item_level).generics().predicates();
612                     for field in struct_def.fields() {
613                         let field_level = self.get(field.id);
614                         if field_level.is_some() {
615                             self.reach(field.id, field_level).ty();
616                         }
617                     }
618                 }
619             }
620         }
621
622         let orig_level = mem::replace(&mut self.prev_level, item_level);
623         intravisit::walk_item(self, item);
624         self.prev_level = orig_level;
625     }
626
627     fn visit_block(&mut self, b: &'tcx hir::Block) {
628         // Blocks can have public items, for example impls, but they always
629         // start as completely private regardless of publicity of a function,
630         // constant, type, field, etc., in which this block resides.
631         let orig_level = mem::replace(&mut self.prev_level, None);
632         intravisit::walk_block(self, b);
633         self.prev_level = orig_level;
634     }
635
636     fn visit_mod(&mut self, m: &'tcx hir::Mod, _sp: Span, id: ast::NodeId) {
637         // This code is here instead of in visit_item so that the
638         // crate module gets processed as well.
639         if self.prev_level.is_some() {
640             let def_id = self.tcx.hir().local_def_id(id);
641             if let Some(exports) = self.tcx.module_exports(def_id) {
642                 for export in exports.iter() {
643                     if export.vis == ty::Visibility::Public {
644                         if let Some(def_id) = export.def.opt_def_id() {
645                             if let Some(node_id) = self.tcx.hir().as_local_node_id(def_id) {
646                                 self.update(node_id, Some(AccessLevel::Exported));
647                             }
648                         }
649                     }
650                 }
651             }
652         }
653
654         intravisit::walk_mod(self, m, id);
655     }
656
657     fn visit_macro_def(&mut self, md: &'tcx hir::MacroDef) {
658         if md.legacy {
659             self.update(md.id, Some(AccessLevel::Public));
660             return
661         }
662
663         let module_did = ty::DefIdTree::parent(
664             self.tcx,
665             self.tcx.hir().local_def_id(md.id)
666         ).unwrap();
667         let mut module_id = self.tcx.hir().as_local_node_id(module_did).unwrap();
668         let level = if md.vis.node.is_pub() { self.get(module_id) } else { None };
669         let level = self.update(md.id, level);
670         if level.is_none() {
671             return
672         }
673
674         loop {
675             let module = if module_id == ast::CRATE_NODE_ID {
676                 &self.tcx.hir().krate().module
677             } else if let hir::ItemKind::Mod(ref module) =
678                           self.tcx.hir().expect_item(module_id).node {
679                 module
680             } else {
681                 unreachable!()
682             };
683             for id in &module.item_ids {
684                 self.update(id.id, level);
685             }
686             let def_id = self.tcx.hir().local_def_id(module_id);
687             if let Some(exports) = self.tcx.module_exports(def_id) {
688                 for export in exports.iter() {
689                     if let Some(node_id) = self.tcx.hir().as_local_node_id(export.def.def_id()) {
690                         self.update(node_id, level);
691                     }
692                 }
693             }
694
695             if module_id == ast::CRATE_NODE_ID {
696                 break
697             }
698             module_id = self.tcx.hir().get_parent_node(module_id);
699         }
700     }
701 }
702
703 impl<'a, 'tcx> ReachEverythingInTheInterfaceVisitor<'_, 'a, 'tcx> {
704     fn generics(&mut self) -> &mut Self {
705         for param in &self.ev.tcx.generics_of(self.item_def_id).params {
706             match param.kind {
707                 GenericParamDefKind::Type { has_default, .. } => {
708                     if has_default {
709                         self.visit(self.ev.tcx.type_of(param.def_id));
710                     }
711                 }
712                 GenericParamDefKind::Lifetime => {}
713             }
714         }
715         self
716     }
717
718     fn predicates(&mut self) -> &mut Self {
719         self.visit_predicates(self.ev.tcx.predicates_of(self.item_def_id));
720         self
721     }
722
723     fn ty(&mut self) -> &mut Self {
724         self.visit(self.ev.tcx.type_of(self.item_def_id));
725         self
726     }
727
728     fn trait_ref(&mut self) -> &mut Self {
729         if let Some(trait_ref) = self.ev.tcx.impl_trait_ref(self.item_def_id) {
730             self.visit_trait(trait_ref);
731         }
732         self
733     }
734 }
735
736 impl<'a, 'tcx> DefIdVisitor<'a, 'tcx> for ReachEverythingInTheInterfaceVisitor<'_, 'a, 'tcx> {
737     fn tcx(&self) -> TyCtxt<'a, 'tcx, 'tcx> { self.ev.tcx }
738     fn visit_def_id(&mut self, def_id: DefId, _kind: &str, _descr: &dyn fmt::Display) -> bool {
739         if let Some(node_id) = self.ev.tcx.hir().as_local_node_id(def_id) {
740             self.ev.update(node_id, self.access_level);
741         }
742         false
743     }
744 }
745
746 //////////////////////////////////////////////////////////////////////////////////////
747 /// Name privacy visitor, checks privacy and reports violations.
748 /// Most of name privacy checks are performed during the main resolution phase,
749 /// or later in type checking when field accesses and associated items are resolved.
750 /// This pass performs remaining checks for fields in struct expressions and patterns.
751 //////////////////////////////////////////////////////////////////////////////////////
752
753 struct NamePrivacyVisitor<'a, 'tcx: 'a> {
754     tcx: TyCtxt<'a, 'tcx, 'tcx>,
755     tables: &'a ty::TypeckTables<'tcx>,
756     current_item: ast::NodeId,
757     empty_tables: &'a ty::TypeckTables<'tcx>,
758 }
759
760 impl<'a, 'tcx> NamePrivacyVisitor<'a, 'tcx> {
761     // Checks that a field in a struct constructor (expression or pattern) is accessible.
762     fn check_field(&mut self,
763                    use_ctxt: Span, // syntax context of the field name at the use site
764                    span: Span, // span of the field pattern, e.g., `x: 0`
765                    def: &'tcx ty::AdtDef, // definition of the struct or enum
766                    field: &'tcx ty::FieldDef) { // definition of the field
767         let ident = Ident::new(keywords::Invalid.name(), use_ctxt);
768         let def_id = self.tcx.adjust_ident(ident, def.did, self.current_item).1;
769         if !def.is_enum() && !field.vis.is_accessible_from(def_id, self.tcx) {
770             struct_span_err!(self.tcx.sess, span, E0451, "field `{}` of {} `{}` is private",
771                              field.ident, def.variant_descr(), self.tcx.item_path_str(def.did))
772                 .span_label(span, format!("field `{}` is private", field.ident))
773                 .emit();
774         }
775     }
776 }
777
778 impl<'a, 'tcx> Visitor<'tcx> for NamePrivacyVisitor<'a, 'tcx> {
779     /// We want to visit items in the context of their containing
780     /// module and so forth, so supply a crate for doing a deep walk.
781     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
782         NestedVisitorMap::All(&self.tcx.hir())
783     }
784
785     fn visit_nested_body(&mut self, body: hir::BodyId) {
786         let orig_tables = mem::replace(&mut self.tables, self.tcx.body_tables(body));
787         let body = self.tcx.hir().body(body);
788         self.visit_body(body);
789         self.tables = orig_tables;
790     }
791
792     fn visit_item(&mut self, item: &'tcx hir::Item) {
793         let orig_current_item = mem::replace(&mut self.current_item, item.id);
794         let orig_tables =
795             mem::replace(&mut self.tables, item_tables(self.tcx, item.id, self.empty_tables));
796         intravisit::walk_item(self, item);
797         self.current_item = orig_current_item;
798         self.tables = orig_tables;
799     }
800
801     fn visit_trait_item(&mut self, ti: &'tcx hir::TraitItem) {
802         let orig_tables =
803             mem::replace(&mut self.tables, item_tables(self.tcx, ti.id, self.empty_tables));
804         intravisit::walk_trait_item(self, ti);
805         self.tables = orig_tables;
806     }
807
808     fn visit_impl_item(&mut self, ii: &'tcx hir::ImplItem) {
809         let orig_tables =
810             mem::replace(&mut self.tables, item_tables(self.tcx, ii.id, self.empty_tables));
811         intravisit::walk_impl_item(self, ii);
812         self.tables = orig_tables;
813     }
814
815     fn visit_expr(&mut self, expr: &'tcx hir::Expr) {
816         match expr.node {
817             hir::ExprKind::Struct(ref qpath, ref fields, ref base) => {
818                 let def = self.tables.qpath_def(qpath, expr.hir_id);
819                 let adt = self.tables.expr_ty(expr).ty_adt_def().unwrap();
820                 let variant = adt.variant_of_def(def);
821                 if let Some(ref base) = *base {
822                     // If the expression uses FRU we need to make sure all the unmentioned fields
823                     // are checked for privacy (RFC 736). Rather than computing the set of
824                     // unmentioned fields, just check them all.
825                     for (vf_index, variant_field) in variant.fields.iter().enumerate() {
826                         let field = fields.iter().find(|f| {
827                             self.tcx.field_index(f.id, self.tables) == vf_index
828                         });
829                         let (use_ctxt, span) = match field {
830                             Some(field) => (field.ident.span, field.span),
831                             None => (base.span, base.span),
832                         };
833                         self.check_field(use_ctxt, span, adt, variant_field);
834                     }
835                 } else {
836                     for field in fields {
837                         let use_ctxt = field.ident.span;
838                         let index = self.tcx.field_index(field.id, self.tables);
839                         self.check_field(use_ctxt, field.span, adt, &variant.fields[index]);
840                     }
841                 }
842             }
843             _ => {}
844         }
845
846         intravisit::walk_expr(self, expr);
847     }
848
849     fn visit_pat(&mut self, pat: &'tcx hir::Pat) {
850         match pat.node {
851             PatKind::Struct(ref qpath, ref fields, _) => {
852                 let def = self.tables.qpath_def(qpath, pat.hir_id);
853                 let adt = self.tables.pat_ty(pat).ty_adt_def().unwrap();
854                 let variant = adt.variant_of_def(def);
855                 for field in fields {
856                     let use_ctxt = field.node.ident.span;
857                     let index = self.tcx.field_index(field.node.id, self.tables);
858                     self.check_field(use_ctxt, field.span, adt, &variant.fields[index]);
859                 }
860             }
861             _ => {}
862         }
863
864         intravisit::walk_pat(self, pat);
865     }
866 }
867
868 ////////////////////////////////////////////////////////////////////////////////////////////
869 /// Type privacy visitor, checks types for privacy and reports violations.
870 /// Both explicitly written types and inferred types of expressions and patters are checked.
871 /// Checks are performed on "semantic" types regardless of names and their hygiene.
872 ////////////////////////////////////////////////////////////////////////////////////////////
873
874 struct TypePrivacyVisitor<'a, 'tcx: 'a> {
875     tcx: TyCtxt<'a, 'tcx, 'tcx>,
876     tables: &'a ty::TypeckTables<'tcx>,
877     current_item: DefId,
878     in_body: bool,
879     span: Span,
880     empty_tables: &'a ty::TypeckTables<'tcx>,
881 }
882
883 impl<'a, 'tcx> TypePrivacyVisitor<'a, 'tcx> {
884     fn item_is_accessible(&self, did: DefId) -> bool {
885         def_id_visibility(self.tcx, did).0.is_accessible_from(self.current_item, self.tcx)
886     }
887
888     // Take node-id of an expression or pattern and check its type for privacy.
889     fn check_expr_pat_type(&mut self, id: hir::HirId, span: Span) -> bool {
890         self.span = span;
891         if self.visit(self.tables.node_id_to_type(id)) || self.visit(self.tables.node_substs(id)) {
892             return true;
893         }
894         if let Some(adjustments) = self.tables.adjustments().get(id) {
895             for adjustment in adjustments {
896                 if self.visit(adjustment.target) {
897                     return true;
898                 }
899             }
900         }
901         false
902     }
903
904     fn check_def_id(&mut self, def_id: DefId, kind: &str, descr: &dyn fmt::Display) -> bool {
905         let is_error = !self.item_is_accessible(def_id);
906         if is_error {
907             self.tcx.sess.span_err(self.span, &format!("{} `{}` is private", kind, descr));
908         }
909         is_error
910     }
911 }
912
913 impl<'a, 'tcx> Visitor<'tcx> for TypePrivacyVisitor<'a, 'tcx> {
914     /// We want to visit items in the context of their containing
915     /// module and so forth, so supply a crate for doing a deep walk.
916     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
917         NestedVisitorMap::All(&self.tcx.hir())
918     }
919
920     fn visit_nested_body(&mut self, body: hir::BodyId) {
921         let orig_tables = mem::replace(&mut self.tables, self.tcx.body_tables(body));
922         let orig_in_body = mem::replace(&mut self.in_body, true);
923         let body = self.tcx.hir().body(body);
924         self.visit_body(body);
925         self.tables = orig_tables;
926         self.in_body = orig_in_body;
927     }
928
929     fn visit_ty(&mut self, hir_ty: &'tcx hir::Ty) {
930         self.span = hir_ty.span;
931         if self.in_body {
932             // Types in bodies.
933             if self.visit(self.tables.node_id_to_type(hir_ty.hir_id)) {
934                 return;
935             }
936         } else {
937             // Types in signatures.
938             // FIXME: This is very ineffective. Ideally each HIR type should be converted
939             // into a semantic type only once and the result should be cached somehow.
940             if self.visit(rustc_typeck::hir_ty_to_ty(self.tcx, hir_ty)) {
941                 return;
942             }
943         }
944
945         intravisit::walk_ty(self, hir_ty);
946     }
947
948     fn visit_trait_ref(&mut self, trait_ref: &'tcx hir::TraitRef) {
949         self.span = trait_ref.path.span;
950         if !self.in_body {
951             // Avoid calling `hir_trait_to_predicates` in bodies, it will ICE.
952             // The traits' privacy in bodies is already checked as a part of trait object types.
953             let (principal, projections) =
954                 rustc_typeck::hir_trait_to_predicates(self.tcx, trait_ref);
955             if self.visit_trait(*principal.skip_binder()) {
956                 return;
957             }
958             for (poly_predicate, _) in projections {
959                 let tcx = self.tcx;
960                 if self.visit(poly_predicate.skip_binder().ty) ||
961                    self.visit_trait(poly_predicate.skip_binder().projection_ty.trait_ref(tcx)) {
962                     return;
963                 }
964             }
965         }
966
967         intravisit::walk_trait_ref(self, trait_ref);
968     }
969
970     // Check types of expressions
971     fn visit_expr(&mut self, expr: &'tcx hir::Expr) {
972         if self.check_expr_pat_type(expr.hir_id, expr.span) {
973             // Do not check nested expressions if the error already happened.
974             return;
975         }
976         match expr.node {
977             hir::ExprKind::Assign(.., ref rhs) | hir::ExprKind::Match(ref rhs, ..) => {
978                 // Do not report duplicate errors for `x = y` and `match x { ... }`.
979                 if self.check_expr_pat_type(rhs.hir_id, rhs.span) {
980                     return;
981                 }
982             }
983             hir::ExprKind::MethodCall(_, span, _) => {
984                 // Method calls have to be checked specially.
985                 self.span = span;
986                 if let Some(def) = self.tables.type_dependent_defs().get(expr.hir_id) {
987                     if self.visit(self.tcx.type_of(def.def_id())) {
988                         return;
989                     }
990                 } else {
991                     self.tcx.sess.delay_span_bug(expr.span,
992                                                  "no type-dependent def for method call");
993                 }
994             }
995             _ => {}
996         }
997
998         intravisit::walk_expr(self, expr);
999     }
1000
1001     // Prohibit access to associated items with insufficient nominal visibility.
1002     //
1003     // Additionally, until better reachability analysis for macros 2.0 is available,
1004     // we prohibit access to private statics from other crates, this allows to give
1005     // more code internal visibility at link time. (Access to private functions
1006     // is already prohibited by type privacy for function types.)
1007     fn visit_qpath(&mut self, qpath: &'tcx hir::QPath, id: hir::HirId, span: Span) {
1008         let def = match *qpath {
1009             hir::QPath::Resolved(_, ref path) => match path.def {
1010                 Def::Method(..) | Def::AssociatedConst(..) |
1011                 Def::AssociatedTy(..) | Def::AssociatedExistential(..) |
1012                 Def::Static(..) => Some(path.def),
1013                 _ => None,
1014             }
1015             hir::QPath::TypeRelative(..) => {
1016                 self.tables.type_dependent_defs().get(id).cloned()
1017             }
1018         };
1019         if let Some(def) = def {
1020             let def_id = def.def_id();
1021             let is_local_static = if let Def::Static(..) = def { def_id.is_local() } else { false };
1022             if !self.item_is_accessible(def_id) && !is_local_static {
1023                 let name = match *qpath {
1024                     hir::QPath::Resolved(_, ref path) => path.to_string(),
1025                     hir::QPath::TypeRelative(_, ref segment) => segment.ident.to_string(),
1026                 };
1027                 let msg = format!("{} `{}` is private", def.kind_name(), name);
1028                 self.tcx.sess.span_err(span, &msg);
1029                 return;
1030             }
1031         }
1032
1033         intravisit::walk_qpath(self, qpath, id, span);
1034     }
1035
1036     // Check types of patterns.
1037     fn visit_pat(&mut self, pattern: &'tcx hir::Pat) {
1038         if self.check_expr_pat_type(pattern.hir_id, pattern.span) {
1039             // Do not check nested patterns if the error already happened.
1040             return;
1041         }
1042
1043         intravisit::walk_pat(self, pattern);
1044     }
1045
1046     fn visit_local(&mut self, local: &'tcx hir::Local) {
1047         if let Some(ref init) = local.init {
1048             if self.check_expr_pat_type(init.hir_id, init.span) {
1049                 // Do not report duplicate errors for `let x = y`.
1050                 return;
1051             }
1052         }
1053
1054         intravisit::walk_local(self, local);
1055     }
1056
1057     // Check types in item interfaces.
1058     fn visit_item(&mut self, item: &'tcx hir::Item) {
1059         let orig_current_item =
1060             mem::replace(&mut self.current_item, self.tcx.hir().local_def_id(item.id));
1061         let orig_in_body = mem::replace(&mut self.in_body, false);
1062         let orig_tables =
1063             mem::replace(&mut self.tables, item_tables(self.tcx, item.id, self.empty_tables));
1064         intravisit::walk_item(self, item);
1065         self.tables = orig_tables;
1066         self.in_body = orig_in_body;
1067         self.current_item = orig_current_item;
1068     }
1069
1070     fn visit_trait_item(&mut self, ti: &'tcx hir::TraitItem) {
1071         let orig_tables =
1072             mem::replace(&mut self.tables, item_tables(self.tcx, ti.id, self.empty_tables));
1073         intravisit::walk_trait_item(self, ti);
1074         self.tables = orig_tables;
1075     }
1076
1077     fn visit_impl_item(&mut self, ii: &'tcx hir::ImplItem) {
1078         let orig_tables =
1079             mem::replace(&mut self.tables, item_tables(self.tcx, ii.id, self.empty_tables));
1080         intravisit::walk_impl_item(self, ii);
1081         self.tables = orig_tables;
1082     }
1083 }
1084
1085 impl<'a, 'tcx> DefIdVisitor<'a, 'tcx> for TypePrivacyVisitor<'a, 'tcx> {
1086     fn tcx(&self) -> TyCtxt<'a, 'tcx, 'tcx> { self.tcx }
1087     fn visit_def_id(&mut self, def_id: DefId, kind: &str, descr: &dyn fmt::Display) -> bool {
1088         self.check_def_id(def_id, kind, descr)
1089     }
1090 }
1091
1092 ///////////////////////////////////////////////////////////////////////////////
1093 /// Obsolete visitors for checking for private items in public interfaces.
1094 /// These visitors are supposed to be kept in frozen state and produce an
1095 /// "old error node set". For backward compatibility the new visitor reports
1096 /// warnings instead of hard errors when the erroneous node is not in this old set.
1097 ///////////////////////////////////////////////////////////////////////////////
1098
1099 struct ObsoleteVisiblePrivateTypesVisitor<'a, 'tcx: 'a> {
1100     tcx: TyCtxt<'a, 'tcx, 'tcx>,
1101     access_levels: &'a AccessLevels,
1102     in_variant: bool,
1103     // Set of errors produced by this obsolete visitor.
1104     old_error_set: NodeSet,
1105 }
1106
1107 struct ObsoleteCheckTypeForPrivatenessVisitor<'a, 'b: 'a, 'tcx: 'b> {
1108     inner: &'a ObsoleteVisiblePrivateTypesVisitor<'b, 'tcx>,
1109     /// Whether the type refers to private types.
1110     contains_private: bool,
1111     /// Whether we've recurred at all (i.e., if we're pointing at the
1112     /// first type on which `visit_ty` was called).
1113     at_outer_type: bool,
1114     /// Whether that first type is a public path.
1115     outer_type_is_public_path: bool,
1116 }
1117
1118 impl<'a, 'tcx> ObsoleteVisiblePrivateTypesVisitor<'a, 'tcx> {
1119     fn path_is_private_type(&self, path: &hir::Path) -> bool {
1120         let did = match path.def {
1121             Def::PrimTy(..) | Def::SelfTy(..) | Def::Err => return false,
1122             def => def.def_id(),
1123         };
1124
1125         // A path can only be private if:
1126         // it's in this crate...
1127         if let Some(node_id) = self.tcx.hir().as_local_node_id(did) {
1128             // .. and it corresponds to a private type in the AST (this returns
1129             // `None` for type parameters).
1130             match self.tcx.hir().find(node_id) {
1131                 Some(Node::Item(ref item)) => !item.vis.node.is_pub(),
1132                 Some(_) | None => false,
1133             }
1134         } else {
1135             return false
1136         }
1137     }
1138
1139     fn trait_is_public(&self, trait_id: ast::NodeId) -> bool {
1140         // FIXME: this would preferably be using `exported_items`, but all
1141         // traits are exported currently (see `EmbargoVisitor.exported_trait`).
1142         self.access_levels.is_public(trait_id)
1143     }
1144
1145     fn check_generic_bound(&mut self, bound: &hir::GenericBound) {
1146         if let hir::GenericBound::Trait(ref trait_ref, _) = *bound {
1147             if self.path_is_private_type(&trait_ref.trait_ref.path) {
1148                 self.old_error_set.insert(trait_ref.trait_ref.ref_id);
1149             }
1150         }
1151     }
1152
1153     fn item_is_public(&self, id: &ast::NodeId, vis: &hir::Visibility) -> bool {
1154         self.access_levels.is_reachable(*id) || vis.node.is_pub()
1155     }
1156 }
1157
1158 impl<'a, 'b, 'tcx, 'v> Visitor<'v> for ObsoleteCheckTypeForPrivatenessVisitor<'a, 'b, 'tcx> {
1159     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'v> {
1160         NestedVisitorMap::None
1161     }
1162
1163     fn visit_ty(&mut self, ty: &hir::Ty) {
1164         if let hir::TyKind::Path(hir::QPath::Resolved(_, ref path)) = ty.node {
1165             if self.inner.path_is_private_type(path) {
1166                 self.contains_private = true;
1167                 // Found what we're looking for, so let's stop working.
1168                 return
1169             }
1170         }
1171         if let hir::TyKind::Path(_) = ty.node {
1172             if self.at_outer_type {
1173                 self.outer_type_is_public_path = true;
1174             }
1175         }
1176         self.at_outer_type = false;
1177         intravisit::walk_ty(self, ty)
1178     }
1179
1180     // Don't want to recurse into `[, .. expr]`.
1181     fn visit_expr(&mut self, _: &hir::Expr) {}
1182 }
1183
1184 impl<'a, 'tcx> Visitor<'tcx> for ObsoleteVisiblePrivateTypesVisitor<'a, 'tcx> {
1185     /// We want to visit items in the context of their containing
1186     /// module and so forth, so supply a crate for doing a deep walk.
1187     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
1188         NestedVisitorMap::All(&self.tcx.hir())
1189     }
1190
1191     fn visit_item(&mut self, item: &'tcx hir::Item) {
1192         match item.node {
1193             // Contents of a private mod can be re-exported, so we need
1194             // to check internals.
1195             hir::ItemKind::Mod(_) => {}
1196
1197             // An `extern {}` doesn't introduce a new privacy
1198             // namespace (the contents have their own privacies).
1199             hir::ItemKind::ForeignMod(_) => {}
1200
1201             hir::ItemKind::Trait(.., ref bounds, _) => {
1202                 if !self.trait_is_public(item.id) {
1203                     return
1204                 }
1205
1206                 for bound in bounds.iter() {
1207                     self.check_generic_bound(bound)
1208                 }
1209             }
1210
1211             // Impls need some special handling to try to offer useful
1212             // error messages without (too many) false positives
1213             // (i.e., we could just return here to not check them at
1214             // all, or some worse estimation of whether an impl is
1215             // publicly visible).
1216             hir::ItemKind::Impl(.., ref g, ref trait_ref, ref self_, ref impl_item_refs) => {
1217                 // `impl [... for] Private` is never visible.
1218                 let self_contains_private;
1219                 // `impl [... for] Public<...>`, but not `impl [... for]
1220                 // Vec<Public>` or `(Public,)`, etc.
1221                 let self_is_public_path;
1222
1223                 // Check the properties of the `Self` type:
1224                 {
1225                     let mut visitor = ObsoleteCheckTypeForPrivatenessVisitor {
1226                         inner: self,
1227                         contains_private: false,
1228                         at_outer_type: true,
1229                         outer_type_is_public_path: false,
1230                     };
1231                     visitor.visit_ty(&self_);
1232                     self_contains_private = visitor.contains_private;
1233                     self_is_public_path = visitor.outer_type_is_public_path;
1234                 }
1235
1236                 // Miscellaneous info about the impl:
1237
1238                 // `true` iff this is `impl Private for ...`.
1239                 let not_private_trait =
1240                     trait_ref.as_ref().map_or(true, // no trait counts as public trait
1241                                               |tr| {
1242                         let did = tr.path.def.def_id();
1243
1244                         if let Some(node_id) = self.tcx.hir().as_local_node_id(did) {
1245                             self.trait_is_public(node_id)
1246                         } else {
1247                             true // external traits must be public
1248                         }
1249                     });
1250
1251                 // `true` iff this is a trait impl or at least one method is public.
1252                 //
1253                 // `impl Public { $( fn ...() {} )* }` is not visible.
1254                 //
1255                 // This is required over just using the methods' privacy
1256                 // directly because we might have `impl<T: Foo<Private>> ...`,
1257                 // and we shouldn't warn about the generics if all the methods
1258                 // are private (because `T` won't be visible externally).
1259                 let trait_or_some_public_method =
1260                     trait_ref.is_some() ||
1261                     impl_item_refs.iter()
1262                                  .any(|impl_item_ref| {
1263                                      let impl_item = self.tcx.hir().impl_item(impl_item_ref.id);
1264                                      match impl_item.node {
1265                                          hir::ImplItemKind::Const(..) |
1266                                          hir::ImplItemKind::Method(..) => {
1267                                              self.access_levels.is_reachable(impl_item.id)
1268                                          }
1269                                          hir::ImplItemKind::Existential(..) |
1270                                          hir::ImplItemKind::Type(_) => false,
1271                                      }
1272                                  });
1273
1274                 if !self_contains_private &&
1275                         not_private_trait &&
1276                         trait_or_some_public_method {
1277
1278                     intravisit::walk_generics(self, g);
1279
1280                     match *trait_ref {
1281                         None => {
1282                             for impl_item_ref in impl_item_refs {
1283                                 // This is where we choose whether to walk down
1284                                 // further into the impl to check its items. We
1285                                 // should only walk into public items so that we
1286                                 // don't erroneously report errors for private
1287                                 // types in private items.
1288                                 let impl_item = self.tcx.hir().impl_item(impl_item_ref.id);
1289                                 match impl_item.node {
1290                                     hir::ImplItemKind::Const(..) |
1291                                     hir::ImplItemKind::Method(..)
1292                                         if self.item_is_public(&impl_item.id, &impl_item.vis) =>
1293                                     {
1294                                         intravisit::walk_impl_item(self, impl_item)
1295                                     }
1296                                     hir::ImplItemKind::Type(..) => {
1297                                         intravisit::walk_impl_item(self, impl_item)
1298                                     }
1299                                     _ => {}
1300                                 }
1301                             }
1302                         }
1303                         Some(ref tr) => {
1304                             // Any private types in a trait impl fall into three
1305                             // categories.
1306                             // 1. mentioned in the trait definition
1307                             // 2. mentioned in the type params/generics
1308                             // 3. mentioned in the associated types of the impl
1309                             //
1310                             // Those in 1. can only occur if the trait is in
1311                             // this crate and will've been warned about on the
1312                             // trait definition (there's no need to warn twice
1313                             // so we don't check the methods).
1314                             //
1315                             // Those in 2. are warned via walk_generics and this
1316                             // call here.
1317                             intravisit::walk_path(self, &tr.path);
1318
1319                             // Those in 3. are warned with this call.
1320                             for impl_item_ref in impl_item_refs {
1321                                 let impl_item = self.tcx.hir().impl_item(impl_item_ref.id);
1322                                 if let hir::ImplItemKind::Type(ref ty) = impl_item.node {
1323                                     self.visit_ty(ty);
1324                                 }
1325                             }
1326                         }
1327                     }
1328                 } else if trait_ref.is_none() && self_is_public_path {
1329                     // `impl Public<Private> { ... }`. Any public static
1330                     // methods will be visible as `Public::foo`.
1331                     let mut found_pub_static = false;
1332                     for impl_item_ref in impl_item_refs {
1333                         if self.item_is_public(&impl_item_ref.id.node_id, &impl_item_ref.vis) {
1334                             let impl_item = self.tcx.hir().impl_item(impl_item_ref.id);
1335                             match impl_item_ref.kind {
1336                                 hir::AssociatedItemKind::Const => {
1337                                     found_pub_static = true;
1338                                     intravisit::walk_impl_item(self, impl_item);
1339                                 }
1340                                 hir::AssociatedItemKind::Method { has_self: false } => {
1341                                     found_pub_static = true;
1342                                     intravisit::walk_impl_item(self, impl_item);
1343                                 }
1344                                 _ => {}
1345                             }
1346                         }
1347                     }
1348                     if found_pub_static {
1349                         intravisit::walk_generics(self, g)
1350                     }
1351                 }
1352                 return
1353             }
1354
1355             // `type ... = ...;` can contain private types, because
1356             // we're introducing a new name.
1357             hir::ItemKind::Ty(..) => return,
1358
1359             // Not at all public, so we don't care.
1360             _ if !self.item_is_public(&item.id, &item.vis) => {
1361                 return;
1362             }
1363
1364             _ => {}
1365         }
1366
1367         // We've carefully constructed it so that if we're here, then
1368         // any `visit_ty`'s will be called on things that are in
1369         // public signatures, i.e., things that we're interested in for
1370         // this visitor.
1371         intravisit::walk_item(self, item);
1372     }
1373
1374     fn visit_generics(&mut self, generics: &'tcx hir::Generics) {
1375         for param in &generics.params {
1376             for bound in &param.bounds {
1377                 self.check_generic_bound(bound);
1378             }
1379         }
1380         for predicate in &generics.where_clause.predicates {
1381             match predicate {
1382                 hir::WherePredicate::BoundPredicate(bound_pred) => {
1383                     for bound in bound_pred.bounds.iter() {
1384                         self.check_generic_bound(bound)
1385                     }
1386                 }
1387                 hir::WherePredicate::RegionPredicate(_) => {}
1388                 hir::WherePredicate::EqPredicate(eq_pred) => {
1389                     self.visit_ty(&eq_pred.rhs_ty);
1390                 }
1391             }
1392         }
1393     }
1394
1395     fn visit_foreign_item(&mut self, item: &'tcx hir::ForeignItem) {
1396         if self.access_levels.is_reachable(item.id) {
1397             intravisit::walk_foreign_item(self, item)
1398         }
1399     }
1400
1401     fn visit_ty(&mut self, t: &'tcx hir::Ty) {
1402         if let hir::TyKind::Path(hir::QPath::Resolved(_, ref path)) = t.node {
1403             if self.path_is_private_type(path) {
1404                 self.old_error_set.insert(t.id);
1405             }
1406         }
1407         intravisit::walk_ty(self, t)
1408     }
1409
1410     fn visit_variant(&mut self,
1411                      v: &'tcx hir::Variant,
1412                      g: &'tcx hir::Generics,
1413                      item_id: ast::NodeId) {
1414         if self.access_levels.is_reachable(v.node.data.id()) {
1415             self.in_variant = true;
1416             intravisit::walk_variant(self, v, g, item_id);
1417             self.in_variant = false;
1418         }
1419     }
1420
1421     fn visit_struct_field(&mut self, s: &'tcx hir::StructField) {
1422         if s.vis.node.is_pub() || self.in_variant {
1423             intravisit::walk_struct_field(self, s);
1424         }
1425     }
1426
1427     // We don't need to introspect into these at all: an
1428     // expression/block context can't possibly contain exported things.
1429     // (Making them no-ops stops us from traversing the whole AST without
1430     // having to be super careful about our `walk_...` calls above.)
1431     fn visit_block(&mut self, _: &'tcx hir::Block) {}
1432     fn visit_expr(&mut self, _: &'tcx hir::Expr) {}
1433 }
1434
1435 ///////////////////////////////////////////////////////////////////////////////
1436 /// SearchInterfaceForPrivateItemsVisitor traverses an item's interface and
1437 /// finds any private components in it.
1438 /// PrivateItemsInPublicInterfacesVisitor ensures there are no private types
1439 /// and traits in public interfaces.
1440 ///////////////////////////////////////////////////////////////////////////////
1441
1442 struct SearchInterfaceForPrivateItemsVisitor<'a, 'tcx: 'a> {
1443     tcx: TyCtxt<'a, 'tcx, 'tcx>,
1444     item_def_id: DefId,
1445     span: Span,
1446     /// The visitor checks that each component type is at least this visible.
1447     required_visibility: ty::Visibility,
1448     has_pub_restricted: bool,
1449     has_old_errors: bool,
1450     in_assoc_ty: bool,
1451 }
1452
1453 impl<'a, 'tcx: 'a> SearchInterfaceForPrivateItemsVisitor<'a, 'tcx> {
1454     fn generics(&mut self) -> &mut Self {
1455         for param in &self.tcx.generics_of(self.item_def_id).params {
1456             match param.kind {
1457                 GenericParamDefKind::Type { has_default, .. } => {
1458                     if has_default {
1459                         self.visit(self.tcx.type_of(param.def_id));
1460                     }
1461                 }
1462                 GenericParamDefKind::Lifetime => {}
1463             }
1464         }
1465         self
1466     }
1467
1468     fn predicates(&mut self) -> &mut Self {
1469         // N.B., we use `explicit_predicates_of` and not `predicates_of`
1470         // because we don't want to report privacy errors due to where
1471         // clauses that the compiler inferred. We only want to
1472         // consider the ones that the user wrote. This is important
1473         // for the inferred outlives rules; see
1474         // `src/test/ui/rfc-2093-infer-outlives/privacy.rs`.
1475         self.visit_predicates(self.tcx.explicit_predicates_of(self.item_def_id));
1476         self
1477     }
1478
1479     fn ty(&mut self) -> &mut Self {
1480         self.visit(self.tcx.type_of(self.item_def_id));
1481         self
1482     }
1483
1484     fn check_def_id(&mut self, def_id: DefId, kind: &str, descr: &dyn fmt::Display) -> bool {
1485         let node_id = match self.tcx.hir().as_local_node_id(def_id) {
1486             Some(node_id) => node_id,
1487             None => return false,
1488         };
1489
1490         let (vis, vis_span, vis_descr) = def_id_visibility(self.tcx, def_id);
1491         if !vis.is_at_least(self.required_visibility, self.tcx) {
1492             let msg = format!("{} {} `{}` in public interface", vis_descr, kind, descr);
1493             if self.has_pub_restricted || self.has_old_errors || self.in_assoc_ty {
1494                 let mut err = if kind == "trait" {
1495                     struct_span_err!(self.tcx.sess, self.span, E0445, "{}", msg)
1496                 } else {
1497                     struct_span_err!(self.tcx.sess, self.span, E0446, "{}", msg)
1498                 };
1499                 err.span_label(self.span, format!("can't leak {} {}", vis_descr, kind));
1500                 err.span_label(vis_span, format!("`{}` declared as {}", descr, vis_descr));
1501                 err.emit();
1502             } else {
1503                 let err_code = if kind == "trait" { "E0445" } else { "E0446" };
1504                 self.tcx.lint_node(lint::builtin::PRIVATE_IN_PUBLIC, node_id, self.span,
1505                                    &format!("{} (error {})", msg, err_code));
1506             }
1507         }
1508         false
1509     }
1510 }
1511
1512 impl<'a, 'tcx> DefIdVisitor<'a, 'tcx> for SearchInterfaceForPrivateItemsVisitor<'a, 'tcx> {
1513     fn tcx(&self) -> TyCtxt<'a, 'tcx, 'tcx> { self.tcx }
1514     fn visit_def_id(&mut self, def_id: DefId, kind: &str, descr: &dyn fmt::Display) -> bool {
1515         self.check_def_id(def_id, kind, descr)
1516     }
1517 }
1518
1519 struct PrivateItemsInPublicInterfacesVisitor<'a, 'tcx: 'a> {
1520     tcx: TyCtxt<'a, 'tcx, 'tcx>,
1521     has_pub_restricted: bool,
1522     old_error_set: &'a NodeSet,
1523 }
1524
1525 impl<'a, 'tcx> PrivateItemsInPublicInterfacesVisitor<'a, 'tcx> {
1526     fn check(&self, item_id: ast::NodeId, required_visibility: ty::Visibility)
1527              -> SearchInterfaceForPrivateItemsVisitor<'a, 'tcx> {
1528         let mut has_old_errors = false;
1529
1530         // Slow path taken only if there any errors in the crate.
1531         for &id in self.old_error_set {
1532             // Walk up the nodes until we find `item_id` (or we hit a root).
1533             let mut id = id;
1534             loop {
1535                 if id == item_id {
1536                     has_old_errors = true;
1537                     break;
1538                 }
1539                 let parent = self.tcx.hir().get_parent_node(id);
1540                 if parent == id {
1541                     break;
1542                 }
1543                 id = parent;
1544             }
1545
1546             if has_old_errors {
1547                 break;
1548             }
1549         }
1550
1551         SearchInterfaceForPrivateItemsVisitor {
1552             tcx: self.tcx,
1553             item_def_id: self.tcx.hir().local_def_id(item_id),
1554             span: self.tcx.hir().span(item_id),
1555             required_visibility,
1556             has_pub_restricted: self.has_pub_restricted,
1557             has_old_errors,
1558             in_assoc_ty: false,
1559         }
1560     }
1561 }
1562
1563 impl<'a, 'tcx> Visitor<'tcx> for PrivateItemsInPublicInterfacesVisitor<'a, 'tcx> {
1564     fn nested_visit_map<'this>(&'this mut self) -> NestedVisitorMap<'this, 'tcx> {
1565         NestedVisitorMap::OnlyBodies(&self.tcx.hir())
1566     }
1567
1568     fn visit_item(&mut self, item: &'tcx hir::Item) {
1569         let tcx = self.tcx;
1570         let item_visibility = ty::Visibility::from_hir(&item.vis, item.id, tcx);
1571
1572         match item.node {
1573             // Crates are always public.
1574             hir::ItemKind::ExternCrate(..) => {}
1575             // All nested items are checked by `visit_item`.
1576             hir::ItemKind::Mod(..) => {}
1577             // Checked in resolve.
1578             hir::ItemKind::Use(..) => {}
1579             // No subitems.
1580             hir::ItemKind::GlobalAsm(..) => {}
1581             // Subitems of these items have inherited publicity.
1582             hir::ItemKind::Const(..) | hir::ItemKind::Static(..) |
1583             hir::ItemKind::Fn(..) | hir::ItemKind::Ty(..) => {
1584                 self.check(item.id, item_visibility).generics().predicates().ty();
1585             }
1586             hir::ItemKind::Existential(..) => {
1587                 // `ty()` for existential types is the underlying type,
1588                 // it's not a part of interface, so we skip it.
1589                 self.check(item.id, item_visibility).generics().predicates();
1590             }
1591             hir::ItemKind::Trait(.., ref trait_item_refs) => {
1592                 self.check(item.id, item_visibility).generics().predicates();
1593
1594                 for trait_item_ref in trait_item_refs {
1595                     let mut check = self.check(trait_item_ref.id.node_id, item_visibility);
1596                     check.in_assoc_ty = trait_item_ref.kind == hir::AssociatedItemKind::Type;
1597                     check.generics().predicates();
1598
1599                     if trait_item_ref.kind == hir::AssociatedItemKind::Type &&
1600                        !trait_item_ref.defaultness.has_value() {
1601                         // No type to visit.
1602                     } else {
1603                         check.ty();
1604                     }
1605                 }
1606             }
1607             hir::ItemKind::TraitAlias(..) => {
1608                 self.check(item.id, item_visibility).generics().predicates();
1609             }
1610             hir::ItemKind::Enum(ref def, _) => {
1611                 self.check(item.id, item_visibility).generics().predicates();
1612
1613                 for variant in &def.variants {
1614                     for field in variant.node.data.fields() {
1615                         self.check(field.id, item_visibility).ty();
1616                     }
1617                 }
1618             }
1619             // Subitems of foreign modules have their own publicity.
1620             hir::ItemKind::ForeignMod(ref foreign_mod) => {
1621                 for foreign_item in &foreign_mod.items {
1622                     let vis = ty::Visibility::from_hir(&foreign_item.vis, item.id, tcx);
1623                     self.check(foreign_item.id, vis).generics().predicates().ty();
1624                 }
1625             }
1626             // Subitems of structs and unions have their own publicity.
1627             hir::ItemKind::Struct(ref struct_def, _) |
1628             hir::ItemKind::Union(ref struct_def, _) => {
1629                 self.check(item.id, item_visibility).generics().predicates();
1630
1631                 for field in struct_def.fields() {
1632                     let field_visibility = ty::Visibility::from_hir(&field.vis, item.id, tcx);
1633                     self.check(field.id, min(item_visibility, field_visibility, tcx)).ty();
1634                 }
1635             }
1636             // An inherent impl is public when its type is public
1637             // Subitems of inherent impls have their own publicity.
1638             // A trait impl is public when both its type and its trait are public
1639             // Subitems of trait impls have inherited publicity.
1640             hir::ItemKind::Impl(.., ref trait_ref, _, ref impl_item_refs) => {
1641                 let impl_vis = ty::Visibility::of_impl(item.id, tcx, &Default::default());
1642                 self.check(item.id, impl_vis).generics().predicates();
1643                 for impl_item_ref in impl_item_refs {
1644                     let impl_item = tcx.hir().impl_item(impl_item_ref.id);
1645                     let impl_item_vis = if trait_ref.is_none() {
1646                         min(ty::Visibility::from_hir(&impl_item.vis, item.id, tcx), impl_vis, tcx)
1647                     } else {
1648                         impl_vis
1649                     };
1650                     let mut check = self.check(impl_item.id, impl_item_vis);
1651                     check.in_assoc_ty = impl_item_ref.kind == hir::AssociatedItemKind::Type;
1652                     check.generics().predicates().ty();
1653                 }
1654             }
1655         }
1656     }
1657 }
1658
1659 pub fn provide(providers: &mut Providers) {
1660     *providers = Providers {
1661         privacy_access_levels,
1662         ..*providers
1663     };
1664 }
1665
1666 pub fn check_crate<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>) -> Lrc<AccessLevels> {
1667     tcx.privacy_access_levels(LOCAL_CRATE)
1668 }
1669
1670 fn privacy_access_levels<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>,
1671                                    krate: CrateNum)
1672                                    -> Lrc<AccessLevels> {
1673     assert_eq!(krate, LOCAL_CRATE);
1674
1675     let krate = tcx.hir().krate();
1676     let empty_tables = ty::TypeckTables::empty(None);
1677
1678     // Check privacy of names not checked in previous compilation stages.
1679     let mut visitor = NamePrivacyVisitor {
1680         tcx,
1681         tables: &empty_tables,
1682         current_item: CRATE_NODE_ID,
1683         empty_tables: &empty_tables,
1684     };
1685     intravisit::walk_crate(&mut visitor, krate);
1686
1687     // Check privacy of explicitly written types and traits as well as
1688     // inferred types of expressions and patterns.
1689     let mut visitor = TypePrivacyVisitor {
1690         tcx,
1691         tables: &empty_tables,
1692         current_item: DefId::local(CRATE_DEF_INDEX),
1693         in_body: false,
1694         span: krate.span,
1695         empty_tables: &empty_tables,
1696     };
1697     intravisit::walk_crate(&mut visitor, krate);
1698
1699     // Build up a set of all exported items in the AST. This is a set of all
1700     // items which are reachable from external crates based on visibility.
1701     let mut visitor = EmbargoVisitor {
1702         tcx,
1703         access_levels: Default::default(),
1704         prev_level: Some(AccessLevel::Public),
1705         changed: false,
1706     };
1707     loop {
1708         intravisit::walk_crate(&mut visitor, krate);
1709         if visitor.changed {
1710             visitor.changed = false;
1711         } else {
1712             break
1713         }
1714     }
1715     visitor.update(ast::CRATE_NODE_ID, Some(AccessLevel::Public));
1716
1717     {
1718         let mut visitor = ObsoleteVisiblePrivateTypesVisitor {
1719             tcx,
1720             access_levels: &visitor.access_levels,
1721             in_variant: false,
1722             old_error_set: Default::default(),
1723         };
1724         intravisit::walk_crate(&mut visitor, krate);
1725
1726
1727         let has_pub_restricted = {
1728             let mut pub_restricted_visitor = PubRestrictedVisitor {
1729                 tcx,
1730                 has_pub_restricted: false
1731             };
1732             intravisit::walk_crate(&mut pub_restricted_visitor, krate);
1733             pub_restricted_visitor.has_pub_restricted
1734         };
1735
1736         // Check for private types and traits in public interfaces.
1737         let mut visitor = PrivateItemsInPublicInterfacesVisitor {
1738             tcx,
1739             has_pub_restricted,
1740             old_error_set: &visitor.old_error_set,
1741         };
1742         krate.visit_all_item_likes(&mut DeepVisitor::new(&mut visitor));
1743     }
1744
1745     Lrc::new(visitor.access_levels)
1746 }
1747
1748 __build_diagnostic_array! { librustc_privacy, DIAGNOSTICS }