[fixed] краш в mail.cpp - Ru-MaNGOS

rsa · 06.12.2010, 22:41

В самых последних ревизиях (зафиксировано на 10828, искать коммит было лень) вылез краш при попытке отправки письма из консоли.

Код:

#0  MailSender (this=0xad13e144, sender=0x0, stationery=MAIL_STATIONERY_GM)
    at ../../../src/game/Mail.cpp:808
808         switch(sender->GetTypeId())

решение:
https://github.com/rsa/mangos/commit...817491af029abd

**Vladimir** · 07.12.2010, 02:14

Я лучше поправлю вызовы в коммандах - не думаю что сильно хорошо когда любые NULL из-за багов в коде будет интерпретироваться как GM-письма. Потом будет невозможно нормально отловить настоящий баг.

**Vladimir** · 07.12.2010, 03:34

В [10831] поправил.

rsa · 07.12.2010, 07:15

Не как ГМ-письма, а как письма от неизвестного адресата. Впрочем принятый вариант лучше. Спасибо.

06.12.2010, 22:41	#1
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	[fixed] краш в mail.cpp В самых последних ревизиях (зафиксировано на 10828, искать коммит было лень) вылез краш при попытке отправки письма из консоли. Код: #0 MailSender (this=0xad13e144, sender=0x0, stationery=MAIL_STATIONERY_GM) at ../../../src/game/Mail.cpp:808 808 switch(sender->GetTypeId()) решение: https://github.com/rsa/mangos/commit...817491af029abd

07.12.2010, 02:14	#2
Vladimir MaNGOS Dev Регистрация: 09.02.2010 Сообщений: 594 Сказал(а) спасибо: 315 Поблагодарили 438 раз(а) в 181 сообщениях	Я лучше поправлю вызовы в коммандах - не думаю что сильно хорошо когда любые NULL из-за багов в коде будет интерпретироваться как GM-письма. Потом будет невозможно нормально отловить настоящий баг. __________________ Так как устал объяснять знайте ICQ не пользуюсь

07.12.2010, 03:34	#3
Vladimir MaNGOS Dev Регистрация: 09.02.2010 Сообщений: 594 Сказал(а) спасибо: 315 Поблагодарили 438 раз(а) в 181 сообщениях	В [10831] поправил. __________________ Так как устал объяснять знайте ICQ не пользуюсь

Похожие темы
Тема	Автор	Раздел	Ответов	Последнее сообщение
[10972] Implement mass mail gm commands.	newsbot	CMaNGOS Commits	0	06.01.2011 06:23
[10862] Fixed not show and lost mail items in returned expire mails.	newsbot	CMaNGOS Commits	0	12.12.2010 09:00
[10831] Fixed crash at send mail from console	newsbot	CMaNGOS Commits	0	07.12.2010 03:40
[10802] Use ObjectGuid in Mail code.	newsbot	CMaNGOS Commits	0	01.12.2010 14:21
e-mail=username = загрузка обновления	Dragon	Баг-репорты	5	05.11.2010 20:00

07.12.2010, 07:15	#4
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	Не как ГМ-письма, а как письма от неизвестного адресата. Впрочем принятый вариант лучше. Спасибо.