A way for a new exploit?

[bowser499]

I've attached a map.
Debugged the hell out of it.
But still can't figure out why it fatals.
All I know it's connected with a struct.
And it can read HDD? O_O
Please give me any clues, I'm lost. Sorry for a wall of text.

 

[TriggerHappy]

And it can read HDD? O_O

What do you mean?

But still can't figure out why it fatals.

Well it seems to happen a few seconds into the game, and the game doesn't freeze so I don't think its a problem with dialogs.

Just check all the code that runs at a second or two and add debugging messages.

 

[bowser499]

@TriggerHappy,
The problem is in last trigger, RacePool, in struct method create. I don't know why it fatals even with proper debugging and/or commenting. No effect!

 

[Bannar]

If I'm not mistaken, spamming two dimmensional array syntax is unappropriate when used with jass natives.string array RaceColor [MAX_INDEX_RACES][MAX_INDEX_SUBRACES] is forbidden.

Use TableArray instead.
After commenting out Init function found within RacePool, map no longer aborts wc3 process.

 

[TriggerHappy]

If I'm not mistaken, spamming two dimmension array syntax is unappropriate

JassHelper automatically handles 2-D arrays for you.

Of course it's just simulated, so you will get a reduced array size.

http://www.wc3c.net/vexorian/jasshelpermanual.html#twodarray

Most likely pJASS or JH would throw an error if it was invalid syntax.

 

[Bannar]

        static method create takes nothing returns thistype
            local thistype this = thistype.allocate()
            local integer  i    = 0
            
            loop
                exitwhen i > @16@
                call CreateRaceDialog(Player(i))
                // (...)
                set i = i + 1
            endloop

            // (...)

            return this
        endmethod

Calling any player function with incorrect index range automaticaly aborts wc3 process. Proper range is: 0 - 15.
16th player's index is 15 instead of 16. Note that its 0-indexed based system.

@TH I'd rather use table/table array instead of those 2-D arrays anyway.