Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assert when clicking on orders button in vehicle window. #3671

Closed
DorpsGek opened this issue Mar 8, 2010 · 2 comments
Closed

Assert when clicking on orders button in vehicle window. #3671

DorpsGek opened this issue Mar 8, 2010 · 2 comments
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Mar 8, 2010

Zuu opened the ticket and wrote:

When I clicked on the orders-button on a train window belonging to AdmiralAI I hit an assert:

"Assertion failed at line 515 of c:\users\leif\programmering\openttd\plain_trunk\src\window_gui.h: widget_index < this->nested_array_size"

I ran a self compiled release build of r19371 with the Visual Studio Debugger attached. As I write this I still have Visual Studio running so I can gather more information if necessary.

Call Stack:
kernel32.dll!7610fbae()
[Frames below may be incorrect and/or missing, no symbols loaded for kernel32.dll]
kernel32.dll!7610fbae()
ntdll.dll!77284e60()
ntdll.dll!7724c6c2()
openttd.exe!CustomAbort(int signal=22) Line 417 C++
openttd.exe!raise(int signum=22) Line 590 + 0x4 bytes C
openttd.exe!abort() Line 71 + 0x7 bytes C

openttd.exe!error(const char * s=0x006d34a8, ...) Line 131 C++
openttd.exe!Window::IsWidgetLowered(unsigned char widget_index=0) Line 516 C++
openttd.exe!OrdersWindow::OnMouseLoop() Line 1265 + 0xb bytes C++
openttd.exe!DecreaseWindowCounters() Line 1322 C++
openttd.exe!InputLoop() Line 2254 C++
openttd.exe!GameLoop() Line 1292 C++
openttd.exe!VideoDriver_Win32::MainLoop() Line 889 C++
openttd.exe!ttd_main(int argc=, char * * argv=) Line 756 C++
openttd.exe!WinMain(HINSTANCE__ * hInstance=0x00400000, HINSTANCE__ * hPrevInstance=0x00000000, char * lpCmdLine=0x00304129, int nCmdShow=1) Line 429 + 0x9 bytes C++
openttd.exe!__tmainCRTStartup() Line 263 + 0x1b bytes C
kernel32.dll!7611d0e9()
ntdll.dll!772619bb()
ntdll.dll!7726198e()

When I stand in IsWidgetLowered and check 'this' it seems like the window is invalid/null.

Within the game I have used the cheat to change company. I've attached a save of the game, though I had been running it for quite long time (IRL hours) before I did hit the assert. CluelessPlus in that savegame is a not yet released version. If it is useful I can upload it.

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/3671
@DorpsGek
Copy link
Member Author

DorpsGek commented Mar 8, 2010

Rubidium wrote:

The crash itself I could not reproduce, but I could reproduce cases where the order window did not remove the buttons when cheating to another company nor was the goto cancelled when changing company. Both are bad and possibly the cause of this crash.


This comment was imported from FlySpray: https://bugs.openttd.org/task/3671#comment7679

@DorpsGek
Copy link
Member Author

Terkhen closed the ticket.

Reason for closing: Fixed

In r19387.


This comment was imported from FlySpray: https://bugs.openttd.org/task/3671

@DorpsGek DorpsGek added component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) bug labels Apr 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

1 participant