Outro Bug de clonagem de dinheiro/itens

[Bruno Carvalho 837]

Bom, estou criando esse tópico (que eu queria ter feito em vídeo no começo do ano) para contar-lhes um dos maiores bugs que existem nos otservs.

 

Como alguns sabem eu participei da equipe do CaterOT e lá descobri muitas falhas em servidores e até dentro da equipe já tive a ideia de publicar estes problemas.

 

Talvez o problema que mais dá dor de cabeça nos servidores é a clonagem de dinheiro... Como aquele player ficou rico?!

Basicamente o problema consiste em duas funções (removeMoney e removeItem), quando elas são executadas, é retornado um booleano (verdadeiro ou falso) para saber se o item/dinheiro foi removido ou não. Mas se essa função não está dentro de um verificador (IF) mesmo que ela seja falsa ela irá funcionar e entregar o dinheiro/o item comprado sem verificar nada.

 

Aos que conhecem C++, leia esse código e me diga se ele retorna algum nil caso o dinheiro não for encontrado:

TFS 1.3: https://github.com/otland/forgottenserver/blob/master/src/luascript.cpp#L8487

Base TFS 0.3: https://github.com/mattyx14/otxserver/blob/otxserv2/path_8_6x/sources/luascript.cpp#L5550

 

Verifiquei vários servidores disponíveis no github e vi que vários tem esse problema. Um dos exemplos é o Otxserver-New do malucooo.

Com uma unica pesquisa , vemos tudo o que há no código.

E assim encontramos vários NPCs, Actions e Moveevents que não tem esse cuidado de verificar se o dinheiro foi realmente removido. Um exemplo é o NPC Black Bert

Ai você me fala: Ah Bruno, mas não tem como dar erro por que ele já utiliza o getMoney ali em cima...

 

NÃO ADIANTA USAR O GETMONEY, VOCÊ SÓ USA ELE SE PRECISAR VERIFICAR MAIS DE DOIS ITENS (EM BAIXO MAIS EXPLICAÇÕES).

 

