Moving *this before calling get_immediate_executor() causes undefined behavior

[Jackarain]

In the following code, std::move(*this) is used before calling get_immediate_executor(). This can result in undefined behavior, as the this object may become invalid after the move operation.

beast/include/boost/beast/core/impl/basic_stream.hpp

Lines 328 to 330 in 21545db

	net::dispatch(this->get_immediate_executor(),
	net::append(std::move(*this), ec, 0));
	}

[ashtum]

@Jackarain Thanks for reporting this. I can confirm that this is a bug, as the order of evaluation of function arguments is unspecified and move can happen before dereferencing this.

[Jackarain]

感谢您的回复，以及迅速的作出修复，感谢

[vinniefalco]

Great work, @Jackarain :) thanks.