[Release] Small fixes

[Nortix]

Some people use these bugs on several servers and here's a fix for it:




DPSrvr.cpp
CDPSrvr::OnFashionCombine
under

for (int i = 0; i < 8; i++)
{
	ar >> dwItem;
	pItemElem[i] = pUser->m_Inventory.GetAtId(dwItem);
	if (!IsUsableItem(pItemElem[i]))
		return;
}

BOOL bHack = FALSE;
if (pItemElem[0]->GetProp()->dwItemKind3 != IK3_HAT || pItemElem[1]->GetProp()->dwItemKind3 != IK3_HAT)
	bHack = TRUE;
if (pItemElem[2]->GetProp()->dwItemKind3 != IK3_CLOTH || pItemElem[3]->GetProp()->dwItemKind3 != IK3_CLOTH)
	bHack = TRUE;
if (pItemElem[4]->GetProp()->dwItemKind3 != IK3_GLOVE || pItemElem[5]->GetProp()->dwItemKind3 != IK3_GLOVE)
	bHack = TRUE;
if (pItemElem[6]->GetProp()->dwItemKind3 != IK3_SHOES || pItemElem[7]->GetProp()->dwItemKind3 != IK3_SHOES)
	bHack = TRUE;

if (bHack)
{
	Error("[CDPSrvr::OnFashionCombine] %s tried to change IK3", pUser->GetName());
	if (pUser->GetWorld())
		g_dpDBClient.SavePlayer(pUser, pUser->GetWorld()->GetID(), pUser->GetPos(), pUser->GetLayer());
	QueryDestroyPlayer(pUser->m_Snapshot.dpidCache, pUser->m_Snapshot.dpidUser, pUser->m_dwSerial, pUser->m_idPlayer);
	return;
}

MoverActEvent.cpp
CMover::OnMagicSkill

under

if( pSkillProp->dwID == SI_KNT_HERO_DRAWING ){

if (!pFocusObj->GetWorld() || !GetWorld())
	return FALSE;
if(pFocusObj->GetWorld()->GetID() != GetWorld()->GetID())
	return FALSE;

[Wanetrain]

Mhm...

BOOL bHack = FALSE;
if (pItemElem[0]->GetProp()->dwItemKind3 != IK3_HAT || pItemElem[1]->GetProp()->dwItemKind3 != IK3_HAT)
bHack = TRUE;
if (pItemElem[2]->GetProp()->dwItemKind3 != IK3_CLOTH || pItemElem[3]->GetProp()->dwItemKind3 != IK3_CLOTH)
bHack = TRUE;
if (pItemElem[4]->GetProp()->dwItemKind3 != IK3_GLOVE || pItemElem[5]->GetProp()->dwItemKind3 != IK3_GLOVE)
bHack = TRUE;
if (pItemElem[6]->GetProp()->dwItemKind3 != IK3_SHOES || pItemElem[7]->GetProp()->dwItemKind3 != IK3_SHOES)
bHack = TRUE;

if (bHack)
{
Error("[CDPSrvr::OnFashionCombine] %s tried to change IK3", pUser->GetName());
if (pUser->GetWorld())
g_dpDBClient.SavePlayer(pUser, pUser->GetWorld()->GetID(), pUser->GetPos(), pUser->GetLayer());
QueryDestroyPlayer(pUser->m_Snapshot.dpidCache, pUser->m_Snapshot.dpidUser, pUser->m_dwSerial, pUser->m_idPlayer);
}

So bHack = true = Function still go. You maybe forgot the return after "QueryDestroy..." dude. (New Bug Created!!)

[Nortix]

Oh right, rewrote it for the release and forgot the return. Thanks for reminding!

[Nightwalker.]

Originally Posted by Avalion

Code:

 Spoiler

So, this can use bActionSlot thingy in Propskill if you change it. But official servers seem to have it all set to true regardless.

So you could do this:

WorldServer\dpSrvr.cpp
- CDPSrvr::OnSkillTaskBar

	SHORTCUT temp; LPSKILL lpSkill;
	ItemProp* iSkill = NULL;
	for (int i = 0; i < nCount; ++i)
	{
		unsigned char nIndex;
		ar >> nIndex;

		if (nIndex >= MAX_SLOT_QUEUE){ return; }
		ar >> temp.m_dwShortcut >> temp.m_dwId >> temp.m_dwType >> temp.m_dwIndex >> temp.m_dwUserId >> temp.m_dwData;
		if (temp.m_dwShortcut == SHORTCUT_SKILL)
		{

Either from here you could grab the skill id, and check if it is asal, or do what the client does, up to you.
What Client does:

			lpSkill = pUser->GetSkill(0, temp.m_dwId);
			iSkill = prj.GetSkillProp(lpSkill->dwSkill);
			AddSkillProp* pAddSkillProp = prj.GetAddSkillProp(iSkill->dwSubDefine, lpSkill->dwLevel);
			if (pAddSkillProp->dwCooldown > 0 || !iSkill->bCanUseActionSlot)
			{ 
				QueryDestroyPlayer(pUser->m_Snapshot.dpidCache, pUser->m_Snapshot.dpidUser, pUser->m_dwSerial, pUser->m_idPlayer);
				return; 
			}
		}
		pUser->m_playTaskBar.m_aSlotQueue[nIndex] = temp;
	}

if you only care about asal, you can remove the addskillprop and then check the skillid

if (lpSkill->dwSkill == SI_BIL_PST_ASALRAALAIKUM || !iSkill->bCanUseActionSlot)

Then if you want bCanUseActionSlot on client just for people who aren't cheating, just add it to CWndTaskBar::SetSkillQueue.

How do you mean it so how i can put it in the source? i dont understand what do you mean. pls help :x

[Nightwalker.]

Yea i was stupid i got it now. sry

[KetchupSamurai]

if( pSkillProp->dwID == SI_KNT_HERO_DRAWING ){

to

if( pSkillProp->dwID == SI_KNT_HERO_DRAWING || pSkillProp->dwID == SI_LOD_SUP_PULLING ){

for 3rd jobs, seeing no one fixes that skill either lol...