![]() |
|
---|
|
CMaNGOS Commits Изменения, принятые в официальный репозиторий ядра CMaNGOS. git://github.com/cmangos/mangos-wotlk.git |
![]() |
|
Опции темы | Поиск в этой теме | Опции просмотра |
![]() |
#1 |
News Bot
Регистрация: 20.11.2009
Сообщений: 1,290
Сказал(а) спасибо: 0
Поблагодарили 5 раз(а) в 5 сообщениях
![]() |
![]()
автор: 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> Дальше... |
![]() |
![]() |
![]() |
#2 |
YTDB dev
Регистрация: 05.11.2009
Адрес: Thunder Bluff
Сообщений: 3,625
Сказал(а) спасибо: 7,839
Поблагодарили 3,255 раз(а) в 498 сообщениях
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() |
![]()
Смысл?
|
![]() |
![]() |
![]() |
#3 |
Брянский волк
|
![]() Код:
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. Но если данные придут в таком порядке из кеша, то при следующем обновлении данными из кеша все вернется назад, снова будут ошибки в логе. |
![]() |
![]() |
Пользователь сказал cпасибо: | Vladimir (18.03.2011) |
![]() |
#4 |
MaNGOS Dev
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
![]() ![]() |
![]()
Вы уверены что такие данные есть? Я проверял по данным из UDB - там таких случаев нет. Если такие данные есть то я откачу, конечно, тем более у меня подтверждения для 3.x ytт креша - но клиент 1.12.1 явно крешится на таких данных.
|
![]() |
![]() |
![]() |
#5 |
YTDB dev
Регистрация: 05.11.2009
Сообщений: 5,920
Сказал(а) спасибо: 91
Поблагодарили 5,383 раз(а) в 4,847 сообщениях
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() |
![]()
Есть там предметы по кешу)):
Код:
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, '', '', '', ''); |
![]() |
![]() |
Пользователь сказал cпасибо: | Vladimir (18.03.2011) |
![]() |
#6 |
MaNGOS Dev
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
![]() ![]() |
![]()
f*ck. Я уже revert сделал
|
![]() |
![]() |
![]() |
#7 | ||
Брянский волк
|
![]()
Не надо торопиться.
![]() Цитата:
Цитата:
Сегодня прогнал скриптом 9408 квестов из кеша, но не встретил ни одного квеста, где бы встречались итемы в качестве вознаграждения, не идущие в полях подряд. В таком случае лучше вернуть коммит, т.к. он, как я понимаю, пойдет для 1.12.1 и мастер-ветки. ![]() |
||
![]() |
![]() |
![]() |
#8 |
MaNGOS Dev
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
![]() ![]() |
![]()
Ну для 1.x он просто необходим так как клиент крешится на такие данные, а для 3.x/2.x видимо верну/добавлю раз помогло поймать глюк в данных.
|
![]() |
![]() |
Пользователь сказал cпасибо: | virusav (18.03.2011) |
![]() |
#9 |
YTDB dev
Регистрация: 05.11.2009
Адрес: Thunder Bluff
Сообщений: 3,625
Сказал(а) спасибо: 7,839
Поблагодарили 3,255 раз(а) в 498 сообщениях
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() |
![]()
Может данные нужны по какомуто квесту, проверить?
335а, (401 убил, но на работе остался вроде) 403а, 406а? (фулл есть с нескольких клиентов) |
![]() |
![]() |
![]() |
#10 |
MaNGOS Dev
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
![]() ![]() |
![]()
Я откатил - т.е. передобавил проверку, так что если какие еще случаи будут это будет сообщенно ядром при загрузке.
|
![]() |
![]() |
![]() |
#11 | |
MaNGOS Dev
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
![]() ![]() |
![]() Цитата:
![]() |
|
![]() |
![]() |
![]() |
#12 |
YTDB dev
Регистрация: 05.11.2009
Адрес: Thunder Bluff
Сообщений: 3,625
Сказал(а) спасибо: 7,839
Поблагодарили 3,255 раз(а) в 498 сообщениях
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() |
![]() Код:
UPDATE `quest_template` SET `RewChoiceItemId1` = 20371, `RewChoiceItemId2` = 20651, `RewChoiceItemId3` = 22114, `RewChoiceItemId4` = 22780, `RewChoiceItemId5` = 33079 WHERE `entry` = 316; |
![]() |
![]() |
![]() |
|
|
![]() |
||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
[10590] Check if GO is requirement of quest to activate sparkle if it is | newsbot | CMaNGOS Commits | 0 | 08.10.2010 14:21 |
[10015] More redundant data check in `spell_proc_event` | newsbot | CMaNGOS Commits | 0 | 01.06.2010 04:58 |
[9334] Check if baseLevel for XP is valid before attempt use data from store | newsbot | CMaNGOS Commits | 0 | 09.02.2010 02:22 |
[9275] Quest reward data rearranged and comments added. | newsbot | CMaNGOS Commits | 0 | 31.01.2010 19:30 |
[9218] Fixed check in CMSG_QUEST_POI_QUERY proccessing for max quest amount. | newsbot | CMaNGOS Commits | 0 | 20.01.2010 07:50 |