Repository navigation
regression: transmute_copy validates sizing #100545
Description
Activity
- addedT-libs-api[DEPRECATED; DO NOT USE][DEPRECATED; DO NOT USE]regression-from-stable-to-betaPerformance or correctness regression from stable to beta.Performance or correctness regression from stable to beta.I-libs-api-nominated[DEPRECATED; DO NOT USE][DEPRECATED; DO NOT USE]
on Aug 14, 2022 - addedI-prioritizeIssue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}Issue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}
on Aug 14, 2022 - removedI-prioritizeIssue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}Issue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}
on Aug 14, 2022 I believe this is the same as #99923
Edit: I meant "the same category as".
Seems like a much easier issue to explain, at least - not sure I would quite call it "same as" pointer/integer transmutes :) Though the overall effect of breaking unsafe code seems quite similar.
Reacted by Esteban KuberI believe this is the same as #99923
The issues seem entirely unrelated to me, in which sense are they "the same"?
Or did you mean they are broadly the same category, being related to UB detection? But even then, #99923 is about compile-time UB, and this here is primarily about run-time UB, and those are very different in how we treat them.
Yeah, we could have a
cfg_attr(debug_assertions, track_caller)but that would only help once we have a good system for letting people use a debug build of the standard library...- addedrelnotesMarks issues that should be documented in the release notes of the next release.Marks issues that should be documented in the release notes of the next release.
on Sep 13, 2022
transmute_copynow validates reasonable sizes and panics if not as of #98839, this causes some breakage (detected at the time in #98839 (comment)).Mostly filing this issue to make sure we add a release notes compat note about this, but I don't expect us to do anything further. Nominating for T-libs-api, though, mostly just for awareness.
cc @tmandry / @joshtriplett -- let's make sure this makes it into 1.64 release notes