Skip to content

Commit

Permalink
Add test for issue rust-lang#33017
Browse files Browse the repository at this point in the history
  • Loading branch information
marmeladema committed Apr 23, 2021
1 parent 9a35232 commit a657e17
Show file tree
Hide file tree
Showing 2 changed files with 62 additions and 0 deletions.
45 changes: 45 additions & 0 deletions src/test/ui/specialization/issue-33017.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
// Test to ensure that trait bounds are propertly
// checked on specializable associated types

#![allow(incomplete_features)]
#![feature(specialization)]

trait UncheckedCopy: Sized {
type Output: From<Self> + Copy + Into<Self>;
}

impl<T> UncheckedCopy for T {
default type Output = Self;
//~^ ERROR: the trait bound `T: Copy` is not satisfied
}

fn unchecked_copy<T: UncheckedCopy>(other: &T::Output) -> T {
(*other).into()
}

fn bug(origin: String) {
// Turn the String into it's Output type...
// Which we can just do by `.into()`, the assoc type states `From<Self>`.
let origin_output = origin.into();

// Make a copy of String::Output, which is a String...
let mut copy: String = unchecked_copy::<String>(&origin_output);

// Turn the Output type into a String again,
// Which we can just do by `.into()`, the assoc type states `Into<Self>`.
let mut origin: String = origin_output.into();

// assert both Strings use the same buffer.
assert_eq!(copy.as_ptr(), origin.as_ptr());

// Any use of the copy we made becomes invalid,
drop(origin);

// OH NO! UB UB UB UB!
copy.push_str(" world!");
println!("{}", copy);
}

fn main() {
bug(String::from("hello"));
}
17 changes: 17 additions & 0 deletions src/test/ui/specialization/issue-33017.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
error[E0277]: the trait bound `T: Copy` is not satisfied
--> $DIR/issue-33017.rs:12:5
|
LL | type Output: From<Self> + Copy + Into<Self>;
| ---- required by this bound in `UncheckedCopy::Output`
...
LL | default type Output = Self;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `Copy` is not implemented for `T`
|
help: consider restricting type parameter `T`
|
LL | impl<T: std::marker::Copy> UncheckedCopy for T {
| ^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error

For more information about this error, try `rustc --explain E0277`.

0 comments on commit a657e17

Please sign in to comment.