Button stacking *patched* ?

Status
Not open for further replies.
Level 3
Joined
Mar 5, 2010
Messages
14
Seems like importing CommandFunc.txt with negative values doesn't work anymore?

Tryed also in local files and MPQ.

How to stack standart buttons (remove Patrol, Hold Position, etc)

Can someone check this out for me?

edit.
here is my prove screenshot

Before patch
Preload.png


After patch(buttons are back)
wtf.png


Scanning out MPQ right now maybe they changed something.
 
Last edited:
Laaaaaame. Was this patched because it was exploitable for malicious code in some way?

I don't think so. I bet they were adding some bounds checking for stuff as they were revising/reorganizing the code.

Oh shit, this is very bad! This breaks my current project right away ...

This is horrible ...

thank you a lot for reporting . I guess, a hot fix is pretty unlikely? ...

I think a hotfix is likely. Blizz has been adamant about not breaking existing shit, and they've been faithful in that regard. We'll definitely try to send them this bug since a decent number of maps exploit that bug.
 

Dr Super Good

Spell Reviewer
Level 64
Joined
Jan 18, 2005
Messages
27,285
Laaaaaame. Was this patched because it was exploitable for malicious code in some way?
It almost certainly was unsafe. Maybe not exploitable but people were likely using it without knowing all possible consequences. Just because an offset does not cause an immediate crash, does not mean it is harmless and not breaking or causing a crash later on in the session.
I think a hotfix is likely. Blizz has been adamant about not breaking existing shit, and they've been faithful in that regard. We'll definitely try to send them this bug since a decent number of maps exploit that bug.
They still need to implement it in a safe way. It is a new feature that has to be added as it never existed before. Before people were relying on broken mechanics to do what they wanted without any regard of side effects.
 
Level 14
Joined
Oct 18, 2013
Messages
724
Any functionality that they remove needs to be AT THE SAME TIME replaced with a solution. This is silly. They should just break spellbooks hiding abilities now because it's "unsafe".

Perhaos JASS needs to be removed because using it can cause crashes.
 

Dr Super Good

Spell Reviewer
Level 64
Joined
Jan 18, 2005
Messages
27,285
They should just break spellbooks hiding abilities now because it's "unsafe".
Hidden spellbook is almost certainly safe. It does not try to manipulate random memory addresses.

Perhaos JASS needs to be removed because using it can cause crashes.
JASS should be fixed so as not to crash by making the unsafe natives safe. For example Player(16) or any other invalid player number should return null instead of trying to access arbitrary memory addresses.
 
Level 19
Joined
Dec 12, 2010
Messages
2,074
Then why did it crash macs? And some people reported other values caused crashes. Crashes mean something unsafe is happening.
because it macs. on top of innate crashes there's macs, its been like that whole time. wanna crash-proof stuff? pick linux, build it up. pick windows 7, set it up. dont pick mac/win10 if you wanna play oldschool games, is htat simple?
most likely game tried to render button somewhere out of screen and that turned out to be bad idea. thing is, it worked 100% flawlessly with some given x/y values.
 

Dr Super Good

Spell Reviewer
Level 64
Joined
Jan 18, 2005
Messages
27,285
wanna crash-proof stuff? pick linux
...

dont pick mac/win10 if you wanna play oldschool games
Windows 10 works well for most well written old school games. It even ran WC3 before all the patches, just with gamma control disabled and some performance problems as a result of the deprecated graphic pipeline used.

It is mostly games which relied on DRM that Windows 10 has problems with.
 
Status
Not open for further replies.
Top