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
Merge branch 'master' of github.com:TrinityCore/TrinityCore into 4.3.4