Core/Misc: Fix some issues detected by static analysis

* Assignment of function parameter has no effect outside the function. Did you forget dereferencing it?
  -> This argument is passed by value, it makes no sence so set it to null here.

* Member variable 'instance_violet_hold_InstanceMapScript::bWiped' is not initialized in the constructor.

* Mismatching allocation and deallocation: Data

src/server/bnetserver/Server/ModuleManager.h

@@ -54,9 +54,10 @@ namespace Battlenet
                 memcpy(Data, right.Data, DataSize);
             }
         }
+
         ~ModuleInfo()
         {
-            delete Data;
+            delete[] Data;
         }
 
         std::string Type;

src/server/game/AuctionHouse/AuctionHouseMgr.cpp

@@ -440,8 +440,6 @@ bool AuctionHouseObject::RemoveAuction(AuctionEntry* auction)
 
     // we need to delete the entry, it is not referenced any more
     delete auction;
-    auction = NULL;
-
     return wasInMap;
 }
 

src/server/scripts/Northrend/VioletHold/instance_violet_hold.cpp

@@ -133,6 +133,7 @@ public:
             uiCyanigosaEventTimer = 3 * IN_MILLISECONDS;
 
             bActive = false;
+            bWiped = false;
             bIsDoorSpellCast = false;
             bCrystalActivated = false;
             defenseless = true;