Help! Hooking VOID multiplier function

DeeNastyle · Jul 29, 2022

Hi guys,

I took my first steps at modding a few days ago but I can't manage to make my function work.
I'm using LGL Model Menu template 3.2 to mod a unity libil2cpp game on android.

I want to do a damage multiplier function with a slider.
Here the offset I've found with dnSpy:

On Main.cpp I've tried the following:

Any clue how to make the function work ?

Thanks guys,

hienngocloveyou · Jul 30, 2022

Maybe the game server side so can not hack

but wait the method has more parameters you must add-in full of parameters.

NullCoder · Jul 30, 2022

C++:

if (instance && DamageMultiplier > 1) {
    hpDamage *= DamageMultiplier;
    shieldDamage *= DamageMultiplier;
}

It must be so, you haven't provided all the necessary arguments to the ApplyAttackDamage method yet.

In the switch construction, value has an integer type, which means that it would be more correct to specify the type not float (real) but int (integer), It's certainly not a bug, but still.

DeeNastyle · Jul 30, 2022

It's an online game yes but battle calculation are client-sided, there is a lot of working mod menu out there with damage multiplier & defence multiplier functions..just trying to make my own :)

I will try to change the switch construction, thank you guys.

alsya · Aug 1, 2022

if( instance != NULL && SliderValue > 1){
myfunction(instance, SliderValue);
}

Arch01 · Aug 1, 2022

NullCoder said:
C++:

if (instance && DamageMultiplier > 1) { hpDamage *= DamageMultiplier; shieldDamage *= DamageMultiplier; }

It must be so, you haven't provided all the necessary arguments to the ApplyAttackDamage method yet.

In the switch construction, value has an integer type, which means that it would be more correct to specify the type not float (real) but int (integer), It's certainly not a bug, but still.

try this

C++:

