[Bug] Keyvalues/DataPacks memory leaks

[naydef]

Fix diff:

------------ addons/sourcemod/scripting/freak_fortress_2/weapons.sp ------------
index 6ef13a2b..1c9529bd 100755
@@ -306,6 +306,7 @@ void Weapons_ShowChanges(int client, int entity)
 		{
 			CPrintToChat(client, "%t%s:", "Prefix", localizedWeaponName);
 		}
+		delete kv;
 	}
 	else
 	#endif
@@ -649,6 +650,7 @@ stock void Weapons_OnHitBoss(int attacker, int newPlayerDamage, int lastPlayerDa
 					if(value)
 						SetEntProp(attacker, Prop_Data, "m_iAmmo", GetEntProp(attacker, Prop_Data, "m_iAmmo", _, 2) + value, _, 2);
 				}
+				delete kv;
 			}
 		}
 	}

[naydef]

Well, it turns out FF2R is also leaking DataPack handles.
It seems it's due to assuming that CreateDataTimer is not creating the DataPack itself.

Fix diff (ignore whitespace changes):

---------- addons/sourcemod/scripting/freak_fortress_2/attributes.sp ----------
index edc65800..8fa2cfd2 100755
@@ -88,7 +88,7 @@ bool Attributes_OnBackstabBoss(int attacker, int victim, float &damage, int weap
 	/*
 	if(Attributes_FindOnWeapon(attacker, weapon, 217))	// sanguisuge
 	{
-		
+
 		int maxoverheal = TF2U_GetMaxOverheal(attacker) * 2;	// 250% overheal (from 200% overheal)
 		int health = GetClientHealth(attacker);
 		if(health < maxoverheal)
@@ -105,7 +105,7 @@ bool Attributes_OnBackstabBoss(int attacker, int victim, float &damage, int weap
 			if(TF2_IsPlayerInCondition(attacker, TFCond_Plague))
 				TF2_RemoveCondition(attacker, TFCond_Plague);
 		}
-		
+
 	}
 	*/
 
@@ -311,7 +311,7 @@ void Attributes_OnHitBoss(int attacker, int victim, int inflictor, float fdamage
 
 	if(Attributes_FindOnPlayer(attacker, 418) > 0.0)	// boost on damage
 	{
-		DataPack pack = new DataPack();
+		DataPack pack;
 		if(Enabled)
 		{
 			CreateDataTimer(0.1, Attributes_BoostDrainStack, pack, TIMER_FLAG_NO_MAPCHANGE|TIMER_REPEAT);

----------- addons/sourcemod/scripting/freak_fortress_2/commands.sp -----------
index 614dc0eb..4f7a3df2 100755
@@ -275,7 +275,7 @@ static Action SwapTeam(int client, int wantTeam)
 
 		if(Cvar[AggressiveSwap].BoolValue)
 		{
-			DataPack pack = new DataPack();
+			DataPack pack;
 			CreateDataTimer(0.2, Command_AggressiveSwap, pack, TIMER_FLAG_NO_MAPCHANGE|TIMER_REPEAT);
 			pack.WriteCell(GetClientUserId(client));
 			pack.WriteCell(newTeam);

[naydef]

The second type of memory leaks (DataPack one) hasn't been taken care of.