Eliminando el rollback del Chaos Mix se ahorran todos los problemas de dupeos a través del Chaos Mix.
El Chaos Mix tiene un rollback exclusivamente por si a alguien se le cierra el juego mientras estaba usando la máquina, entonces antes de cerrar el juego verifica si su transacción con el chaos mix está finalizada y en caso de que no, simplemente le restaura los items hacia el inventario. El problema de esto es que guarda el estado al momento de abrir el chaos mix y todo lo que pase en el medio, si no está bien controlado, se presta a este tipo de cosas. Es por eso que yo opté por directamente eliminar el rollback del chaos mix y ahorrarme mil dolores de cabeza, ya que es muy poco común que a un usuario se le desconecte estando en la máquina chaos.

Si quieren eliminar el rollback del chaos mix hagan esto:

Code: [Select]

bool CObjectManager::CharacterGameClose(int aIndex) // OK
{
	if (OBJECT_RANGE(aIndex) == 0)
	{
		return 0;
	}

	LPOBJ lpObj = &gObj[aIndex];

	if (lpObj->Connected != OBJECT_ONLINE)
	{
		return 0;
	}

	if (lpObj->Type != OBJECT_USER || lpObj->MapServerMoveQuit != 0)
	{
		return 0;
	}

	if (OBJECT_RANGE(lpObj->SummonIndex) != 0)
	{
		gObjSummonKill(lpObj->Index);
	}

	if (lpObj->Interface.use != 0 && (lpObj->Interface.type == INTERFACE_CHAOS_BOX || lpObj->Interface.type == INTERFACE_TRAINER))
	{
		if (lpObj->IsChaosMixCompleted != 0 || lpObj->IsCastleNPCUpgradeCompleted != 0)
		{
			gChaosBox.ChaosBoxItemSave(lpObj);
		}
		//Comment this to fix chaos dupe
		//else
		//{
		//	gObjInventoryRollback(aIndex);
		//}
	}

	if (OBJECT_RANGE(lpObj->TargetNumber) != 0)
	{
		gTrade.ResetTrade(lpObj->TargetNumber);
		gTrade.GCTradeResultSend(lpObj->TargetNumber, 0);
		gTrade.ResetTrade(aIndex);
	}

Y por último paso a explicarles el bug.

-Al tirar party, el gameserver coloca Interface State en 1 y empieza el contador de tiempo del party.
-Al apretar Tab 2 veces, el main envía al gameserver "gNpcTalk.CGNpcTalkCloseRecv" haciendo que cancele el Interface State en 1 y lo ponga en 0
-Al estar el Interface State en 0, nos permite abrir el Chaos Mix, lo cual nuevamente vuelve a poner el Interface State en 1
-Como tiramos party y no recibimos una respuesta, luego de unos segundos automáticamente el party se cancela, por lo que vuelve a colocar el Interface State en 0 y eso genera el bug de tener la máquina de chaos abierta pero que en el gameserver no la tenga en cuenta y nos permita mover, usar baul, etc.
-Una vez que colocamos los items en el baul, si volvemos a la máquina de chaos y cerramos el juego, se ejecutará el rollback debido a que nuestra transacción con la máquina de chaos nunca se completo ya que no realizamos ningún mix, entonces nos restaura los items hasta el punto en donde abrimos la máquina por primera vez pero esos items también quedarán guardados en el baul, lo que genera el dupe.

Esta es mi interpretación del dupeo.

finalizar con el rollback es buena opción, y siguiendo la lógica del closenpctalk se debería agregar check para validar tanto el duel request y party interface, ya que party interface por ende volvería en automatico a none dejando esto como una forma de evitar que se resete los valores de validación de INTERFACE Y use

Code: [Select]

void CNpcTalk::CGNpcTalkCloseRecv(int aIndex) // OK
{
	LPOBJ lpObj = &gObj[aIndex];

	if(gObjIsConnectedGP(aIndex) == 0)
	{
		return;
	}

	if(lpObj->Interface.use == 0 
		|| lpObj->Interface.type == INTERFACE_NONE 
		|| lpObj->Interface.type == INTERFACE_PARTY
		|| lpObj->Interface.type == INTERFACE_CHAOS_BOX
		|| lpObj->Interface.DuelUserReserved != -1
		|| lpObj->Interface.DuelUserRequested != -1
		)
	{
		return;
	}

	switch(lpObj->Interface.type)
	{
		case INTERFACE_TRADE:
			gTrade.CGTradeCancelButtonRecv(aIndex);
			break;
		case INTERFACE_SHOP:
			if(((lpObj->Interface.state==1)?(lpObj->Interface.state=0):1)==0){return;}
			lpObj->TargetShopNumber = -1;
			break;
		case INTERFACE_WAREHOUSE:
			gWarehouse.CGWarehouseClose(aIndex);
			break;
		case INTERFACE_TRAINER:
			gChaosBox.ChaosBoxInit(lpObj);
			gObjInventoryCommit(aIndex);
			break;
		case INTERFACE_QUEST_WORLD:
			lpObj->QuestWorldMonsterClass = -1;
			break;
	}

	lpObj->Interface.use = 0;
	lpObj->Interface.type = INTERFACE_NONE;
	lpObj->Interface.state = 0;
}

no lo he probado, pero siguiendo la lógica que estoy leyendo esto debería impedir que se resete los valores de Interface mientras se use PARTY_REQUEST y duel, mañana voy a tocar todo este tema de fixes, para ver a que mas se puede hacer...  dance dance dance dance

Otra cosa también es eliminar el envío del npctalkclose desde el main al presionar el tab y cerrar el minimapa. Ya que no surte ningún efecto lado servidor abrirlo o cerrarlo, directamente no debería hacer ningún envío.
Lo mismo pasa al presionar la T y cerrar la interface de quest si no me equivoco

buen punto

Code: [Select]

SetCompleteHook(0xE9, 0x0082A983, 0x0082A9F3); //ok

fix all done.

thanks all fix now :D

Edit Dupe, find it from the latest Solar files :)