float DamageMultiplier;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule){
     if (instance != NULL && DamageMultiplier > 1){
          return old_ApplyAttackDamage(instance,int hpDamage * DamageMultiplier ,int shieldDamage * DamageMultiplier,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
     }
     return old_ApplyAttackDamage(instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
}

HOOK("OFFSETS",ApplyAttackDamage,old_ApplyAttackDamage);


switch(featNum):
case 10:
          if (value >= 1){
               DamageMultiplier = value;
          }
break;

·҉ dollaz·҉. . · Aug 1, 2022

Create a function pointer, and call it in an update hook

DeeNastyle · Aug 1, 2022

Thank you for the answers.

Arch01 said:

try this

C++:

float DamageMultiplier;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule){
     if (instance != NULL && DamageMultiplier > 1){
          return old_ApplyAttackDamage(instance,int hpDamage * DamageMultiplier ,int shieldDamage * DamageMultiplier,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
     }
     return old_ApplyAttackDamage(instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
}

HOOK("OFFSETS",ApplyAttackDamage,old_ApplyAttackDamage);


switch(featNum):
case 10:
          if (value >= 1){
               DamageMultiplier = value;
          }
break;

I can't even build LGL Mod Menu with this code, it does not understand "BattleUnitController", "attacker", "UnitMoveType", "moveType" etc.
"BattleUnitController" and so on are lib classes
"attacker" and so on are lib parameters

NullCoder said:
C++:

if (instance && DamageMultiplier > 1) { hpDamage *= DamageMultiplier; shieldDamage *= DamageMultiplier; }

It must be so, you haven't provided all the necessary arguments to the ApplyAttackDamage method yet.

In the switch construction, value has an integer type, which means that it would be more correct to specify the type not float (real) but int (integer), It's certainly not a bug, but still.

It didn't worked neither but I guess it also about void calling..

So I've tryied the following:

C++:

int DamageMultiplier = 1;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* BattleUnitController,void* UnitMoveType,void* DamageTypeEnum,bool showUiMessaging,bool canCounterAttack,void* BattleEffectDO,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* BattleUnitController,void* UnitMoveType,void* DamageTypeEnum,bool showUiMessaging,bool canCounterAttack,void* BattleEffectDO,bool fromDestroyModule)
 {
    if (instance != NULL && DamageMultiplier > 1)
    {
        return old_ApplyAttackDamage(instance, hpDamage * DamageMultiplier , shieldDamage * DamageMultiplier, isCrit, BattleUnitController, UnitMoveType, DamageTypeEnum, showUiMessaging, canCounterAttack, BattleEffectDO, fromDestroyModule);
    }
    return old_ApplyAttackDamage(instance, hpDamage, shieldDamage, isCrit, BattleUnitController, UnitMoveType, DamageTypeEnum, showUiMessaging, canCounterAttack, BattleEffectDO, fromDestroyModule);

 }

HOOK("0xd92288", ApplyAttackDamage, old_ApplyAttackDamage);

    switch (featNum) {
        
        case 11:
            if (value >= 1) {
                DamageMultiplier = value;
            }
            break;

I can build the Mod Menu and recompile the apk with this one but it's not working, any other clue ?

Raebydett · Aug 1, 2022

DeeNastyle said:
any other clue ?

This is normal when modding, is all about try and error. Just keep searching it until u find the right one

·҉ dollaz·҉. . · Aug 2, 2022

DeeNastyle said:

Thank you for the answers.

I can't even build LGL Mod Menu with this code, it does not understand "BattleUnitController", "attacker", "UnitMoveType", "moveType" etc.
"BattleUnitController" and so on are lib classes
"attacker" and so on are lib parameters

It didn't worked neither but I guess it also about void calling..

So I've tryied the following:

C++:

int DamageMultiplier = 1;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* BattleUnitController,void* UnitMoveType,void* DamageTypeEnum,bool showUiMessaging,bool canCounterAttack,void* BattleEffectDO,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* BattleUnitController,void* UnitMoveType,void* DamageTypeEnum,bool showUiMessaging,bool canCounterAttack,void* BattleEffectDO,bool fromDestroyModule)
{
    if (instance != NULL && DamageMultiplier > 1)
    {
        return old_ApplyAttackDamage(instance, hpDamage * DamageMultiplier , shieldDamage * DamageMultiplier, isCrit, BattleUnitController, UnitMoveType, DamageTypeEnum, showUiMessaging, canCounterAttack, BattleEffectDO, fromDestroyModule);
    }
    return old_ApplyAttackDamage(instance, hpDamage, shieldDamage, isCrit, BattleUnitController, UnitMoveType, DamageTypeEnum, showUiMessaging, canCounterAttack, BattleEffectDO, fromDestroyModule);

}

HOOK("0xd92288", ApplyAttackDamage, old_ApplyAttackDamage);

    switch (featNum) {
       
        case 11:
            if (value >= 1) {
                DamageMultiplier = value;
            }
            break;

I can build the Mod Menu and recompile the apk with this one but it's not working, any other clue ?

you need to make a function pointer

alsya · Aug 4, 2022

Arch01 said:

try this

C++:

float DamageMultiplier;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule){
     if (instance != NULL && DamageMultiplier > 1){
          return old_ApplyAttackDamage(instance,int hpDamage * DamageMultiplier ,int shieldDamage * DamageMultiplier,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
     }
     return old_ApplyAttackDamage(instance,int hpDamage,int shieldDamage,bool isCrit,BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType,bool showUiMessaging,bool canCounterAttack,[Optional] BattleEffectDO sourceEffect,bool fromDestroyModule);
}

HOOK("OFFSETS",ApplyAttackDamage,old_ApplyAttackDamage);


switch(featNum):
case 10:
          if (value >= 1){
               DamageMultiplier = value;
          }
break;

this is not recommended

Arch01 · Aug 5, 2022

alsya said:
this is not recommended

You say its not recomended , but it works for me so i dont care , :face09:

Raebydett · Aug 5, 2022

Arch01 said:
You say its not recomended , but it works for me so i dont care ,

He said that bcs the sample u give wont be able to compile it is bcs this

C++:

BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType

Enum is usually Int, If u not sure just put void

Arch01 · Aug 5, 2022

Raebydett said:
He said that bcs the sample u give wont be able to compile it is bcs this

C++:

BattleUnitController attacker,BattleUnitController target,UnitMoveType moveType,DamageTypeEnum damageType

Enum is usually Int, If u not sure just put void

yea i mean , its work for me because most of my mod was have somethink like that in the dump , so yea need to cast all the think into hook variable

Arch01 · Aug 5, 2022

try this , this is was improved version

C++:

float DamageMultiplier;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* attacker,void* target,void* moveType,int damageType,bool showUiMessaging,bool canCounterAttack,[Optional] void* sourceEffect,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* attacker,void* target,void* moveType,int damageType,bool showUiMessaging,bool canCounterAttack,[Optional] void* sourceEffect,bool fromDestroyModule){
     if (instance != NULL && DamageMultiplier > 1){
          return old_ApplyAttackDamage(instance,int hpDamage * DamageMultiplier ,int shieldDamage * DamageMultiplier,isCrit,attacker,target,damageType,showUiMessaging,canCounterAttack,[Optional] sourceEffect,fromDestroyModule);
     }
     return old_ApplyAttackDamage(instance,hpDamage,shieldDamage,isCrit,attacker,target,moveType,damageType,showUiMessaging,canCounterAttack,[Optional] sourceEffect,fromDestroyModule);
}

HOOK("OFFSETS",ApplyAttackDamage,old_ApplyAttackDamage);


switch(featNum):
case 10:
          if (value >= 1){
               DamageMultiplier = value;
          }
break;

[/QUOTE]

alsya · Aug 5, 2022

Arch01 said:

try this , this is was improved version

C++:

float DamageMultiplier;

void (*old_ApplyAttackDamage)(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* attacker,void* target,void* moveType,int damageType,bool showUiMessaging,bool canCounterAttack,[Optional] void* sourceEffect,bool fromDestroyModule);
void ApplyAttackDamage(void *instance,int hpDamage,int shieldDamage,bool isCrit,void* attacker,void* target,void* moveType,int damageType,bool showUiMessaging,bool canCounterAttack,[Optional] void* sourceEffect,bool fromDestroyModule){
     if (instance != NULL && DamageMultiplier > 1){
          return old_ApplyAttackDamage(instance,int hpDamage * DamageMultiplier ,int shieldDamage * DamageMultiplier,isCrit,attacker,target,damageType,showUiMessaging,canCounterAttack,[Optional] sourceEffect,fromDestroyModule);
     }
     return old_ApplyAttackDamage(instance,hpDamage,shieldDamage,isCrit,attacker,target,moveType,damageType,showUiMessaging,canCounterAttack,[Optional] sourceEffect,fromDestroyModule);
}

HOOK("OFFSETS",ApplyAttackDamage,old_ApplyAttackDamage);


switch(featNum):
case 10:
          if (value >= 1){
               DamageMultiplier = value;
          }
break;

[/QUOTE]
this is not recommended, why should type void return? and consuming too much space should be easy, just replace the parameter with a damagemultiplier.

Arch01 · Aug 5, 2022

alsya said:
this is not recommended, why should type void return? and consuming too much space should be easy, just replace the parameter with a damagemultiplier.

Help! Hooking VOID multiplier function

DeeNastyle

Platinian

Attachments

hienngocloveyou

Solid & Active Platinian

NullCoder

Inactive Approved Modder

DeeNastyle

Platinian

alsya

Solid & Active Platinian

Arch01

2/3 Games Approved

·҉ dollaz·҉. .

Approved iModder

DeeNastyle

Platinian

Raebydett

Awesome Active Platinian

·҉ dollaz·҉. .

Approved iModder

alsya

Solid & Active Platinian

Arch01

2/3 Games Approved

Raebydett

Awesome Active Platinian

Arch01

2/3 Games Approved

Arch01

2/3 Games Approved

alsya

Solid & Active Platinian

Arch01

2/3 Games Approved

About us

Quick Navigation

Share this page

Forum statistics