Ru-MaNGOS  

Вернуться   Ru-MaNGOS > Ядро > Патчи > Отвергнутые патчи

Важная информация

Отвергнутые патчи Патчи, отвергнутые от приёма в GIT

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 21.04.2010, 20:34   #1
BLaD
Новичок
 
Регистрация: 21.04.2010
Сообщений: 3
Сказал(а) спасибо: 0
Поблагодарили 0 раз(а) в 0 сообщениях
BLaD На верном пути
По умолчанию Fix typo in [9765]

Должно предотвратить краши
Код:
diff --git a/src/game/Creature.cpp b/src/game/Creature.cpp
index ff7ea86..d5513fb 100644
--- a/src/game/Creature.cpp
+++ b/src/game/Creature.cpp
@@ -64,7 +64,7 @@ bool VendorItemData::RemoveItem( uint32 item_id )
         // can have many examples
         if((*i)->item == item_id)
         {
-            m_items.erase(i++);
+            m_items.erase(i);
             found = true;
         }
         else
BLaD вне форума   Ответить с цитированием
Старый 21.04.2010, 20:59   #2
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 440 раз(а) в 183 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Извините, но бред.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 21.04.2010, 21:29   #3
BLaD
Новичок
 
Регистрация: 21.04.2010
Сообщений: 3
Сказал(а) спасибо: 0
Поблагодарили 0 раз(а) в 0 сообщениях
BLaD На верном пути
По умолчанию

Да, ошибку увидел, просто с крашами при удалении последнего итема у вендора помогло...

Последний раз редактировалось BLaD; 21.04.2010 в 22:17.
BLaD вне форума   Ответить с цитированием
Старый 21.04.2010, 21:31   #4
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 440 раз(а) в 183 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Вроде это должно было быть исправлено как раз переходом к i++
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 21.04.2010, 21:43   #5
BLaD
Новичок
 
Регистрация: 21.04.2010
Сообщений: 3
Сказал(а) спасибо: 0
Поблагодарили 0 раз(а) в 0 сообщениях
BLaD На верном пути
По умолчанию

Версия сборки 9765 без патча при удалении итема серв падает...
Я бы хотел помочь, только возможности нету

Последний раз редактировалось BLaD; 21.04.2010 в 22:00.
BLaD вне форума   Ответить с цитированием
Старый 22.04.2010, 05:16   #6
zhenya
Пользователь
 
Регистрация: 12.03.2010
Сообщений: 85
Сказал(а) спасибо: 5
Поблагодарили 42 раз(а) в 17 сообщениях
zhenya Скоро придёт к известности
По умолчанию

не должно падать с i++.
zhenya вне форума   Ответить с цитированием
Старый 22.04.2010, 08:46   #7
Insider42
Ученый
 
Регистрация: 15.03.2010
Сообщений: 261
Сказал(а) спасибо: 84
Поблагодарили 257 раз(а) в 96 сообщениях
Insider42 Как самоцвет среди гранитаInsider42 Как самоцвет среди гранитаInsider42 Как самоцвет среди гранита
По умолчанию

без патча падает, подтверждаю
Код:
Revision: * * 9771 5efb5f0f9d07232a97477b21f4abb68b4eb458e5
Date 22:4:2010. Time 13:42 
//=====================================================
*** Hardware ***
Processor: Intel(R) Core(TM)2 Duo CPU     E4500  @ 2.20GHz
Number Of Processors: 2
Physical Memory: 2095396 KB (Available: 1050464 KB)
Commit Charge Limit: 3943460 KB

*** Operation System ***
Microsoft Windows Server 2003 Enterprise Edition Service Pack 2 (Version 5.2, Build 3790)

//=====================================================
Exception code: C0000005 ACCESS_VIOLATION
Fault address:  004BC11C 01:000BB11C C:\WoW Server\wow\wotlk_server\mangosd.exe

Registers:
EAX:00000001
EBX:F35F7A60
ECX:F47F4CC4
EDX:00000000
ESI:0AFEF840
EDI:008D9710
CS:EIP:0023:004BC11C
SS:ESP:002B:0AFE012C  EBP:0AFE0180
DS:002B  ES:002B  FS:0053  GS:002B
Flags:00010202

Call stack:
Address   Frame     Function      SourceFile
004BC11C  00000000  VendorItemData::RemoveItem+4C
005B9AAF  00000000  ObjectMgr::RemoveVendorItem+5F
008057BC  00000000  ChatHandler::HandleNpcDelVendorItemCommand+EC
005941AC  00000000  ChatHandler::ExecuteCommandInTable+2AC
00594018  00000000  ChatHandler::ExecuteCommandInTable+118
0059488B  00000000  ChatHandler::ParseCommands+18B
008A6C0B  00000000  WorldSession::HandleMessagechatOpcode+4FB
00627260  00000000  WorldSession::Update+120
00637693  00000000  World::UpdateSessions+C3
00635D15  00000000  World::Update+3E5
00455B6E  00000000  WorldRunnable::run+8E
008D9729  00000000  ACE_Based::Thread::ThreadTask+19
003214D4  00000000  ?invoke@ACE_OS_Thread_Adapter@@UAEKXZ+74
78543433  00000000  _endthreadex+44
785434C7  00000000  _endthreadex+D8
7D4DFE21  00000000  FlsSetValue+136

