[JoneM]

Hi, in logs always get the same error all the time, what does mean "index of bounds" and how to fix this? Plugin works, but get the errors..

L 10/09/2023 - 03:10:04: [AMXX] Displaying debug trace (plugin "plugin.amxx", version "1.0.0")
L 10/09/2023 - 03:10:04: [AMXX] Run time error 4: index out of bounds
L 10/09/2023 - 03:10:04: [AMXX] [0] plugin.sma::CBasePlayerWeapon_DefaultDeploy (line 201)


L 10/09/2023 - 11:52:00: [AMXX] Displaying debug trace (plugin "plugin.amxx", version "1.0.0")
L 10/09/2023 - 11:52:00: [AMXX] Run time error 4: index out of bounds
L 10/09/2023 - 11:52:00: [AMXX] [0] plugin.sma::CBasePlayerWeapon_DefaultDeploy (line 201)
L 10/09/2023 - 11:52:00: [AMXX] Displaying debug trace (plugin "plugin.amxx", version "1.0.0")
L 10/09/2023 - 11:52:00: [AMXX] Run time error 4: index out of bounds
L 10/09/2023 - 11:52:00: [AMXX] [0] plugin.sma::CWeaponBox_SetModel (line 229)




201 line:

PHP Code:

public CBasePlayerWeapon_DefaultDeploy(pWeapon, viewModel[], weaponModel[], anim, animExt[], skiplocal)
{
    if (get_entvar(pWeapon, var_impulse) != 0)
        return HC_CONTINUE;
    new iId = get_member(pWeapon, m_iId);
    new iNum = g_iPlayerSkins[get_member(pWeapon, m_pPlayer)][iId];
    if (iNum == -1)
        return HC_CONTINUE;
    
    for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)
    {
        ArrayGetArray(g_aWeaponSkins, i, aWeapons);
        if (aWeapons[WEAPON_ID] != iId)
            continue;
        if (iNum != i)
            continue;
        if (aWeapons[V_MODEL][0])
            SetHookChainArg(2, ATYPE_STRING, aWeapons[V_MODEL]);
        if (aWeapons[P_MODEL][0])
            SetHookChainArg(3, ATYPE_STRING, aWeapons[P_MODEL]);
        break;
    }
    return HC_CONTINUE;
}



public CWeaponBox_SetModel(pWeaponBox, const szModel[])
{
    new pWeapon = UTIL_GetWeaponBoxWeapon(pWeaponBox);
229 line    if (pWeapon == NULLENT)
        return HC_CONTINUE;
    if (get_entvar(pWeapon, var_impulse) != 0)
        return HC_CONTINUE;
    new iId = get_member(pWeapon, m_iId);
    new iNum = g_iPlayerSkins[get_entvar(pWeaponBox, var_owner)][iId];
    if (iNum == -1)
        return HC_CONTINUE;
    for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)
    {
        ArrayGetArray(g_aWeaponSkins, i, aWeapons);
        if (aWeapons[WEAPON_ID] != iId)
            continue;
        if (iNum != i)
            continue;
        if (aWeapons[W_MODEL][0])
            SetHookChainArg(2, ATYPE_STRING, aWeapons[W_MODEL]);
        break;
    }
    return HC_CONTINUE;
} 


[mlibre]

show the full code and use the php tag to highlight it [php]code here[/php]

[JoneM]

sss

[mlibre]

Originally Posted by AlliedModders Rules Do not blank out posts or threads. If you posted a question and later solve your problem, post the solution for others to find.

[Ace67]

