| Age | Commit message (Collapse) | Author |
|
|
|
Fix uninitialized values in Ulduar script reported by Valgrind.
|
|
Conflicts:
sql/old/3.3.5a/TDB52_to_TDB53_updates/world/2013_07_24_00_world_spell_script_names.sql
sql/updates/world/2013_07_24_00_world_spell_script_names.sql
sql/updates/world/2013_07_24_00_world_spell_script_names_335.sql
src/server/game/Achievements/AchievementMgr.cpp
src/server/game/Achievements/AchievementMgr.h
src/server/game/Battlegrounds/ArenaTeamMgr.cpp
src/server/game/Chat/ChatLink.cpp
src/server/game/DataStores/DBCStores.cpp
src/server/game/DataStores/DBCStructure.h
src/server/game/DataStores/DBCfmt.h
src/server/game/Entities/DynamicObject/DynamicObject.h
src/server/game/Entities/Object/Object.cpp
src/server/game/Entities/Player/Player.cpp
src/server/game/Entities/Unit/StatSystem.cpp
src/server/game/Entities/Unit/Unit.cpp
src/server/game/Entities/Unit/Unit.h
src/server/game/Groups/Group.cpp
src/server/game/Handlers/QuestHandler.cpp
src/server/game/Miscellaneous/SharedDefines.h
src/server/game/Spells/Auras/SpellAuraEffects.cpp
src/server/game/Spells/Auras/SpellAuras.cpp
src/server/game/World/World.h
src/server/scripts/Commands/cs_character.cpp
src/server/scripts/Commands/cs_lookup.cpp
src/server/scripts/Commands/cs_titles.cpp
src/server/scripts/EasternKingdoms/ZulAman/boss_halazzi.cpp
src/server/scripts/EasternKingdoms/ZulAman/boss_zuljin.cpp
src/server/scripts/EasternKingdoms/ZulGurub/boss_jeklik.cpp
src/server/scripts/Kalimdor/zone_darkshore.cpp
src/server/scripts/Kalimdor/zone_mulgore.cpp
src/server/scripts/Kalimdor/zone_tanaris.cpp
src/server/scripts/Northrend/IcecrownCitadel/boss_the_lich_king.cpp
src/server/scripts/Spells/spell_dk.cpp
src/server/scripts/Spells/spell_mage.cpp
src/server/scripts/Spells/spell_rogue.cpp
src/server/shared/Database/Implementation/CharacterDatabase.cpp
src/server/shared/Database/Implementation/CharacterDatabase.h
src/tools/vmap4_extractor/mpq_libmpq04.h
|
|
|
|
Fix Position members not being always initialized.
Valgrind log:
at : Position::NormalizeOrientation(float) (Object.h:388)
by : Position::SetOrientation(float) (Object.h:315)
by : Position::Relocate(Position const&) (Object.h:310)
by : spell_ulduar_stone_grip::spell_ulduar_stone_grip_AuraScript::OnRemoveVehicle(AuraEffect const*, AuraEffectHandleModes) (boss_kologarn.cpp:592)
|
|
|
|
Add some asserts and additional NULL checks as sanity checks.
|
|
|
|
Fix uninitialized values, most of which are false positives, always initialized before being accessed.
Add some asserts and additional NULL checks as sanity checks. Use SpellMgr::EnsureSpellInfo() if the spell id is valid and always supposed to return a valid not-NULL SpellInfo* .
|
|
|
|
Remove hack SAI, use aura script
|
|
Gunship Battle
Closes #11786
|
|
* Deathbringer Saurfang will now evade if any player stands on the gunship
* Fixed players being stuck in combat after defeating gunship
* Fixed players not sticking to gunships
|
|
|
|
|
|
|
|
players logging in
|
|
Fix some static analysis issues about:
- uninitialized values, most of which are false positives, always initialized before being accessed
- unchecked return values
- dead code never executed
- bad formatting leading to wrong behavior
Please ensure EventMap is never used with event id set to 0 or those events will never execute.
|
|
|
|
Add some asserts and additional NULL checks as sanity checks.
|
|
Core/Scripts: Move 'The Defias Traitor' script to SAI
|
|
Core/Scripts: Move condition for 'npc_marin_noggenfogger' gossip menu op...
|
|
Core/Scripts: Move 'npc_deathly_usher' scripts to SAI.
|
|
Core/Scripts: Move condition for 'npc_skorn_whitecloud' gossip menu opti...
|
|
|
|
|
|
to DB and remove 'npc_marin_noggenfogger' script
|
|
to DB and remove 'npc_skorn_whitecloud' script
|
|
|
|
|
|
Instance Script/Stratholme: Correct logic in Baron Run event and add SAI...
|
|
Fix some static analysis issues about uninitialized values. Most of them are false positives, always initialized before being accessed.
|
|
Ysida Harmon
|
|
triggered
Fix EVENT_ANUBAR_CHARGE not being triggered due to value 0 being set to the EVENT_ANUBAR_CHARGE enum, even if 0 is a special value for EventMap to specify no event was triggered.
Issue added in a127c0e1a7612094c2bf8b2f5092f55aa96f0556
|
|
Initialize all class fields in the constructor, even if they are set in Reset() and they shouldn't be accessed before Reset() is called.
This fixes 2 Coverity reports.
|
|
|
|
Apply safe NULL checks added in 7d881a073961887d446d4430905f315b52928bb1 to similar code in same script
|
|
Fix a NULL dereference exception happening when using .kick command caused by a strtok(NULL) call without a previous call to the string to be tokenized.
Issue added in 101cad1f2872a87be925b74b9d17760381bb9808
|
|
Good Help is Hard to Find (14069) quest script
|
|
character thx @Expecto
- also rename some prepared statements
Closes #11658
|
|
|
|
some achieves).
|
|
- fixed arena season title achievements and rewards
- added mailtemplate support for `achievement_reward` table
- fixed typo in GetBattleMasterBG which causes achievement reward mail expire in 1 day
Ref #8777
|
|
|
|
|
|
Scripts/Krikthir the Gatewatcher: converted to EventMap and small optimalization
|
|
|
|
Fix npc id 17209 in Eastern Plaguelands enabling taxi cheat to players just by talking to it.
|
|
take or return quests
Closes #11269
Closes #8898
Closes #10204
Closes #11410
Closes #7053
Closes #6189
Closes #9474
|
|
|