Call stack:
Address   Frame     Function      SourceFile
7D61C828  00000000  ZwWaitForSingleObject+15
7D4D8BF1  00000000  WaitForSingleObject+12

Call stack:
Address   Frame     Function      SourceFile
7D61CCA8  00000000  ZwDelayExecution+15
7D4D14EF  00000000  Sleep+F
008D9940  00000000  ACE_Based::Thread::Sleep+30
008DFDD0  00000000  SqlDelayThread::run+50
008D9729  00000000  ACE_Based::Thread::ThreadTask+19
003214D4  00000000  ?invoke@ACE_OS_Thread_Adapter@@UAEKXZ+74
78543433  00000000  _endthreadex+44
785434C7  00000000  _endthreadex+D8
7D4DFE21  00000000  FlsSetValue+136

Call stack:
Address   Frame     Function      SourceFile
7D61CCA8  00000000  ZwDelayExecution+15
7D4D14EF  00000000  Sleep+F
008D9940  00000000  ACE_Based::Thread::Sleep+30
008DFDD0  00000000  SqlDelayThread::run+50
008D9729  00000000  ACE_Based::Thread::ThreadTask+19
003214D4  00000000  ?invoke@ACE_OS_Thread_Adapter@@UAEKXZ+74
78543433  00000000  _endthreadex+44
785434C7  00000000  _endthreadex+D8
7D4DFE21  00000000  FlsSetValue+136

Call stack:
Address   Frame     Function      SourceFile
7D61CCA8  00000000  ZwDelayExecution+15
7D4D14EF  00000000  Sleep+F
008D9940  00000000  ACE_Based::Thread::Sleep+30
008DFDD0  00000000  SqlDelayThread::run+50
008D9729  00000000  ACE_Based::Thread::ThreadTask+19
003214D4  00000000  ?invoke@ACE_OS_Thread_Adapter@@UAEKXZ+74
78543433  00000000  _endthreadex+44
785434C7  00000000  _endthreadex+D8
7D4DFE21  00000000  FlsSetValue+136

Call stack:
Address   Frame     Function      SourceFile
7D61CCA8  00000000  ZwDelayExecution+15
7D4D14EF  00000000  Sleep+F
008D9940  00000000  ACE_Based::Thread::Sleep+30
008DFDD0  00000000  SqlDelayThread::run+50
008D9729  00000000  ACE_Based::Thread::ThreadTask+19
003214D4  00000000  ?invoke@ACE_OS_Thread_Adapter@@UAEKXZ+74
78543433  00000000  _endthreadex+44
785434C7  00000000  _endthreadex+D8
7D4DFE21  00000000  FlsSetValue+136

Call stack:
Address   Frame     Function      SourceFile
7D61D6C6  00000000  ZwGetContextThread+12
Воспроизвел мой гм на тестовом сервере, говорит что случилось при удалении последней вещи у вендора
Insider42 вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
Laise (22.04.2010)
Старый 22.04.2010, 10:02   #8
Laise
MaNGOS Dev
 
Аватар для Laise
 
Регистрация: 09.03.2010
Сообщений: 33
Сказал(а) спасибо: 27
Поблагодарили 26 раз(а) в 11 сообщениях
Laise На верном пути
По умолчанию

вроде с этим не должно падать:
Код:
diff --git a/src/game/Creature.cpp b/src/game/Creature.cpp
index 583cf59..ae4293a 100644
--- a/src/game/Creature.cpp
+++ b/src/game/Creature.cpp
@@ -64,7 +64,7 @@ bool VendorItemData::RemoveItem( uint32 item_id )
         // can have many examples
         if((*i)->item == item_id)
         {
-            m_items.erase(i++);
+            i = m_items.erase(i);
             found = true;
         }
         else
Laise вне форума   Ответить с цитированием
3 пользователя(ей) сказали cпасибо:
Den (22.04.2010), Vladimir (24.04.2010), zhenya (22.04.2010)
Старый 24.04.2010, 02:05   #9
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 440 раз(а) в 183 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Ах, уже добавил аналогичный в [9786] а только потом страницу обновил.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Ответ

Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
[9765] Prevent crash at use .npc delitem to item with multiply extanded costs. newsbot CMaNGOS Commits 0 19.04.2010 12:40


Текущее время: 22:26. Часовой пояс GMT +3.


ru-mangos.ru - Русское сообщество MaNGOS
Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума Ru-MaNGOS будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot