Core/Spell: Fix Blood Tap array overflow

Fix Blood Tap spellid 45529 accessing the 7th element of a size 6 array.
Coverity issue id 1023095.
Merge pull request #10821 from jackpoz/blood_tap_overflow

Core/Spell: Fix Blood Tap array overflow
Core/Misc: Fixed some more issues found by static code analysis tools.
Merge pull request #10819 from Ascathor/master

Core/Misc: Fixed some more issues found by static code analysis tools.
Tools/VMAP: Fix not null terminated string

Fix not null terminated string in vmap assembler tool, fix coverity issue id 1010351 .
Scripts/HoR: Fix possible array overflow

Fix possible array overflow reported by coverity, issue id 1062436 .
Core/BGs: Fix a TC_LOG_ERROR call
Merge pull request #10823 from jackpoz/icewall_overflow

Scripts/HoR: Fix possible array overflow
Merge pull request #10822 from jackpoz/collisions_nullterm_string

Tools/VMAP: Fix not null terminated string
Shared/Build: Restore disabled warnings on Visual Studio

Restore 2 disabled warnings, 1 already disabled in CMake with the WITH_WARNINGS flag set to False and the other disabled by default as stated in VS documentation http://msdn.microsoft.com/en-us/library/aa984150.aspx
Merge branch 'master' of github.com:TrinityCore/TrinityCore into 4.3.4
DB/SAI: Fate of the Titans
DB/SAI: So Far, So Bad
DB/SAI: The Murketh and Shaadraz Gateways.
DB/Error: Delete empty guid addon
DB/SAI: Misc Fixes 

Authors @untaught @dr-j @MitchesD
Fixes #10847 Fixes #10844 Fixes #10830 Fixes #10829 Fixes #10830
DB/SAI: Missing addition
Core/Auras:
Fix removing auras when player reset talents or changing/leaving shapeshift

Close #10763
Close #10809
Merge pull request #10864 from zorix/auras

Core/Auras: Fix removing auras when player reset talents or changing/leaving shapeshift

Close #10763
Close #10809
Core/Achievements: Implemented ACHIEVEMENT_CRITERIA_TYPE_KILL_CREATURE_TYPE