1 files changed, 153 insertions, 84 deletions

diff --git a/src/map/trade.c b/src/map/trade.c
index 6ab75651a..b9a22a79a 100644
--- a/src/map/trade.c
+++ b/src/map/trade.c
@@ -1,3 +1,6 @@
+// $Id: trade.c 375 2005-03-04 21:33:31Z Yor $
+//#include <config.h>
+
 #include <stdio.h>
 #include <string.h>
 
@@ -8,45 +11,43 @@
 #include "pc.h"
 #include "npc.h"
 #include "battle.h"
-#include "nullpo.h"
-#include "log.h"
 #include "chrif.h"
+#include "intif.h"
+#include "atcommand.h"
+
+#include "nullpo.h"
 
 /*==========================================
  * 取引要請を相手に送る
  *------------------------------------------
  */
-void trade_traderequest(struct map_session_data *sd,int target_id)
-{
+void trade_traderequest(struct map_session_data *sd, int target_id) {
 	struct map_session_data *target_sd;
 
 	nullpo_retv(sd);
 
-	if((target_sd = map_id2sd(target_id)) != NULL){
-		if(!battle_config.invite_request_check) {
-			if(target_sd->guild_invite>0 || target_sd->party_invite>0){
-				clif_tradestart(sd,2);	// 相手はPT要請中かGuild要請中
+	if ((target_sd = map_id2sd(target_id)) != NULL) {
+		if (!battle_config.invite_request_check) {
+			if (target_sd->guild_invite > 0 || target_sd->party_invite > 0) {
+				clif_tradestart(sd, 2); // 相手はPT要請中かGuild要請中
 				return;
 			}
 		}
-		if((target_sd->trade_partner !=0) || (sd->trade_partner !=0)) {
-			trade_tradecancel(sd); //person is in another trade
-		}
-		else{
-			if((pc_isGM(sd) < 60) && (sd->bl.m != target_sd->bl.m
-			 || (sd->bl.x - target_sd->bl.x <= -5 || sd->bl.x - target_sd->bl.x >= 5)
-			 || (sd->bl.y - target_sd->bl.y <= -5 || sd->bl.y - target_sd->bl.y >= 5))) {
-				clif_tradestart(sd,0); //too far
-			}
-			else if(sd!=target_sd) {
+		if ((target_sd->trade_partner != 0) || (sd->trade_partner != 0)) {
+			trade_tradecancel(sd); // person is in another trade
+		} else {
+			if (sd->bl.m != target_sd->bl.m ||
+			    (sd->bl.x - target_sd->bl.x <= -5 || sd->bl.x - target_sd->bl.x >= 5) ||
+			    (sd->bl.y - target_sd->bl.y <= -5 || sd->bl.y - target_sd->bl.y >= 5)) {
+				clif_tradestart(sd, 0); // too far
+			} else if (sd != target_sd) {
 				target_sd->trade_partner = sd->status.account_id;
 				sd->trade_partner = target_sd->status.account_id;
-				clif_traderequest(target_sd,sd->status.name);
+				clif_traderequest(target_sd, sd->status.name);
 			}
 		}
-	}
-	else{
-		clif_tradestart(sd,1); //character does not exist
+	} else {
+		clif_tradestart(sd, 1); // character does not exist
 	}
 }
 
@@ -54,34 +55,94 @@ void trade_traderequest(struct map_session_data *sd,int target_id)
  * 取引要請
  *------------------------------------------
  */
-void trade_tradeack(struct map_session_data *sd,int type)
-{
+void trade_tradeack(struct map_session_data *sd, int type) {
 	struct map_session_data *target_sd;
 
 	nullpo_retv(sd);
 
-	if((target_sd = map_id2sd(sd->trade_partner)) != NULL){
-		clif_tradestart(target_sd,type);
-		clif_tradestart(sd,type);
-		if(type == 4){ // Cancel
-			sd->deal_locked =0;
-			sd->trade_partner=0;
-			target_sd->deal_locked=0;
-			target_sd->trade_partner=0;
+	if ((target_sd = map_id2sd(sd->trade_partner)) != NULL) {
+		clif_tradestart(target_sd, type);
+		clif_tradestart(sd, type);
+		if (type == 4) { // Cancel
+			sd->deal_locked = 0;
+			sd->trade_partner = 0;
+			target_sd->deal_locked = 0;
+			target_sd->trade_partner = 0;
 		}
-		if(sd->npc_id != 0)
+		if (sd->npc_id != 0)
 			npc_event_dequeue(sd);
-		if(target_sd->npc_id != 0)
+		if (target_sd->npc_id != 0)
 			npc_event_dequeue(target_sd);
 	}
 }
 
 /*==========================================
+ * Check here hacker for duplicate item in trade
+ * normal client refuse to have 2 same types of item (except equipment) in same trade window
+ * normal client authorise only no equiped item and only from inventory
+ *------------------------------------------
+ */
+int impossible_trade_check(struct map_session_data *sd) {
+	struct item inventory[MAX_INVENTORY];
+	char message_to_gm[200];
+	int i, index;
+
+	nullpo_retr(0, sd);
+
+	// get inventory of player
+	memcpy(&inventory, &sd->status.inventory, sizeof(struct item) * MAX_INVENTORY);
+
+/* remove this part: arrows can be trade and equiped
+	// remove equiped items (they can not be trade)
+	for (i = 0; i < MAX_INVENTORY; i++)
+		if (inventory[i].nameid > 0 && inventory[i].equip)
+			memset(&inventory[i], 0, sizeof(struct item));
+*/
+
+	// check items in player inventory
+	for(i = 0; i < 10; i++)
+		if (sd->deal_item_amount[i] < 0) { // negativ? -> hack
+//			printf("Negativ amount in trade, by hack!\n"); // normal client send cancel when we type negativ amount
+			return -1;
+		} else if (sd->deal_item_amount[i] > 0) {
+			index = sd->deal_item_index[i] - 2;
+			inventory[index].amount -= sd->deal_item_amount[i]; // remove item from inventory
+//			printf("%d items left\n", inventory[index].amount);
+			if (inventory[index].amount < 0) { // if more than the player have -> hack
+//				printf("A player try to trade more items that he has: hack!\n");
+				sprintf(message_to_gm, msg_txt(538), sd->status.name, sd->status.account_id); // Hack on trade: character '%s' (account: %d) try to trade more items that he has.
+				intif_wis_message_to_gm(wisp_server_name, battle_config.hack_info_GM_level, message_to_gm);
+				sprintf(message_to_gm, msg_txt(539), sd->status.inventory[index].amount, sd->status.inventory[index].nameid, sd->status.inventory[index].amount - inventory[index].amount); // This player has %d of a kind of item (id: %d), and try to trade %d of them.
+				intif_wis_message_to_gm(wisp_server_name, battle_config.hack_info_GM_level, message_to_gm);
+				// if we block people
+				if (battle_config.ban_hack_trade < 0) {
+					chrif_char_ask_name(-1, sd->status.name, 1, 0, 0, 0, 0, 0, 0); // type: 1 - block
+					clif_setwaitclose(sd->fd); // forced to disconnect because of the hack
+					// message about the ban
+					sprintf(message_to_gm, msg_txt(540), battle_config.ban_spoof_namer); //  This player has been definitivly blocked.
+				// if we ban people
+				} else if (battle_config.ban_hack_trade > 0) {
+					chrif_char_ask_name(-1, sd->status.name, 2, 0, 0, 0, 0, battle_config.ban_hack_trade, 0); // type: 2 - ban (year, month, day, hour, minute, second)
+					clif_setwaitclose(sd->fd); // forced to disconnect because of the hack
+					// message about the ban
+					sprintf(message_to_gm, msg_txt(507), battle_config.ban_spoof_namer); //  This player has been banned for %d minute(s).
+				} else {
+					// message about the ban
+					sprintf(message_to_gm, msg_txt(508)); //  This player hasn't been banned (Ban option is disabled).
+				}
+				intif_wis_message_to_gm(wisp_server_name, battle_config.hack_info_GM_level, message_to_gm);
+				return 1;
+			}
+		}
+
+	return 0;
+}
+
+/*==========================================
  * アイテム追加
  *------------------------------------------
  */
-void trade_tradeadditem(struct map_session_data *sd, int index, int amount)
-{
+void trade_tradeadditem(struct map_session_data *sd, int index, int amount) {
 	struct map_session_data *target_sd;
 	int trade_i;
 	int trade_weight = 0;
@@ -90,7 +151,7 @@ void trade_tradeadditem(struct map_session_data *sd, int index, int amount)
 	nullpo_retv(sd);
 
 	if (((target_sd = map_id2sd(sd->trade_partner)) != NULL) && (sd->deal_locked < 1)){
-		if (index < 2 || index >= MAX_INVENTORY + 2){
+		if (index < 2 || index >= MAX_INVENTORY + 2) {
 			if (index == 0) {
 				if (amount > 0 && amount <= MAX_ZENY && amount <= sd->status.zeny && // check amount
 				    (target_sd->status.zeny + amount) <= MAX_ZENY) { // fix positiv overflow
@@ -109,7 +170,7 @@ void trade_tradeadditem(struct map_session_data *sd, int index, int amount)
 					trade_weight += sd->inventory_data[index-2]->weight * amount;
 					if (target_sd->weight + trade_weight > target_sd->max_weight){
 						clif_tradeitemok(sd, index, 1); // fail to add item -- the player was over weighted.
-						amount = 0; // [MouseJstr]
+						amount = 0;
 					} else {
 						for(c = 0; c == trade_i - 1; c++) { // re-deal exploit protection [Valaris]
 							if (sd->deal_item_index[c] == index) {
@@ -119,6 +180,10 @@ void trade_tradeadditem(struct map_session_data *sd, int index, int amount)
 						}
 						sd->deal_item_index[trade_i] = index;
 						sd->deal_item_amount[trade_i] += amount;
+						if (impossible_trade_check(sd)) { // check exploit (trade more items that you have)
+							trade_tradecancel(sd);
+							return;
+						}
 						clif_tradeitemok(sd, index, 0); // success to add item
 						clif_tradeadditem(sd, target_sd, index, amount);
 					}
@@ -135,21 +200,25 @@ void trade_tradeadditem(struct map_session_data *sd, int index, int amount)
  * アイテム追加完了(ok押し)
  *------------------------------------------
  */
-void trade_tradeok(struct map_session_data *sd)
-{
+void trade_tradeok(struct map_session_data *sd) {
 	struct map_session_data *target_sd;
 	int trade_i;
 
 	nullpo_retv(sd);
 
 	// check items
-	for(trade_i=0;trade_i<10;trade_i++) {
-		int idx = sd->deal_item_index[trade_i]-2;
-		if(((idx >= 0) && (sd->deal_item_amount[trade_i]>sd->status.inventory[idx].amount)) || sd->deal_item_amount[trade_i]<0) {
+	for(trade_i = 0; trade_i < 10; trade_i++) {
+		if (sd->deal_item_amount[trade_i] > sd->status.inventory[sd->deal_item_index[trade_i]-2].amount ||
+		    sd->deal_item_amount[trade_i] < 0) {
 			trade_tradecancel(sd);
 			return;
 		}
-	
+	}
+
+	// check exploit (trade more items that you have)
+	if (impossible_trade_check(sd)) {
+		trade_tradecancel(sd);
+		return;
 	}
 
 	// check zeny
@@ -157,12 +226,12 @@ void trade_tradeok(struct map_session_data *sd)
 		trade_tradecancel(sd);
 		return;
 	}
-	
-	if((target_sd = map_id2sd(sd->trade_partner)) != NULL){
-		sd->deal_locked=1;
-		clif_tradeitemok(sd,0,0);
-		clif_tradedeal_lock(sd,0);
-		clif_tradedeal_lock(target_sd,1);
+
+	if ((target_sd = map_id2sd(sd->trade_partner)) != NULL) {
+		sd->deal_locked = 1;
+		clif_tradeitemok(sd, 0, 0);
+		clif_tradedeal_lock(sd, 0);
+		clif_tradedeal_lock(target_sd, 1);
 	}
 }
 
@@ -170,38 +239,37 @@ void trade_tradeok(struct map_session_data *sd)
  * 取引キャンセル
  *------------------------------------------
  */
-void trade_tradecancel(struct map_session_data *sd)
-{
+void trade_tradecancel(struct map_session_data *sd) {
 	struct map_session_data *target_sd;
 	int trade_i;
 
 	nullpo_retv(sd);
 
-	if((target_sd = map_id2sd(sd->trade_partner)) != NULL){
-		for(trade_i=0; trade_i<10;trade_i++) { //give items back (only virtual)
-			if(sd->deal_item_amount[trade_i] != 0) {
-				clif_additem(sd,sd->deal_item_index[trade_i]-2,sd->deal_item_amount[trade_i],0);
-				sd->deal_item_index[trade_i] =0;
-				sd->deal_item_amount[trade_i]=0;
+	if ((target_sd = map_id2sd(sd->trade_partner)) != NULL) {
+		for(trade_i = 0; trade_i < 10; trade_i++) { // give items back (only virtual)
+			if (sd->deal_item_amount[trade_i] != 0) {
+				clif_additem(sd, sd->deal_item_index[trade_i] - 2, sd->deal_item_amount[trade_i], 0);
+				sd->deal_item_index[trade_i] = 0;
+				sd->deal_item_amount[trade_i] = 0;
 			}
-			if(target_sd->deal_item_amount[trade_i] != 0) {
-				clif_additem(target_sd,target_sd->deal_item_index[trade_i]-2,target_sd->deal_item_amount[trade_i],0);
-				target_sd->deal_item_index[trade_i] =0;
-				target_sd->deal_item_amount[trade_i]=0;
+			if (target_sd->deal_item_amount[trade_i] != 0) {
+				clif_additem(target_sd, target_sd->deal_item_index[trade_i] - 2, target_sd->deal_item_amount[trade_i], 0);
+				target_sd->deal_item_index[trade_i] = 0;
+				target_sd->deal_item_amount[trade_i] = 0;
 			}
 		}
-		if(sd->deal_zeny) {
-			clif_updatestatus(sd,SP_ZENY);
-			sd->deal_zeny=0;
+		if (sd->deal_zeny) {
+			clif_updatestatus(sd, SP_ZENY);
+			sd->deal_zeny = 0;
 		}
-		if(target_sd->deal_zeny) {
-			clif_updatestatus(target_sd,SP_ZENY);
-			target_sd->deal_zeny=0;
+		if (target_sd->deal_zeny) {
+			clif_updatestatus(target_sd, SP_ZENY);
+			target_sd->deal_zeny = 0;
 		}
-		sd->deal_locked =0;
-		sd->trade_partner=0;
-		target_sd->deal_locked=0;
-		target_sd->trade_partner=0;
+		sd->deal_locked = 0;
+		sd->trade_partner = 0;
+		target_sd->deal_locked = 0;
+		target_sd->trade_partner = 0;
 		clif_tradecancelled(sd);
 		clif_tradecancelled(target_sd);
 	}
@@ -220,8 +288,20 @@ void trade_tradecommit(struct map_session_data *sd) {
 
 	if ((target_sd = map_id2sd(sd->trade_partner)) != NULL) {
 		if ((sd->deal_locked >= 1) && (target_sd->deal_locked >= 1)) { // both have pressed 'ok'
-			if (sd->deal_locked < 2) {sd->deal_locked = 2;} // set locked to 2
+			if (sd->deal_locked < 2) { // set locked to 2
+				sd->deal_locked = 2;
+			}
 			if (target_sd->deal_locked == 2) { // the other one pressed 'trade' too
+				// check exploit (trade more items that you have)
+				if (impossible_trade_check(sd)) {
+					trade_tradecancel(sd);
+					return;
+				}
+				// check exploit (trade more items that you have)
+				if (impossible_trade_check(target_sd)) {
+					trade_tradecancel(target_sd);
+					return;
+				}
 				// check zenys value against hackers
 				if (sd->deal_zeny >= 0 && sd->deal_zeny <= MAX_ZENY && sd->deal_zeny <= sd->status.zeny && // check amount
 				    (target_sd->status.zeny + sd->deal_zeny) <= MAX_ZENY && // fix positiv overflow
@@ -236,9 +316,6 @@ void trade_tradecommit(struct map_session_data *sd) {
 							if (sd->status.inventory[n].amount < sd->deal_item_amount[trade_i])
 								sd->deal_item_amount[trade_i] = sd->status.inventory[n].amount;
 
-							if(log_config.trade > 0)
-								log_trade(sd,target_sd,n,sd->deal_item_amount[trade_i]);
-
 							flag = pc_additem(target_sd, &sd->status.inventory[n], sd->deal_item_amount[trade_i]);
 							if (flag == 0)
 								pc_delitem(sd, n, sd->deal_item_amount[trade_i], 1);
@@ -253,9 +330,6 @@ void trade_tradecommit(struct map_session_data *sd) {
 							if (target_sd->status.inventory[n].amount < target_sd->deal_item_amount[trade_i])
 								target_sd->deal_item_amount[trade_i] = target_sd->status.inventory[n].amount;
 
-							if(log_config.trade > 0)
-								log_trade(target_sd,sd,n,target_sd->deal_item_amount[trade_i]);
-
 							flag = pc_additem(sd, &target_sd->status.inventory[n], target_sd->deal_item_amount[trade_i]);
 							if (flag == 0)
 								pc_delitem(target_sd, n, target_sd->deal_item_amount[trade_i], 1);
@@ -266,15 +340,10 @@ void trade_tradecommit(struct map_session_data *sd) {
 						}
 					}
 					if (sd->deal_zeny) {
-						if (log_config.trade > 0 && log_config.zeny > 0)
-							log_zeny(sd, target_sd, sd->deal_zeny);
 						sd->status.zeny -= sd->deal_zeny;
 						target_sd->status.zeny += sd->deal_zeny;
 					}
 					if (target_sd->deal_zeny) {
-						if (log_config.trade > 0 && log_config.zeny > 0)
-							log_zeny(target_sd, sd, sd->deal_zeny);
-
 						target_sd->status.zeny -= target_sd->deal_zeny;
 						sd->status.zeny += target_sd->deal_zeny;
 					}