News:

Simutrans Tools
Know our tools that can help you to create add-ons, install and customize Simutrans.

Crash after convoy withdraw or sell

Started by michal., January 29, 2022, 09:21:04 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

michal.

Hello,

I've encountered a weird bug that crashes the game immediately after selling or withdrawing some vehicles.

Reproduction
1) Load attached game: https://ufile.io/17gd08c8
2) Go to convoy 2080 at 377:3119 and click on "Sell now" or "Withdraw" button
3) The game crashes

The crash happens also after withdrawing fly-boat convoy 2949 at 307:2331 and some other vehicles. Some convoys are affected, while other can be withdrawn without any problems. I'm not aware of anything suspicious that could have caused the bug.

Unfortunately, I have not been able to reproduce the bug in minimal game.

I've tested the most recent Windows and Linux versions with the same outcome.
Below is the relevant debug message. Afterwards, the program stops responding and SIGSEGV error is printed out...

Debug: convoi::dump():
vehicle_count = 5
wait_lock = 0
owner_n = 0
akt_speed = 89
sp_soll = 1489
state = 6
statename = DRIVING
alte_direction = 2
jahresgewinn = -139
name = '(2080) Pack horses (long goods)'
line_id = '188'
schedule = '0x56518a46bcb0'


I would be very happy to help with anything.

Ranran(retired)

#1
Thank you for reporting the bug.
I can certainly recreate the crash.
I suppose it may be related to the incorporation of r10351 and 10352.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

Ranran(retired)

In the standard version, the schedule editing function was included in the convoy info, but not in the extended version. I think I was adding unnecessary processing.
Now I think I have fixed it. Please check pull request #483.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

jamespetts

Excellent, thank you. Fix now incorporated.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

michal.

Thank you very much, Ranran and James! I can confirm that withdrawing works well in today's version.

jamespetts

Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.