Баг с атрибутом.

[Литион]

Подскажите как пофиксить баг с атрибутом через сервис вставки атрибут через Hxd ставят сколько угодно атрибута в шмот и оружие Сборка овер.

[KilRoy]

Описание на 5 с +. Если сервис самописный, то явно же, рукожопие кодера не заставило себя ждать
Прежде всего, нужно учитывать уязвимость байпассов и уже потом, сразу же, приходит мысль о том, что нужно не передавать параметры в строках, а использовать заготовленные варианты, либо же, ПРОВЕРЯТЬ то что происходит на клиенте и на сервере! Нельзя доверять клиенту, клиент - зло

[Литион]

Вот сам сервис

Сервис

PHP код:

<?php 

package services.community;



import java.util.StringTokenizer;



import l2ft.gameserver.Config;

import l2ft.gameserver.data.htm.HtmCache;

import l2ft.gameserver.handler.bbs.CommunityBoardManager;

import l2ft.gameserver.model.base.Element;

import l2ft.gameserver.network.l2.s2c.ShowBoard;

import l2ft.gameserver.scripts.Functions;

import l2ft.gameserver.model.Player;

import l2ft.gameserver.model.items.ItemInstance;

import l2ft.gameserver.data.xml.holder.ItemHolder;

import l2ft.gameserver.network.l2.s2c.InventoryUpdate;

import l2ft.gameserver.handler.bbs.ICommunityBoardHandler;

import l2ft.gameserver.templates.item.EtcItemTemplate;

import l2ft.gameserver.network.l2.components.SystemMsg;

import l2ft.gameserver.model.base.TeamType;

import l2ft.gameserver.templates.item.ItemTemplate;

import l2ft.gameserver.scripts.ScriptFile;

import l2ft.gameserver.templates.item.ArmorTemplate.ArmorType;

import l2ft.gameserver.templates.item.support.EnchantItem;

import l2ft.gameserver.utils.BbsUtil;

import l2ft.gameserver.utils.Log;

import org.slf4j.Logger;

import org.slf4j.LoggerFactory;



public class ManageEnchant extends Functions implements ScriptFile, ICommunityBoardHandler

{ 

    static final Logger _log = LoggerFactory.getLogger(ManageEnchant.class);

    private int enchant_item = Config.COMMUNITYBOARD_ENCHANT_ITEM;

    private int max_enchant = Config.COMMUNITYBOARD_MAX_ENCHANT;

    private int[] enchant_level = Config.COMMUNITYBOARD_ENCHANT_LVL;

    private int[] ench_price_weapon = Config.COMMUNITYBOARD_ENCHANT_PRICE_WEAPON;

    private int[] ench_price_armor = Config.COMMUNITYBOARD_ENCHANT_PRICE_ARMOR;

    private int[] atr_lvl_weapon = Config.COMMUNITYBOARD_ENCHANT_ATRIBUTE_LVL_WEAPON;

    private int[] atr_price_weapon = Config.COMMUNITYBOARD_ENCHANT_ATRIBUTE_PRICE_WEAPON;

    private int[] atr_lvl_armor = Config.COMMUNITYBOARD_ENCHANT_ATRIBUTE_LVL_ARMOR;

    private int[] atr_price_armor = Config.COMMUNITYBOARD_ENCHANT_ATRIBUTE_PRICE_ARMOR;

    private boolean atr_pvp = Config.COMMUNITYBOARD_ENCHANT_ATRIBUTE_PVP;

    

    @Override

    public void onLoad(){

    if(Config.COMMUNITYBOARD_ENABLED)

    {

        _log.info("CommunityBoard: Enchant Community service loaded.");

        CommunityBoardManager.getInstance().registerHandler(this);

    }

    }



    @Override

    public void onReload(){

    if(Config.COMMUNITYBOARD_ENABLED)

    {

        CommunityBoardManager.getInstance().removeHandler(this);

    }

    }



    @Override

    public void onShutdown(){}

    

    @Override

    public String[] getBypassCommands(){

    return new String[] {

        "_bbsechant",

        "_bbsechantlist",

        "_bbsechantChus",

        "_bbsechantAtr",

        "_bbsechantgo",

        "_bbsechantuseAtr"

        };

    }



    @Override

