[10046][patch] Хранение GUIDов передаваемых предметов при обмене

[zergtmn]

Фикс эксплоита: http://paste2.org/p/867712
Надо ли обнулять гуид в tradeItems, если вдруг вещь, находящаяся в обмене была удалена?

ASSERT-то там на кой? return NULL и обработку этого случая в данном конкретном примере сделать не так уж и тяжело. А специально крашить свое приложение из-за каждого доморощенного Кулибина - это перебор, я считаю. И так падает часто.

[zergtmn]

Крашить не будет, т.к. проверка номера слота, присылаемого клиентом есть в HandleSetTradeItemOpcode и SpellCastTargets::Update (при наложении чар через обмен).

Сообщение от zergtmn

Крашить не будет, т.к. проверка номера слота, присылаемого клиентом есть в HandleSetTradeItemOpcode и SpellCastTargets::Update (при наложении чар через обмен).

Я-то думал Вы дважды проверить решили. Значит придираюсь дальше. Если есть проверка номера слота перед вызовом функции, зачем проверять еще раз с помощью ассерта? Число - это не указатель на что-то ведь, память по адресу не убьется, если очень не постараться.

[Vladimir]

В [10046]. Спасибо