Issue information

Issue ID
#7366
Status
Fixed
Severity
None
Started
Angelmelody
Jun 15, 2013 0:51
Last Post
mkbu95
Jun 15, 2013 17:46
Confirmation
N/A

Angelmelody - Jun 15, 2013 0:51

my test script to refine head gear[code=auto:0] prontera,155,177,5 script kjdhfksdjf 100,{ input .@c; for(;.@i<.@c;.@i++) successrefitem 1; close; } [/code]


I was running server on RENEWAL mode and MAX_REFINE setting to 20

,but the successrefitem stop at 21 when I input a over 20 value

I think it might be a bug... ;)

mkbu95 - Jun 15, 2013 6:28

Please update with b745cb176a16e771bf5bbae43a7445cf160d1342 and see if it works. Thanks for reporting.

@malufett/Ind
Is it bad to double check MAX_REFINE limits? It worked here...

Angelmelody - Jun 15, 2013 7:21

After pulled the new commits, My map server crashed.

I try to patch the update(b745cb176a) of script.c by manual

It work for me , now it stop at MAX_REFINE

thanks :)

This post has been edited by Angelmelody on Jun 15, 2013 11:17

malufett - Jun 15, 2013 14:56

[quote name="mkbu95" timestamp="1371277732"]
Please update with b745cb176a16e771bf5bbae43a7445cf160d1342 and see if it works. Thanks for reporting.

@malufett/Ind
Is it bad to double check MAX_REFINE limits? It worked here...[/quote]

I think we should,this is to avoid server exploits specially to those who inject malicious code to scripts..
and also aegis cap it to 20 even iterating successrefine more than 20...

:meow: