fonsinchen opened the ticket and wrote:

When a consist leaves a station for whatever reason Vehicle::CancelReservation is called. This method checks if the consist still has cargo reserved for loading and returns that. In the latter case it also resets the counters for cargo to be unloaded or transferred. The logic is obviously not correct. A vehicle can unload or transfer cargo without loading new cargo. Thus the counters should always be reset when cancelling reservations. Normally this is not a problem but in certain special cases, as most likely observed at http://www.tt-forums.net/viewtopic.php?f=33&t=41992&start=2740# p1076344 , the invalid counters can crash the game later on.

The included patch fixes the problem by always resetting the counters.

Attachments

• counters.diff (0.59 KiB)

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/5543