Core/Script:
ICC: Convert possible non valid pointer to guid

Close #9954
Merge pull request #10875 from zorix/PR2

Core/Script:
DB/SAI: Zeth'Gor Must Burn!

Fixes #10904 by @untaught
Core/SAI: Using SMART_TARGET_NONE no longer works the same way as SMART_TARGET_ACTION_INVOKER but rather return NULL instead.

Fixes #10908
Merge branch 'master' of https://github.com/TrinityCore/TrinityCore
Scripts/SteamVault: reworked InstanceScript
Core/Logs: Random changes
Merge branch 'master' of github.com:TrinityCore/TrinityCore into 4.3.4

Conflicts:
	src/server/game/Achievements/AchievementMgr.cpp
DB/SAI: Non-repeatable flags change & credit circumvention

By dr-j fixes #10929 fixes #6444
Core/AI: Fix uninitialized values and array overflows in SAI

Proper initialize POD struct members to 0 in SMART_ACTION_CREATE_TIMED_EVENT action type.
Fix array overflows and access uninitialized values in SMART_ACTION_CALL_RANDOM_TIMED_ACTIONLIST, SMART_ACTION_RANDOM_PHASE and SMART_ACTION_RANDOM_EMOTE .

Valgrind log for SMART_ACTION_RANDOM_PHASE case:
 Conditional jump or move depends on uninitialised value(s)
   at : SmartScript::UpdateTimer(SmartScriptHolder&, unsigned int) (SmartScript.cpp:3086)
   by : SmartScript::OnUpdate(unsigned int) (SmartScript.cpp:3176)
   by : SmartAI::UpdateAI(unsigned int) (SmartAI.cpp:335)
   by : Creature::Update(unsigned int) (Creature.cpp:542)
   by : TempSummon::Update(unsigned int) (TemporarySummon.cpp:47)

 Uninitialised value was created by a stack allocation
   at : SmartScript::ProcessAction(SmartScriptHolder&, Unit*, unsigned int, unsigned int, bool, SpellInfo const*, GameObject*) (SmartScript.cpp:143)
Merge branch 'master' of github.com:TrinityCore/TrinityCore
Core/SAI: Get rid of unused parameter 'storageId' (third) in SMART_ACTION_SUMMON_CREATURE. This means the 4th parameter (actionInvoker 0/1) must move back one parameter. On a clean TDB there is not a single case of this parameter being used.

Updated wiki accordingly.
Merge pull request #10933 from MitchesD/SteamVault

Scripts/SteamVault: reworked InstanceScript
Core/Scripts: added some OVERRIDEs
DB/Update: Quest credit fix-up

Vehicles will not accept quest credit in the form of a spell cast, credit needs to be a monster kill.
DB/SAI: Wanton Warlord

Fixes #5126 by @dr-j
Core/Movement: Port Ascent r1637 http://sourceforge.net/p/ascentemubackup/code/1637/ to TrinityCore

Port some movement packet fixes, taking into account client/server connection lag.
Credits to original author, check http://sourceforge.net/p/ascentemubackup/code/1637/ for more details.
MOVEMENT_PACKET_TIME_DELAY default value is currently set to 0, feel free to set it to different values and report feedback.
DB/Core: The Storm King's Vengeance
Core/RBAC: Move RBAC code to namespace rbac
Core/RBAC: Add default groups to accounts based on their security level.
- Removed config option RBAC.DefaultGroups

Use the table rbac_security_level_groups to configure the groups to be added to the account at load time.

Note: Those groups are only used at run time, never saved to DB