    public void onBypassCommand(Player activeChar, String bypass){

        if(!CheckCondition(activeChar))

            return;

        

        if (bypass.startsWith("_bbsechant"))

        {

            String name = "None Name";

            String html = HtmCache.getInstance().getNotNull(Config.BBS_HOME_DIR + "pages/enchant.htm", activeChar);

            name = ItemHolder.getInstance().getTemplate(enchant_item).getName();

            StringBuilder sb = new StringBuilder("");

            sb.append("<table width=400>");

            ItemInstance[] arr = activeChar.getInventory().getItems();

            int len = arr.length;

            for (int i = 0; i < len; i++)

            {

                ItemInstance _item = arr[i];

                if (_item == null || _item.getTemplate().isBelt() || _item.isCursed() || _item.isArrow() 

                    || _item.getTemplate().isBracelet() || _item.getTemplate().isCloak() || _item.isNoEnchant() 

                    || !_item.isEquipped() || _item.isShieldNoEnchant() || _item.getItemType() == ArmorType.SIGIL || _item.isHeroWeapon() 

                    || _item.getItemId() >= 7816 && _item.getItemId() <= 7831 || _item.isShadowItem() 

                    || _item.isCommonItem() || _item.getEnchantLevel() >= (max_enchant + 1) || !_item.canBeEnchanted(true) 

                    || _item.getEquipSlot() == ItemTemplate.SLOT_HAIR || _item.getEquipSlot() == ItemTemplate.SLOT_DHAIR)

                    continue;

                sb.append(new StringBuilder("<tr><td><img src=icon." + _item.getTemplate().getIcon() + " width=32 height=32></td><td>"));

                sb.append(new StringBuilder("<font color=\"LEVEL\">" + _item.getTemplate().getName() + " " + (_item.getEnchantLevel() <= 0 ? "" : new StringBuilder("</font><br1><font color=3293F3>Заточено на: +" + _item.getEnchantLevel())) + "</font><br1>"));

                sb.append(new StringBuilder("Заточка за: <font color=\"LEVEL\">" + name + "</font>"));

                sb.append("<img src=\"l2ui.squaregray\" width=\"170\" height=\"1\">");

                sb.append("</td><td>");

                if(Config.ALLOW_BBS_ENCHANT_ELEMENTAR)

                    sb.append(new StringBuilder("<button value=\"Обычная\" action=\"bypass _bbsechantlist:" + _item.getObjectId() + ";\" width=75 height=18 back=\"L2UI_ct1.button_df\" fore=\"L2UI_ct1.button_df\">"));

                sb.append("</td><td>");

                if(Config.ALLOW_BBS_ENCHANT_ATT)

                    sb.append(new StringBuilder("<button value=\"Аттрибут\" action=\"bypass _bbsechantChus:" + _item.getObjectId() + ";\" width=75 height=18 back=\"L2UI_ct1.button_df\" fore=\"L2UI_ct1.button_df\">"));

                sb.append("</td></tr>");

            }



            sb.append("</table>");

            html = html.replace("%enchanter%",  sb.toString());

            html = BbsUtil.htmlBuff(html, activeChar);

            ShowBoard.separateAndSend(html, activeChar);

        }

        if (bypass.startsWith("_bbsechantlist"))

        {

            StringTokenizer st2 = new StringTokenizer(bypass, ";");

            String[] mBypass = st2.nextToken().split(":");

            int ItemForEchantObjID = Integer.parseInt(mBypass[1]);

            String name = "None Name";

            String html = HtmCache.getInstance().getNotNull(Config.BBS_HOME_DIR + "pages/enchant.htm", activeChar);

            name = ItemHolder.getInstance().getTemplate(enchant_item).getName();

            ItemInstance EhchantItem = activeChar.getInventory().getItemByObjectId(ItemForEchantObjID);



            StringBuilder sb = new StringBuilder("");

            sb.append("Для обычной заточки выбрана вещь:<br1><table width=300>");

            sb.append(new StringBuilder("<tr><td width=32><img src=icon." + EhchantItem.getTemplate().getIcon() + " width=32 height=32> <img src=\"l2ui.squaregray\" width=\"32\" height=\"1\"></td><td width=236><center>"));

            sb.append(new StringBuilder("<font color=\"LEVEL\">" + EhchantItem.getTemplate().getName() + " " + (EhchantItem.getEnchantLevel() <= 0 ? "" : new StringBuilder("</font><br1><font color=3293F3>Заточено на: +" + EhchantItem.getEnchantLevel())) + "</font><br1>"));



            sb.append(new StringBuilder("Заточка производится за: <font color=\"LEVEL\">" + name + "</font>"));

            sb.append("<img src=\"l2ui.squaregray\" width=\"236\" height=\"1\"><center></td>");

            sb.append(new StringBuilder("<td width=32><img src=icon." + EhchantItem.getTemplate().getIcon() + " width=32 height=32> <img src=\"l2ui.squaregray\" width=\"32\" height=\"1\"></td>"));

            sb.append("</tr>");

            sb.append("</table>");

            sb.append("<br>");

            sb.append("<br>");

            sb.append("<table border=0 width=400><tr><td width=200>");

            for(int i = 0; i < enchant_level.length; i++)

            {

                sb.append(new StringBuilder("<button value=\"На +" + enchant_level[i] + " (Цена:" + (EhchantItem.getTemplate().isWeapon() != false ? ench_price_weapon[i] : ench_price_armor[i]) + " " + name + ")\" action=\"bypass _bbsechantgo:" + enchant_level[i] + ":" + (EhchantItem.getTemplate().isWeapon() != false ? ench_price_weapon[i] : ench_price_armor[i]) + ":" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

                //sb.append("<br1>");

            }

            sb.append("</td></tr></table><br1><button value=\"Назад\" action=\"bypass _bbsechant;\" width=70 height=18 back=\"L2UI_ct1.button_df\" fore=\"L2UI_ct1.button_df\">");

            html = html.replace("%enchanter%",  sb.toString());

            html = BbsUtil.htmlBuff(html, activeChar);

            ShowBoard.separateAndSend(html, activeChar);

        }

        if (bypass.startsWith("_bbsechantChus"))

        {

            StringTokenizer st2 = new StringTokenizer(bypass, ";");

            String[] mBypass = st2.nextToken().split(":");

            int ItemForEchantObjID = Integer.parseInt(mBypass[1]);

            String name = "None Name";

            String html = HtmCache.getInstance().getNotNull(Config.BBS_HOME_DIR + "pages/enchant.htm", activeChar);

            name = ItemHolder.getInstance().getTemplate(enchant_item).getName();

            ItemInstance EhchantItem = activeChar.getInventory().getItemByObjectId(ItemForEchantObjID);



            StringBuilder sb = new StringBuilder("");

            sb.append("Для заточки на атрибут выбрана вещь:<br><table width=300>");

            sb.append(new StringBuilder("<tr><td width=32><img src=icon." + EhchantItem.getTemplate().getIcon() + " width=32 height=32> <img src=\"l2ui.squaregray\" width=\"32\" height=\"1\"></td><td width=236><center>"));

            sb.append(new StringBuilder("<font color=\"LEVEL\">" + EhchantItem.getTemplate().getName() + " " + (EhchantItem.getEnchantLevel() <= 0 ? "" : new StringBuilder("</font><br1><font color=3293F3>Заточено на: +" + EhchantItem.getEnchantLevel())) + "</font><br1>"));



            sb.append(new StringBuilder("Заточка производится за: <font color=\"LEVEL\">" + name + "</font>"));

            sb.append("<img src=\"l2ui.squaregray\" width=\"236\" height=\"1\"><center></td>");

            sb.append(new StringBuilder("<td width=32><img src=icon." + EhchantItem.getTemplate().getIcon() + " width=32 height=32> <img src=\"l2ui.squaregray\" width=\"32\" height=\"1\"></td>"));

            sb.append("</tr>");

            sb.append("</table>");

            sb.append("<br>");

            sb.append("<br>");

            sb.append("<table border=0 width=400><tr><td width=200>");

            sb.append("<center><img src=icon.etc_wind_stone_i00 width=32 height=32></center><br>");

            sb.append(new StringBuilder("<button value=\"Wind \" action=\"bypass _bbsechantAtr:2:" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

            sb.append("<br><center><img src=icon.etc_earth_stone_i00 width=32 height=32></center><br>");

            sb.append(new StringBuilder("<button value=\"Earth \" action=\"bypass _bbsechantAtr:3:" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

            sb.append("<br><center><img src=icon.etc_fire_stone_i00 width=32 height=32></center><br>");

            sb.append(new StringBuilder("<button value=\"Fire \" action=\"bypass _bbsechantAtr:0:" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

            sb.append("</td><td width=200>");

            sb.append("<center><img src=icon.etc_water_stone_i00 width=32 height=32></center><br>");

            sb.append(new StringBuilder("<button value=\"Water \" action=\"bypass _bbsechantAtr:1:" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

            sb.append("<br><center><img src=icon.etc_holy_stone_i00 width=32 height=32></center><br>");

            sb.append(new StringBuilder("<button value=\"Divine \" action=\"bypass _bbsechantAtr:4:" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

            sb.append("<br><center><img src=icon.etc_unholy_stone_i00 width=32 height=32></center><br>");

            sb.append(new StringBuilder("<button value=\"Dark \" action=\"bypass _bbsechantAtr:5:" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

            sb.append("</td></tr></table><br1><button value=\"Назад\" action=\"bypass _bbsechant;\" width=70 height=18 back=\"L2UI_ct1.button_df\" fore=\"L2UI_ct1.button_df\">");

            html = html.replace("%enchanter%",  sb.toString());

            html = BbsUtil.htmlBuff(html, activeChar);

            ShowBoard.separateAndSend(html, activeChar);

        }

        if (bypass.startsWith("_bbsechantAtr"))

        {

            StringTokenizer st2 = new StringTokenizer(bypass, ";");

            String[] mBypass = st2.nextToken().split(":");

            int AtributType = Integer.parseInt(mBypass[1]);

            int ItemForEchantObjID = Integer.parseInt(mBypass[2]);

            String ElementName = "";

            if (AtributType == 0)

                ElementName = "Fire";

            else if (AtributType == 1)

                ElementName = "Water";

            else if (AtributType == 2)

                ElementName = "Wind";

            else if (AtributType == 3)

                ElementName = "Earth";

            else if (AtributType == 4)

                ElementName = "Divine";

            else if (AtributType == 5)

                ElementName = "Dark";

            String name = "None Name";

            String html = HtmCache.getInstance().getNotNull(Config.BBS_HOME_DIR + "pages/enchant.htm", activeChar);

            name = ItemHolder.getInstance().getTemplate(enchant_item).getName();

            ItemInstance EhchantItem = activeChar.getInventory().getItemByObjectId(ItemForEchantObjID);

            StringBuilder sb = new StringBuilder("");

            sb.append(new StringBuilder("Выбран элемент: <font color=\"LEVEL\">" + ElementName + "</font><br1> Для заточки выбрана вещь:<br1><table width=300>"));

            sb.append(new StringBuilder("<tr><td width=32><img src=icon." + EhchantItem.getTemplate().getIcon() + " width=32 height=32> <img src=\"l2ui.squaregray\" width=\"32\" height=\"1\"></td><td width=236><center>"));

            sb.append(new StringBuilder("<font color=\"LEVEL\">" + EhchantItem.getTemplate().getName() + " " + (EhchantItem.getEnchantLevel() <= 0 ? "" : new StringBuilder("</font><br1><font color=3293F3>Заточено на: +" + EhchantItem.getEnchantLevel())) + "</font><br1>"));



            sb.append(new StringBuilder("Заточка производится за: <font color=\"LEVEL\">" + name + "</font>"));

            sb.append("<img src=\"l2ui.squaregray\" width=\"236\" height=\"1\"><center></td>");

            sb.append(new StringBuilder("<td width=32><img src=icon." + EhchantItem.getTemplate().getIcon() + " width=32 height=32> <img src=\"l2ui.squaregray\" width=\"32\" height=\"1\"></td>"));

            sb.append("</tr>");

            sb.append("<br1>");

            sb.append("<br1>");

            if (EhchantItem.getTemplate().getCrystalType() == ItemTemplate.Grade.S || EhchantItem.getTemplate().getCrystalType() == ItemTemplate.Grade.S80 || EhchantItem.getTemplate().getCrystalType() == ItemTemplate.Grade.S84)

            {

                sb.append("<table border=0 width=400><tr><td width=200>");

                for(int i = 0; i < (EhchantItem.getTemplate().isWeapon() != false ? atr_lvl_weapon.length : atr_lvl_armor.length); i++)

                {

                    sb.append("<center><button value=\"На +");

                    sb.append(new StringBuilder((EhchantItem.getTemplate().isWeapon() != false ? atr_lvl_weapon[i] : atr_lvl_armor[i]) + " (Цена:" + (EhchantItem.getTemplate().isWeapon() != false ? atr_price_weapon[i] : atr_price_armor[i]) + " " + name + ")\" action=\"bypass _bbsechantuseAtr:" + (EhchantItem.getTemplate().isWeapon() != false ? atr_lvl_weapon[i] : atr_lvl_armor[i]) + ":" + AtributType + ":" + (EhchantItem.getTemplate().isWeapon() != false ? atr_price_weapon[i] : atr_price_armor[i]) + ":" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

                    sb.append("<br1>");

                }

                sb.append("</td></tr></table><br1>");

            }

            else if (EhchantItem.getTemplate().getCrystalType() == ItemTemplate.Grade.S || EhchantItem.getTemplate().getCrystalType() == ItemTemplate.Grade.S80 || EhchantItem.getTemplate().getCrystalType() == ItemTemplate.Grade.S84)

            {

                sb.append("<table border=0 width=400><tr><td width=200>");

                for(int i = 0; i < (EhchantItem.getTemplate().isWeapon() != false ? atr_lvl_weapon.length : atr_lvl_armor.length); i++)

                {

                    sb.append(new StringBuilder("<center><button value=\"На +" + (EhchantItem.getTemplate().isWeapon() != false ? atr_lvl_weapon[i] : atr_lvl_armor[i]) + " (Цена:" + (EhchantItem.getTemplate().isWeapon() != false ? atr_price_weapon[i] : atr_price_armor[i]) + " " + name + ")\" action=\"bypass _bbsechantuseAtr:" + (EhchantItem.getTemplate().isWeapon() != false ? atr_lvl_weapon[i] : atr_lvl_armor[i]) + ":" + AtributType + ":" + (EhchantItem.getTemplate().isWeapon() != false ? atr_price_weapon[i] : atr_price_armor[i]) + ":" + ItemForEchantObjID + ";\" width=200 height=20 back=\"L2UI_CT1.Button_DF\" fore=\"L2UI_CT1.Button_DF\">"));

                    sb.append("<br1>");

                }

                sb.append("</td></tr></table><br1>");

                sb.append("</table>");

            }

            else

            {

                sb.append("<table border=0 width=400><tr><td width=200>");

                sb.append("<br1>");

                sb.append("<br1>");

                sb.append("<br1>");

                sb.append("<br1>");

                sb.append("<center><font color=\"LEVEL\">Заточка данной вещи не возможна!</font></center>");

                sb.append("<br1>");

                sb.append("<br1>");

                sb.append("<br1>");

                sb.append("<br1>");

                sb.append("</td></tr></table><br1>");

            }

            sb.append("<button value=\"Назад\" action=\"bypass _bbsechant;\" width=70 height=18 back=\"L2UI_ct1.button_df\" fore=\"L2UI_ct1.button_df\">");

            html = html.replace("%enchanter%",  sb.toString());

            html = BbsUtil.htmlBuff(html, activeChar);

            ShowBoard.separateAndSend(html, activeChar);

        }

        if (bypass.startsWith("_bbsechantgo"))

        {

            StringTokenizer st2 = new StringTokenizer(bypass, ";");

            String[] mBypass = st2.nextToken().split(":");

            int EchantVal = Integer.parseInt(mBypass[1]);

            int EchantPrice = Integer.parseInt(mBypass[2]);

            int EchantObjID = Integer.parseInt(mBypass[3]);

            ItemTemplate item = ItemHolder.getInstance().getTemplate(enchant_item);

            ItemInstance pay = activeChar.getInventory().getItemByItemId(item.getItemId());

            ItemInstance EhchantItem = activeChar.getInventory().getItemByObjectId(EchantObjID);

            if (pay != null && pay.getCount() >= EchantPrice)

            {

                activeChar.getInventory().destroyItem(pay, EchantPrice);

                EhchantItem.setEnchantLevel(EchantVal);

                activeChar.getInventory().equipItem(EhchantItem);

                activeChar.sendPacket(new InventoryUpdate().addModifiedItem(EhchantItem));

                activeChar.broadcastUserInfo(true);

                activeChar.broadcastCharInfo();

                activeChar.sendMessage(new StringBuilder("" + EhchantItem.getTemplate().getName() + " было заточено до " + EchantVal + ".").toString());

                Log.add(new StringBuilder(activeChar.getName() + " enchant item:" + EhchantItem.getTemplate().getName() + " val: " + EchantVal + "").toString(), "wmzSeller");

                onBypassCommand(activeChar, "_bbsechant");

            }

            else

                activeChar.sendPacket(SystemMsg.INCORRECT_ITEM_COUNT);

        }



        if (bypass.startsWith("_bbsechantuseAtr"))

        {

            StringTokenizer st2 = new StringTokenizer(bypass, ";");

            String[] mBypass = st2.nextToken().split(":");

            int EchantVal = Integer.parseInt(mBypass[1]);

            int AtrType = Integer.parseInt(mBypass[2]);

            int EchantPrice = Integer.parseInt(mBypass[3]);

            int EchantObjID = Integer.parseInt(mBypass[4]);

            ItemTemplate item = ItemHolder.getInstance().getTemplate(enchant_item);

            ItemInstance pay = activeChar.getInventory().getItemByItemId(item.getItemId());

            ItemInstance EhchantItem = activeChar.getInventory().getItemByObjectId(EchantObjID);

            if(EhchantItem.isWeapon()){

                if (pay != null && pay.getCount() >= EchantPrice){

                    activeChar.getInventory().destroyItem(pay, EchantPrice);

                    activeChar.getInventory().unEquipItem(EhchantItem);

                    EhchantItem.setAttributeElement(getAttr(AtrType), EchantVal);

                    activeChar.getInventory().equipItem(EhchantItem);

                    activeChar.sendPacket(new InventoryUpdate().addModifiedItem(EhchantItem));

                    activeChar.broadcastUserInfo(true);

                    activeChar.sendMessage(new StringBuilder("Значение атрибута " + EhchantItem.getTemplate().getName() + " увеличено до " + EchantVal + ".").toString());

                    Log.add(new StringBuilder(activeChar.getName() + " enchant item:" + EhchantItem.getTemplate().getName() + " val: " + EchantVal + " AtributType:" + AtrType).toString(), "wmzSeller");

                    onBypassCommand(activeChar, "_bbsechant");

                }

                else

                    activeChar.sendPacket(SystemMsg.INCORRECT_ITEM_COUNT);

            }

            else if(EhchantItem.isArmor()){

                if(!canEnchantArmorAttribute(AtrType, EhchantItem)){

                   activeChar.sendMessage("Невозможно вставить аттрибут в броню, не соблюдены условия");

                   return;

                }

                if (pay != null && pay.getCount() >= EchantPrice){

                    activeChar.getInventory().destroyItem(pay, EchantPrice);

                    activeChar.getInventory().unEquipItem(EhchantItem);

                    EhchantItem.setAttributeElement(getAttr(AtrType), EchantVal);

                    activeChar.getInventory().equipItem(EhchantItem);

                    activeChar.sendPacket(new InventoryUpdate().addModifiedItem(EhchantItem));

                    activeChar.broadcastUserInfo(true);

                    activeChar.sendMessage(new StringBuilder("Значение атрибута " + EhchantItem.getTemplate().getName() + " увеличено до " + EchantVal + ".").toString());

                    Log.add(new StringBuilder(activeChar.getName() + " enchant item:" + EhchantItem.getTemplate().getName() + " val: " + EchantVal + " AtributType:" + AtrType).toString(), "wmzSeller");

                    onBypassCommand(activeChar, "_bbsechant");

               }

           }

            else{

                if (activeChar.isLangRus())

                    activeChar.sendMessage("В данную вещь нельзя вставить атрибут.");

                 else

                    activeChar.sendMessage("In this thing you can not insert the attribute.");

            }

        }

    }



    public void onWriteCommand(Player player, String bypass, String arg1, String arg2, String arg3, String arg4, String arg5) {

        //To change body of implemented methods use File | Settings | File Templates.

    }



    private boolean canEnchantArmorAttribute(int attr, ItemInstance item){

        switch(attr)

        {

            case 0:

                if(item.getDefenceWater() != 0)

                    return false;

                break;

            case 1:

                if(item.getDefenceFire() != 0)

                    return false;

                break;

            case 2:

                if(item.getDefenceEarth() != 0)

                    return false;

                break;

            case 3:

                if(item.getDefenceWind() != 0)

                    return false;

                break;

            case 4:

                if(item.getDefenceUnholy() != 0)

                    return false;

                break;

            case 5:

                if(item.getDefenceHoly() != 0)

                    return false;

                break;

        }

        return true;

    }



    private Element getAttr(int attr){

        Element El = Element.NONE;

        switch(attr)

        {

            case 0:

                El = Element.FIRE;

                break;

            case 1:

                El = Element.WATER;

                break;

            case 2:

                El = Element.WIND;

                break;

            case 3:

                El = Element.EARTH;

                break;

            case 4:

                El = Element.HOLY;

                break;

            case 5:

                El = Element.UNHOLY;

                break;

        }

        return El;

    }



    private static boolean CheckCondition(Player player){

        if(player == null)

            return false;



        if(player.isDead())

            return false;



        if((player.getPvpFlag() != 0 || player.isInDuel() || player.isInCombat() || player.isAttackingNow()))

        {

            if (player.isLangRus())

                player.sendMessage("Во время боя нельзя использовать данную функцию.");

            else

                player.sendMessage("During combat, you can not use this feature.");

             return false;

        }



        if (player.isInOlympiadMode()) 

        {

            if (player.isLangRus())

                player.sendMessage("Во время Олимпиады нельзя использовать данную функцию.");

            else

                player.sendMessage("During the Olympics you can not use this feature.");

            return false;

        }



        if (!Config.COMMUNITYBOARD_ENCHANT_ENABLED) 

        {

            if (player.isLangRus())

                player.sendMessage("Функция заточки отключена.");

            else

                player.sendMessage("Enchant off function.");

            return false;

        }



        if (player.getTeam() != TeamType.NONE) 

        {

            if (player.isLangRus())

                player.sendMessage("Нельзя использовать заточку во время эвентов.");

            else

                player.sendMessage("You can not use the enchant during Events.");

            return false;

        }

        return true;

    }

        

} 

[KilRoy]

Замечательно, и тот кто писал данное чудо - всех и подставил Переписываем участки кода где имеются манипуляции с получением параметров с байпасса(_ббсЕнчантГо, _ббсЕнчантЮзеАтр). Повторюсь - не нужно доверять клиенту!

[Donatte]

В последних шарах оверов видел уже кэширование байпасов. :confused:

[Archiel]

Описание на 5 с +. Если сервис самописный, то явно же, рукожопие кодера не заставило себя ждать
Прежде всего, нужно учитывать уязвимость байпассов и уже потом, сразу же, приходит мысль о том, что нужно не передавать параметры в строках, а использовать заготовленные варианты, либо же, ПРОВЕРЯТЬ то что происходит на клиенте и на сервере! Нельзя доверять клиенту, клиент - зло

В связи с этим возник вопрос, а какие есть простые и действенные способы проверки параметров в байпасе или какие есть возможности передавать заготовленные варианты? Ну вот, например, есть самописный сервис, где администратор вынужден передавать параметры ItemId и количество.

Я так понимаю, что простейший вариант проверки на стороне сервера это наличие указанного предмета в указанном количестве у игрока в инвентаре. На сколько это действенно? Может ли злоумышленник отправлять ложные данные и об этих параметрах?

Интересуюсь, потому что сам написал уже много различных байпасов, которые, в том числе, иногда сильно параметризированы.

[neonv1]

Такое вроде происходит только в Alt+B, как проверить подручными средствами
Брать l2phx открыть alt+b проверить наличие байпасов... Если они с параметрами, то все оч плохо...

[linliss]

Такое вроде происходит только в Alt+B, как проверить подручными средствами
Брать l2phx открыть alt+b проверить наличие байпасов... Если они с параметрами, то все оч плохо...

Что плохо? Перед показом диалога сохранить байпасы в список и если потом от клиента пришел пакет с байпасом которого нету в списке у серверва, то шлем лесом. Сложно?

[Zubastic]

Самое тупейшее, но действенное решение: создаем в классе персонажа список. При вызове любой html (будь то alt+b или же обычная html) парсим ее (например на этапе отдачи клиенту) и делаем список валидных байпассов. В итоге даже пошариться по мультиселлам не выйдет. Сервер просто кикнет.