Basicamente as pessoas que fazem esse bug utilizam programas que colocam para o npc comprar/vender o item tendo o dinheiro/item e removem o item do personagem em fração de milésimos após clicar, assim o sistema "buga" (não é um bug, ele somente retorna falso no código pelo fato de o player “não ter“ o item, passando pelo verificador (getItem e getMoney) sem dar erro... O único verificador que funciona é o que diz se o item foi realmente removido.

 

Antes/Depois

Spoiler

Antes:

(Não quero nem comentar os erros amadores que há neste código)

 

Depois:

 

Mas ai surge o problema de: Ah, e quando há mais de um item para ser removido (ou um item junto com dinheiro)?

Dai você usa o getItem/getMoney e abaixo dele utiliza a função de remoção, para evitar que um item seja removido do player caso ele não houver o segundo item necessário.

 

Exemplo:

 

Exemplo 2:

 

Este é um exemplo de como verificar se há mais de um item necessário. E ainda depois do segundo if poderia ser colocado um else alertando aos administradores o nome do player que está tentando bugar, pois se o player não fez nada de errado esse fator nunca irá acontecer.

 

Ainda duvidam de mim?

Você que usa TFS 1.2+ execute esse código em uma talkaction, tendo o item 2160 ou não.

function onSay(player, words, param)
	if not player:getGroup():getAccess() then
		return true
	end

	if player:getAccountType() < ACCOUNT_TYPE_GOD then
		return false
	end

	player:removeItem(2160, 1)
	player:sendTextMessage(MESSAGE_INFO_DESCR, "Item removido!")
end

 

Isto serve para qualquer distribuição/versão de servidor.

Não é um bug de distribuição, é um erro cometido por quase todos os SCRIPTERS.

Editado Maio 20, 2017 por Comedinha (veja o histórico de edições)

[LeoTK 795]

realmente esse bug é muito chato -.-' quando eu estava com meu server online tinha muito desse problema nele e não era só com dinheiro mas com item também isso me faz pensar se o problema de clonar item também é feito usando o mesmo método ou algum parecido

[Bruno Carvalho 837]

Agora, Brunds disse:

realmente esse bug é muito chato -.-' quando eu estava com meu server online tinha muito desse problema nele e não era só com dinheiro mas com item também isso me faz pensar se o problema de clonar item também é feito usando o mesmo método ou algum parecido

 

Clonar item tem esse método ou bugar alguma coisa com conexões inter tabelas (bando de dados), assim que o servidor cai ele surge em 2 lugares... Mas isso vou explicar melhor em um outro dia.

[leozincorsair 42]

E como podemos remover esse bug ? 

[vankk 455]

>> Não quero nem comentar os erros amadores que há neste código

lul.

 

Obrigado por ter reportado Bruno. Talvez deveríamos levar esse bug para o GitHub e fazermos uma fix em C++?

[LeoTK 795]

12 minutos atrás, vankk disse:

>> Não quero nem comentar os erros amadores que há neste código

lul.

 

Obrigado por ter reportado Bruno. Talvez deveríamos levar esse bug para o GitHub e fazermos uma fix em C++?

seria muito bom esses bugs serem fixados

 

1 hora atrás, Comedinha disse:

 

Clonar item tem esse método ou bugar alguma coisa com conexões inter tabelas (bando de dados), assim que o servidor cai ele surge em 2 lugares... Mas isso vou explicar melhor em um outro dia.

entendi, bom irei aguardar então esse dia xD +rep por trazer soluções para problemas que muitos nem dão bola 

[Weslley Kiyo 40]

+ um motivo que eu estou fazendo meu servidor limpo, script por script

[Bruno Carvalho 837]

21 horas atrás, vankk disse:

>> Não quero nem comentar os erros amadores que há neste código

lul.

 

Obrigado por ter reportado Bruno. Talvez deveríamos levar esse bug para o GitHub e fazermos uma fix em C++?

 

O pior é que a primeira coisa que eu pensei foi fazer uma correção na source, mas fazer ele retornar null ao invés de false todas as vezes que o player não tiver o item é muito inviável. Pelo simples fato de que até no IF ele retornaria null e daria um erro no console todas as vezes...

Não achei outra maneira a não ser revisar todos os arquivos...

[KotZletY 1095]

Nada que um check não resolva. Hahaha!

Quando comecei a scriptar, me deparei muito com isso, com o tempo, pude evoluir e fazer checks mais precisos e evitar esse tipo de falha, e essa explicação serviu para fixar isso.

 

Ótima explicação bruno!! 

 

[Weslley Kiyo 40]

Tem do npc bank também. Você precisa fazer um que sempre use check no valor para saber se é verdadeiro.

 

Vejo muitos scritps de bank que nem possui check.

 

E o shopsystem é bom deixa-lo para enviar o item sempre para o depot. Assim, evitando possíveis bugs!

Editado Maio 21, 2017 por Weslley Kiyo (veja o histórico de edições)

[Weslley Kiyo 40]

Em 2017-5-20 ás 16:17, Comedinha disse:

...

 

Nesse caso, qual seria o correto?

 

elseif msgcontains(msg, "yes") then
		if npcHandler.topic[cid] == 1 then	
			if getPlayerItemCount(cid, 2677) >= 5 then -- blueberries
				npcHandler:say("It was a great help! If you have more you can come.", cid)
				doPlayerRemoveItem(cid, 2677, 5) -- honey flower
				doPlayerAddMoney(cid, 1) -- dinheiro (1 gold)
				npcHandler.topic[cid] = 0
			else
				npcHandler:say("You don't have it... Buy from 5 blueberries.", cid)
			end	
		end

 

elseif msgcontains(msg, "yes") then
		if npcHandler.topic[cid] == 1 then	
			if getPlayerItemCount(cid, 2677) >= 5 then -- blueberries
				if doPlayerRemoveItem(cid, 2677, 5) then -- honey flower 
					npcHandler:say("It was a great help! If you have more you can come.", cid)
					doPlayerAddMoney(cid, 1) -- dinheiro (1 gold)
					npcHandler.topic[cid] = 0
				else
					npcHandler:say("Error 404! Contact the administrator.", cid)
					npcHandler:releaseFocus(cid)
				end
			else
				npcHandler:say("You don't have it... Buy from 5 blueberries.", cid)
			end	
		end

 

[Bruno Carvalho 837]

@Weslley Kiyo O segundo modo está correto. Mas não precisaria nem do getPlayerItemCount pois ele já faz isso ao remover... O porém o sistema que você fez seria a maneira mais segura.

[Weslley Kiyo 40]

8 horas atrás, Comedinha disse:

@Weslley Kiyo O segundo modo está correto. Mas não precisaria nem do getPlayerItemCount pois ele já faz isso ao remover... O porém o sistema que você fez seria a maneira mais segura.

 

Eu não vou mentir. Estava fazendo do primeiro modo porém ainda da tempo de corrigir

Muito obrigado @Comedinha pelas dicas

[sdaasd 0]

Alguem sabe como realizar esse ''bug'' Queria ver como funciona isso sei que talvez seja usando um programa chamado WPE