PVS-Studio Analysis Results


Date:Sun Oct 17 09:52:31 2021
PVS-Studio Version:7.15.53134.163
Command Line:plog-converter -a GA\:1,2 -t fullhtml -d V1042,V576,V728,V768,V1037 -o /var/www/tcubuntu.northeurope.cloudapp.azure.com/html/pvsstudio/analysis PVS-Studio.log
Total Warnings (GA):62
Group Location Level Code Message
General Analysis WorldModel.h:73 Medium V690 The 'GroupModel' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class.
General Analysis Util.cpp:500 High V557 Array overrun is possible. The 'declension' index is pointing beyond array bound.
General Analysis Creature.cpp:909 Medium V636 The 'maxValue / 3' expression was implicitly cast from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;.
General Analysis Creature.cpp:920 Medium V636 The expression was implicitly cast from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;.
General Analysis Player.h:1112 Medium V1071 Consider inspecting the 'StoreNewItem' function. The return value is not always used. Total calls: 10, discarded results: 1.
General Analysis CharacterHandler.cpp:995 Medium V681 The language standard does not define an order in which the 'GetXPForNextLevel' functions will be called during evaluation of arguments.
General Analysis CharacterHandler.cpp:1536 High V557 Array overrun is possible. The 'i' index is pointing beyond array bound.
General Analysis PetHandler.cpp:803 Medium V1002 The 'SpellCastTargets' class, containing pointers, constructor and destructor, is copied by the automatically generated operator=.
General Analysis TradeHandler.cpp:384 Medium V1002 The 'SpellCastTargets' class, containing pointers, constructor and destructor, is copied by the automatically generated operator=.
General Analysis TradeHandler.cpp:419 Medium V1002 The 'SpellCastTargets' class, containing pointers, constructor and destructor, is copied by the automatically generated operator=.
General Analysis LootMgr.h:73 Medium V1071 Consider inspecting the 'LoadAndCollectLootIds' function. The return value is not always used. Total calls: 12, discarded results: 1.
General Analysis WorldSession.cpp:596 Medium V681 The language standard does not define an order in which the 'GetXPForNextLevel' functions will be called during evaluation of arguments.
General Analysis boss_chromaggus.cpp:176 High V1053 Calling the 'EnterEvadeMode' virtual function in the constructor may lead to unexpected result at runtime.
General Analysis npc_arthas.cpp:527 High V557 Array overrun is possible. The 'ARTHAS_PURGE_PENDING_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:528 High V557 Array overrun is possible. The 'ARTHAS_PURGE_PENDING_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:529 High V557 Array overrun is possible. The 'ARTHAS_WAVES_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:530 High V557 Array overrun is possible. The 'ARTHAS_WAVES_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:531 High V557 Array overrun is possible. The 'ARTHAS_TOWN_HALL_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:532 High V557 Array overrun is possible. The 'ARTHAS_TOWN_HALL_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:533 High V557 Array overrun is possible. The 'ARTHAS_TOWN_HALL_END_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:534 High V557 Array overrun is possible. The 'ARTHAS_TOWN_HALL_END_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:535 High V557 Array overrun is possible. The 'ARTHAS_GAUNTLET_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:536 High V557 Array overrun is possible. The 'ARTHAS_GAUNTLET_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:537 High V557 Array overrun is possible. The 'ARTHAS_GAUNTLET_END_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:538 High V557 Array overrun is possible. The 'ARTHAS_GAUNTLET_END_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:539 High V557 Array overrun is possible. The 'ARTHAS_FINAL_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:595 High V557 Array overrun is possible. The 'RP1_UTHER_SPAWN' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:600 High V557 Array overrun is possible. The 'RP1_JAINA_SPAWN' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:698 High V557 Array overrun is possible. The 'RP5_MALGANIS_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:868 High V557 Array overrun is possible. The 'ARTHAS_FINAL_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1050 High V557 Array overrun is possible. The 'ARTHAS_PURGE_PENDING_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1122 High V557 Array overrun is possible. The 'RP2_MALGANIS_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1236 High V557 Array overrun is possible. The 'RP3_SPAWN1_LOC1' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1238 High V557 Array overrun is possible. The 'RP3_SPAWN1_LOC2' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1240 High V557 Array overrun is possible. The 'RP3_SPAWN1_LOC3' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1242 High V557 Array overrun is possible. The 'RP3_SPAWN1_LOC4' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1244 High V557 Array overrun is possible. The 'RP3_SPAWN1_RIFT' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1277 High V557 Array overrun is possible. The 'RP3_SPAWN2_LOC1' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1279 High V557 Array overrun is possible. The 'RP3_SPAWN2_LOC2' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1281 High V557 Array overrun is possible. The 'RP3_SPAWN2_LOC3' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1283 High V557 Array overrun is possible. The 'RP3_SPAWN2_LOC4' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1285 High V557 Array overrun is possible. The 'RP3_SPAWN2_RIFT1' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1287 High V557 Array overrun is possible. The 'RP3_SPAWN2_RIFT2' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1307 High V557 Array overrun is possible. The 'RP3_SPAWN3_LOC1' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1309 High V557 Array overrun is possible. The 'RP3_SPAWN3_LOC2' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1311 High V557 Array overrun is possible. The 'RP3_SPAWN3_LOC3' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1313 High V557 Array overrun is possible. The 'RP3_SPAWN3_LOC4' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1315 High V557 Array overrun is possible. The 'RP3_SPAWN3_RIFT1' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1317 High V557 Array overrun is possible. The 'RP3_SPAWN3_RIFT2' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1329 High V557 Array overrun is possible. The 'RP3_EPOCH_SPAWN' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1331 High V557 Array overrun is possible. The 'RP3_EPOCH_RIFT' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1394 High V557 Array overrun is possible. The 'ARTHAS_GAUNTLET_END_POS' index is pointing beyond array bound.
General Analysis npc_arthas.cpp:1466 High V557 Array overrun is possible. The 'RP5_CHROMIE_SPAWN' index is pointing beyond array bound.
General Analysis Map.h:502 Medium V1071 Consider inspecting the 'SummonCreature' function. The return value is not always used. Total calls: 22, discarded results: 1.
General Analysis instance_dire_maul.cpp:79 High V730 Not all members of a class are initialized inside the constructor. Consider inspecting: _crystalCreatureGUIDs.
General Analysis boss_sapphiron.cpp:488 Medium V1004 The 'me' pointer was used unsafely after it was verified against nullptr. Check lines: 486, 488.
General Analysis boss_lady_vashj.cpp:736 High V1053 Calling the 'EnterEvadeMode' virtual function in the constructor may lead to unexpected result at runtime.
General Analysis spell_pet.cpp:1518 Medium V636 The expression was implicitly cast from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;.
General Analysis ByteBuffer.cpp:58 Medium V1067 Throwing from an exception constructor may lead to unexpected behavior. Consider to check function call.
General Analysis dbcfile.h:52 Medium V690 The copy assignment operator is declared as private in the 'Record' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class.
General Analysis dbcfile.h:58 Medium V690 The 'Record' class implements the copy assignment operator, but lacks a copy constructor. It is dangerous to use such a class.
General Analysis vmapexport.cpp:363 Medium V557 Array overrun is possible. The value of 'strlen(input_path) - 1' index could reach 18446744073709551615.