Originally Posted by JoneM Hi, in logs always get the same error all the time, what does mean "index of bounds" and how to fix this? Plugin works, but get the errors.. L 10/09/2023 - 03:10:04: [AMXX] Displaying debug trace (plugin "plugin.amxx", version "1.0.0") L 10/09/2023 - 03:10:04: [AMXX] Run time error 4: index out of bounds L 10/09/2023 - 03:10:04: [AMXX] [0] plugin.sma::CBasePlayerWeapon_DefaultDeploy (line 201) L 10/09/2023 - 11:52:00: [AMXX] Displaying debug trace (plugin "plugin.amxx", version "1.0.0") L 10/09/2023 - 11:52:00: [AMXX] Run time error 4: index out of bounds L 10/09/2023 - 11:52:00: [AMXX] [0] plugin.sma::CBasePlayerWeapon_DefaultDeploy (line 201) L 10/09/2023 - 11:52:00: [AMXX] Displaying debug trace (plugin "plugin.amxx", version "1.0.0") L 10/09/2023 - 11:52:00: [AMXX] Run time error 4: index out of bounds L 10/09/2023 - 11:52:00: [AMXX] [0] plugin.sma::CWeaponBox_SetModel (line 229) 201 line: PHP Code: public CBasePlayerWeapon_DefaultDeploy(pWeapon, viewModel[], weaponModel[], anim, animExt[], skiplocal) {     if (get_entvar(pWeapon, var_impulse) != 0)         return HC_CONTINUE;     new iId = get_member(pWeapon, m_iId);     new iNum = g_iPlayerSkins[get_member(pWeapon, m_pPlayer)][iId];     if (iNum == -1)         return HC_CONTINUE;          for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)     {         ArrayGetArray(g_aWeaponSkins, i, aWeapons);         if (aWeapons[WEAPON_ID] != iId)             continue;         if (iNum != i)             continue;         if (aWeapons[V_MODEL][0])             SetHookChainArg(2, ATYPE_STRING, aWeapons[V_MODEL]);         if (aWeapons[P_MODEL][0])             SetHookChainArg(3, ATYPE_STRING, aWeapons[P_MODEL]);         break;     }     return HC_CONTINUE; } public CWeaponBox_SetModel(pWeaponBox, const szModel[]) {     new pWeapon = UTIL_GetWeaponBoxWeapon(pWeaponBox); 229 line    if (pWeapon == NULLENT)         return HC_CONTINUE;     if (get_entvar(pWeapon, var_impulse) != 0)         return HC_CONTINUE;     new iId = get_member(pWeapon, m_iId);     new iNum = g_iPlayerSkins[get_entvar(pWeaponBox, var_owner)][iId];     if (iNum == -1)         return HC_CONTINUE;     for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)     {         ArrayGetArray(g_aWeaponSkins, i, aWeapons);         if (aWeapons[WEAPON_ID] != iId)             continue;         if (iNum != i)             continue;         if (aWeapons[W_MODEL][0])             SetHookChainArg(2, ATYPE_STRING, aWeapons[W_MODEL]);         break;     }     return HC_CONTINUE; }

In both functions, I added checks to ensure that iNum is within the bounds of the g_iWeaponSkins array to prevent the "index out of bounds" error.

Try it and let me know

public CBasePlayerWeapon_DefaultDeploy(pWeapon, viewModel[], weaponModel[], anim, animExt[], skiplocal)
{
    if (get_entvar(pWeapon, var_impulse) != 0)
        return HC_CONTINUE;
    new iId = get_member(pWeapon, m_iId);
    new iNum = g_iPlayerSkins[get_member(pWeapon, m_pPlayer)][iId];
    if (iNum == -1 || iNum >= g_iWeaponSkinsNum)
        return HC_CONTINUE;

    for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)
    {
        ArrayGetArray(g_aWeaponSkins, i, aWeapons);
        if (aWeapons[WEAPON_ID] != iId)
            continue;
        if (iNum != i)
            continue;
        if (aWeapons[V_MODEL][0])
            SetHookChainArg(2, ATYPE_STRING, aWeapons[V_MODEL]);
        if (aWeapons[P_MODEL][0])
            SetHookChainArg(3, ATYPE_STRING, aWeapons[P_MODEL]);
        break;
    }
    return HC_CONTINUE;
}

public CWeaponBox_SetModel(pWeaponBox, const szModel[])
{
    new pWeapon = UTIL_GetWeaponBoxWeapon(pWeaponBox);
    if (pWeapon == NULLENT)
        return HC_CONTINUE;
    if (get_entvar(pWeapon, var_impulse) != 0)
        return HC_CONTINUE;
    new iId = get_member(pWeapon, m_iId);
    new iNum = g_iPlayerSkins[get_entvar(pWeaponBox, var_owner)][iId];
    if (iNum == -1 || iNum >= g_iWeaponSkinsNum)
        return HC_CONTINUE;

    for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)
    {
        ArrayGetArray(g_aWeaponSkins, i, aWeapons);
        if (aWeapons[WEAPON_ID] != iId)
            continue;
        if (iNum != i)
            continue;
        if (aWeapons[W_MODEL][0])
            SetHookChainArg(2, ATYPE_STRING, aWeapons[W_MODEL]);
        break;
    }
    return HC_CONTINUE;
}