Edit Dupe, find it from the latest Solar files :)

SHow video :D

Edit Dupe, find it from the latest Solar files :)

SHow video :D

Sent to you to view in PM.

User.cpp


NpcTalk.cpp


Credits : SolarMu

Is there anyway to fix other Dupe in NPC? like NPC ELPIS/AIDA/SEED MASTER/TRAINER?

Thanks in Advance

Is there anyway to fix other Dupe in NPC? like NPC ELPIS/AIDA/SEED MASTER/TRAINER?

Thanks in Advance

void gObjInterfaceCheckTime(LPOBJ lpObj) // OK
{
   if(lpObj->Interface.use == 0)
   {
      return;
   }

   if((GetTickCount()-lpObj->InterfaceTime) < 5000)
   {
      return;
   }

   if(lpObj->Interface.type == INTERFACE_TRADE)
   {
      if(lpObj->Interface.state == 0)
      {
         if(OBJECT_RANGE(lpObj->TargetNumber) != 0)
         {
            gObj[lpObj->TargetNumber].Interface.use = 0;
            gObj[lpObj->TargetNumber].Interface.type = INTERFACE_NONE;
            gObj[lpObj->TargetNumber].Interface.state = 0;
            gObj[lpObj->TargetNumber].TargetNumber = -1;

            gTrade.GCTradeResultSend(lpObj->TargetNumber,3);
         }

         lpObj->Interface.use = 0;
         lpObj->Interface.type = INTERFACE_NONE;
         lpObj->Interface.state = 0;
         lpObj->TargetNumber = -1;

         gTrade.GCTradeResultSend(lpObj->Index,3);
      }
   }

   if(lpObj->Interface.type == INTERFACE_PARTY)
   {
      if(lpObj->Interface.state == 0)
      {
         if(OBJECT_RANGE(lpObj->TargetNumber) != 0)
         {
            gObj[lpObj->TargetNumber].Interface.use = 0;
            gObj[lpObj->TargetNumber].Interface.type = INTERFACE_NONE;
            gObj[lpObj->TargetNumber].Interface.state = 0;
            gObj[lpObj->TargetNumber].TargetNumber = -1;
            gObj[lpObj->TargetNumber].PartyTargetUser = -1;

            gParty.GCPartyResultSend(lpObj->TargetNumber,0);
         }

         lpObj->Interface.use = 0;
         lpObj->Interface.type = INTERFACE_NONE;
         lpObj->Interface.state = 0;
         lpObj->TargetNumber = -1;
         lpObj->PartyTargetUser = -1;

         gParty.GCPartyResultSend(lpObj->Index,0);
      }
   }

   lpObj->InterfaceTime = GetTickCount();
}

 

DUPE OTHER NPC

DUPE OTHER NPC

As I said above, it's very easy, either way, let's just do it normally and don't have to fix it. Fixed at Rollback Only, even if you can make a bug, the item can't be copied anyway.

use base ROX

DUPE OTHER NPC

As I said above, it's very easy, either way, let's just do it normally and don't have to fix it. Fixed at Rollback Only, even if you can make a bug, the item can't be copied anyway.

use base ROX

you share?