widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-1738553-zip-filesystem-overflow into lp:widelands

To: mp+339493@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Sun, 25 Feb 2018 18:31:25 -0000
In-reply-to: <151955828206.29828.8941253047284372206.launchpad@ackee.canonical.com>
Reply-to: mp+339493@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

I think I ended up with corrupt savegames because I killed Widelands. I don't think that it's a problem with the actual saving routine.

An additional check can't hurt, but that will probably be quite a lot of new code.

And unless you're using ASAN, this branch doesn't appear to change anything.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1738553-zip-filesystem-overflow/+merge/339493
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1738553-zip-filesystem-overflow into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/bug-1738553-zip-filesystem-overflow into lp:widelands
From: GunChleoc, 2018-02-25