[JoneM]

Originally Posted by Ace67 In both functions, I added checks to ensure that iNum is within the bounds of the g_iWeaponSkins array to prevent the "index out of bounds" error. Try it and let me know public CBasePlayerWeapon_DefaultDeploy(pWeapon, viewModel[], weaponModel[], anim, animExt[], skiplocal) { if (get_entvar(pWeapon, var_impulse) != 0) return HC_CONTINUE; new iId = get_member(pWeapon, m_iId); new iNum = g_iPlayerSkins[get_member(pWeapon, m_pPlayer)][iId]; if (iNum == -1 || iNum >= g_iWeaponSkinsNum) return HC_CONTINUE; for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++) { ArrayGetArray(g_aWeaponSkins, i, aWeapons); if (aWeapons[WEAPON_ID] != iId) continue; if (iNum != i) continue; if (aWeapons[V_MODEL][0]) SetHookChainArg(2, ATYPE_STRING, aWeapons[V_MODEL]); if (aWeapons[P_MODEL][0]) SetHookChainArg(3, ATYPE_STRING, aWeapons[P_MODEL]); break; } return HC_CONTINUE; } public CWeaponBox_SetModel(pWeaponBox, const szModel[]) { new pWeapon = UTIL_GetWeaponBoxWeapon(pWeaponBox); if (pWeapon == NULLENT) return HC_CONTINUE; if (get_entvar(pWeapon, var_impulse) != 0) return HC_CONTINUE; new iId = get_member(pWeapon, m_iId); new iNum = g_iPlayerSkins[get_entvar(pWeaponBox, var_owner)][iId]; if (iNum == -1 || iNum >= g_iWeaponSkinsNum) return HC_CONTINUE; for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++) { ArrayGetArray(g_aWeaponSkins, i, aWeapons); if (aWeapons[WEAPON_ID] != iId) continue; if (iNum != i) continue; if (aWeapons[W_MODEL][0]) SetHookChainArg(2, ATYPE_STRING, aWeapons[W_MODEL]); break; } return HC_CONTINUE; }

Same error, marked which line, with "this ->"

PHP Code:

public CBasePlayerWeapon_DefaultDeploy(pWeapon, viewModel[], weaponModel[], anim, animExt[], skiplocal)
{
    if (get_entvar(pWeapon, var_impulse) != 0)
        return HC_CONTINUE;
    new iId = get_member(pWeapon, m_iId);
this ->    new iNum = g_iPlayerSkins[get_member(pWeapon, m_pPlayer)][iId];
    if (iNum == -1 || iNum >= g_iWeaponSkinsNum)
        return HC_CONTINUE;

    for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)
    {
        ArrayGetArray(g_aWeaponSkins, i, aWeapons);
        if (aWeapons[WEAPON_ID] != iId)
            continue;
        if (iNum != i)
            continue;
        if (aWeapons[V_MODEL][0])
            SetHookChainArg(2, ATYPE_STRING, aWeapons[V_MODEL]);
        if (aWeapons[P_MODEL][0])
            SetHookChainArg(3, ATYPE_STRING, aWeapons[P_MODEL]);
        break;
    }
    return HC_CONTINUE;
} 


PHP Code:

public CWeaponBox_SetModel(pWeaponBox, const szModel[])
{
    new pWeapon = UTIL_GetWeaponBoxWeapon(pWeaponBox);
    if (pWeapon == NULLENT)
        return HC_CONTINUE;
    if (get_entvar(pWeapon, var_impulse) != 0)
        return HC_CONTINUE;
    new iId = get_member(pWeapon, m_iId);
this ->    new iNum = g_iPlayerSkins[get_entvar(pWeaponBox, var_owner)][iId];
    if (iNum == -1 || iNum >= g_iWeaponSkinsNum)
        return HC_CONTINUE;

    for (new i, aWeapons[WEAPON_DATA]; i < g_iWeaponSkinsNum; i++)
    {
        ArrayGetArray(g_aWeaponSkins, i, aWeapons);
        if (aWeapons[WEAPON_ID] != iId)
            continue;
        if (iNum != i)
            continue;
        if (aWeapons[W_MODEL][0])
            SetHookChainArg(2, ATYPE_STRING, aWeapons[W_MODEL]);
        break;
    }
    return HC_CONTINUE;
}