Weapon level command

hello
this command work with __WEAPON_LEVEL system

put an item in 1st inventory slot and use command like /WeaponLevel 100 to put your weapon level to 100

credit : me

Possible crash :
CItemElem* pItemElem = pUser->m_Inventory.GetAt(0);
ItemProp* pProp = pItemElem->GetProp();

pItemElem n'est pas vérifier avant de l'appeler dans pItemElem->GetProp();


if( nLevel > 100 )
nLevel = 100;

Que se passe-t-il si une valeur négative est passer ?

Pareil pUser n'est pas vérifier dans if( pUser->IsMode( TRANSPARENT_MODE ) == 0 )

J'adore les noobs :p

Quote:

#ifdef __WEAPON_LEVELING BOOL TextCmd_SetWeaponLevel( CScanner& scanner ) { #ifdef __WORLDSERVER CUser* pUser = (CUser*)scanner.dwValue; if( !IsValidObj( pUser ) ) return FALSE; CItemElem* pItemElem = pUser->m_Inventory.GetAt(0); if( !pItemElem ) return FALSE; ItemProp* pProp = pItemElem->GetProp(); if( !pProp ) return FALSE; int nLevel = scanner.GetNumber(); int nExp = 0; if( pItemElem ) { if( pProp->dwItemKind3 == IK3_SWD || pProp->dwItemKind3 == IK3_AXE || pProp->dwItemKind3 == IK3_KNUCKLEHAMMER || pProp->dwItemKind3 == IK3_CHEERSTICK || pProp->dwItemKind3 == IK3_WAND || pProp->dwItemKind3 == IK3_STAFF || pProp->dwItemKind3 == IK3_BOW || pProp->dwItemKind3 == IK3_YOYO || pProp->dwItemKind3 == IK3_SHIELD ) { if( nLevel < 1 ) nLevel = 1; if( nLevel > 100 ) nLevel = 100; if( pUser && pUser->IsMode( TRANSPARENT_MODE ) == 0 ) { g_UserMng.AddCreateSfxObj( pUser, XI_INT_SUCCESS, pUser->GetPos().x, pUser->GetPos().y, pUser->GetPos().z ); pUser->AddPlaySound( SND_INF_UPGRADESUCCESS ); } pUser->UpdateItem( (BYTE)pItemElem->m_dwObjId, UI_WEAPON_LEVEL, nLevel ); pUser->UpdateItem( (BYTE)pItemElem->m_dwObjId, UI_WEAPON_EXP, nExp ); } else pUser->AddDefinedText( TID_REALLY_NIGGA ); } #endif // __WORLDSERVER return TRUE; } #endif // __WEAPON_LEVELING

Quote:

Originally Posted by Meutledaron Possible crash : CItemElem* pItemElem = pUser->m_Inventory.GetAt(0); ItemProp* pProp = pItemElem->GetProp(); pItemElem n'est pas vérifier avant de l'appeler dans pItemElem->GetProp(); if( nLevel > 100 ) nLevel = 100; Que se passe-t-il si une valeur négative est passer ? Pareil pUser n'est pas vérifier dans if( pUser->IsMode( TRANSPARENT_MODE ) == 0 )

C'est une commande que j'ai fait en vitesse c'est vrai que je ne l'ai pas faite correctement merci de ta remarque je vais la corriger ;)

Toujours pas bon...

Ma commande est tout à fait bonne
pProp vérifié
pItem vérifié
pUser vérifié

Tu fais appel à

CItemElem* pItemElem = pUser->m_Inventory.GetAt(0); sans l'avoir vérifier.

Ensuite à
ItemProp* pProp = pItemElem->GetProp(); pareil sans vérification, il faut y aller étape par étape.

CUser* pUser = (CUser*)scanner.dwValue;
On vérifier pUser avant d'aller plus loin.

CItemElem* pItemElem = pUser->m_Inventory.GetAt(0);
On vérifie pItemElem avant d'aller plus loin.

Les vérifications dans ton script arrivent trop tard et si l'objet n'est pas bon c'est le crash du worldserver.

Ps : Il manque également la seconde partie de la commande ou tu attribue les droits.

Je pensais qu'il n'était pas nécessaire de le poster car facile à refaire ^^

Toujours pas bon...

Tu doit vérifier ton puser avant, car ici tu l'appel :
CItemElem* pItemElem = pUser->m_Inventory.GetAt(0);

Idem tu doit vérifier ton pItemElem avant, car ici tu l'appel :
ItemProp* pProp = pItemElem->GetProp();

Did you fix the script? I look forward to it :)

The script is good now.

Which post contains the complete final script? Is it the original post?

Quote:

Originally Posted by direktbottle Is it the original post

:)