feedc0de/boost_function
1
0
Fork 0
forked from boostorg/function
Code Pull Requests Activity

Merge pull request #16 from joakimtosteberg/fix-aliasing-issue

Browse Source 
Use char type directly to avoid invalid aliasing in one more place
This commit is contained in:
Edward Diener
2018-04-04 08:48:29 -04:00
committed by GitHub
parent 0482db3a79 9d0acd5195
commit 52fae95412
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
include/boost/function/function_template.hpp
View File

@ -904,7 +904,9 @@ namespace boost {
if (!f.empty()) { if (!f.empty()) {
this->vtable = f.vtable; this->vtable = f.vtable;
if (this->has_trivial_copy_and_destroy()) if (this->has_trivial_copy_and_destroy())
this->functor = f.functor; // Don't operate on storage directly since union type doesn't relax
// strict aliasing rules, despite of having member char type.
std::memcpy(this->functor.data, f.functor.data, sizeof(boost::detail::function::function_buffer));
else else
get_vtable()->base.manager(f.functor, this->functor, get_vtable()->base.manager(f.functor, this->functor,
boost::detail::function::clone_functor_tag); boost::detail::function::clone_functor_tag);