[11261] Check gaps in quest RewChoiceItem* data.

[newsbot]

автор: VladimirMangos


<pre>m src/game/ObjectMgr.cpp
</pre>
<pre style='white-space:pre-wrap;width:81ex'>[11261] Check gaps in quest RewChoiceItem* data.

Client can crash if data not continues.

Thanks Chero and Joey for help in research problem ^^</pre>




Дальше...

[YuruY]

Смысл?

[virusav]

2011-03-17 22:29:43 Quest 316 has `RewChoiceItemId4` = 0 but `RewChoiceItemId5` = 33079, client can crash at like data.
2011-03-17 22:29:43 Quest 316 has `RewChoiceItemId2` = 0 but `RewChoiceItemId3` = 22114, client can crash at like data.

Поставил итемы и количества по-порядку - ошибки исчезли.

Но если данные придут в таком порядке из кеша, то при следующем обновлении данными из кеша все вернется назад, снова будут ошибки в логе.

[Vladimir]

Вы уверены что такие данные есть? Я проверял по данным из UDB - там таких случаев нет. Если такие данные есть то я откачу, конечно, тем более у меня подтверждения для 3.x ytт креша - но клиент 1.12.1 явно крешится на таких данных.

[NeatElves]

Есть там предметы по кешу)):

REPLACE INTO quest_template VALUES ('enGB', 316, 415, 316, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 7, 50, 60, 0, 0, 0, 0, 0, 8, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 20371, 1, 20651, 1, 22114, 1, 22780, 1, 33079, 1, 0, 0, 469, 0, 0, 0, 0, 7, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, '<UNUSED 1>', '', '', '', 'Return to Test Squirrel.', 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2688, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, '', '', '', '');

[Vladimir]

f*ck. Я уже revert сделал

[virusav]

Не надо торопиться.

Но если данные придут в таком порядке из кеша, то при следующем обновлении данными из кеша все вернется назад, снова будут ошибки в логе.

Вы уверены что такие данные есть?

Я не успел ответить, т.к. ночь была на дворе, спать пошел.

Сегодня прогнал скриптом 9408 квестов из кеша, но не встретил ни одного квеста, где бы встречались итемы в качестве вознаграждения, не идущие в полях подряд.

В таком случае лучше вернуть коммит, т.к. он, как я понимаю, пойдет для 1.12.1 и мастер-ветки.

[Vladimir]

Ну для 1.x он просто необходим так как клиент крешится на такие данные, а для 3.x/2.x видимо верну/добавлю раз помогло поймать глюк в данных.

[YuruY]

Может данные нужны по какомуто квесту, проверить?
335а, (401 убил, но на работе остался вроде) 403а, 406а? (фулл есть с нескольких клиентов)

[Vladimir]

Я откатил - т.е. передобавил проверку, так что если какие еще случаи будут это будет сообщенно ядром при загрузке.

[Vladimir]

Я не успел ответить, т.к. ночь была на дворе, спать пошел.

Мои эмоции касались исключительно моей торопыжкости

[YuruY]

 UPDATE `quest_template` SET `RewChoiceItemId1` = 20371, `RewChoiceItemId2` = 20651, `RewChoiceItemId3` = 22114, `RewChoiceItemId4` = 22780, `RewChoiceItemId5` = 33079 WHERE `entry` = 316;

вот так по кешу, но второго и четвертого предмета нет в базе (квест не юзабельный, в кеше они не приходят), если их занулить - ядро даст ошибку