News:

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

[Crash] The game crashes when public player manipulates access permissions

Started by Ranran(retired), May 26, 2021, 08:54:32 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Ranran(retired)

Reproducibility: 100% in demo.sve, 0% in new map (immediately after the start)

Reproduction procedure:
(1) Load demo.sve
(2) Open the player list
(3) Switch to public player
(4) Click the access check button in the row of other players


Strictly speaking, the game is stopped by the index out of bounds message of vector_tpl rather than the game crashing.
If you exclude that code, the game won't crash, but it will produce unexpected results.
For example, when a public player withdraws access, the route disappears and many convoys teleport to depots.
I'm guessing that the new map won't crash because this doesn't happen.


And one contradiction is that the new map defaults to allowing normal players access to public players. However, public players are viewed as hostile in demo.sve and no one allows access to public player, but he still remains public. (´;ω;`)
This may not be related to the crash.


I don't know which part of the code is causing the out of index error.

Anyway, I think the necessary measure is to lock the buttons related to it so that it cannot be operated on public player's access.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

jamespetts

Thank you for your report. I believe that I have now fixed this; I should be grateful if you could re-test with the next nightly build.
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.

Ranran(retired)

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