* Revert "fix"

This reverts commit 51aa8940.

* check sizeof deckBuffer in LoadDeck

* clear current_deck when selecting a wrong file

* use local var in ServerEchoRead, ClientRead

* zero init in GetResponse

* verify the source of BufferIO::CopyWStr

* add BufferIO::NullTerminate

* use MAX_STRING_ID

* use constant

* use constant

This is very popular in custom card or rush duel communities. So we have to change it back.

* use memcpy in DuelClient, NetServer

* NetServer: discard too large packet

* DuelClient: discard too large packet

* replace memcpy with std::memcpy

* disconnect when the packet is too large

* check packet_len

* check len in HandleCTOSPacket()

* check len in HandleSTOCPacketLan()

* remove bool type in HostInfo

* check len in packet handling

* add boundary check to STOC_CHAT packet

* use memcpy in DuelMode::JoinGame

* add DECK_MAX_SIZE, DECK_MIN_SIZE

* fix DuelMode::UpdateDeck()

* fix

* fix Manager::OpenDeckFile()

* update DuelMode init

* update DuelMode::JoinGame

* update DuelMode::GetResponse

* Revert "check len in packet handling"

This reverts commit 2b5c12f3.

* use memcpy in GetPosition

* check struct size by static_assert

* update DuelMode::UpdateDeck

* check if the struct is trivially copyable

* replace unsigned short to uint16_t

* replace unsigned int to uint32_t

* add padding member to STOC_HS_PlayerEnter

* workaround: use STOC_HS_PlayerEnter_size

* use std::memcpy in class Replay

* remove pointer casting

* check sizeof(wchar_t)

* EncodeUTF8: use unsigned operand

* DecodeUTF8: use unsigned operand

* EncodeUTF8: check array size

* DecodeUTF8: check array size

* replace sprintf with snprintf

* zero initialize

* remove outdated macro

* remove outdated macro in clzma

enable windbot as default

See merge request mycard/ygopro!11