Saving a game a second time causes a crash

I think I changed something in the fs class from char* to std::string recently.

I have recompiled from scratch, deleting the build directory and without ASAN.

I save the first time with name 'test'. When i want to save the second time some unrecognized characters are shown in the save dialog. See at the bottom of the attached image.

When i hit save the file is really saved with a bad name:

~/.widelands/save
$:> ls -l
insgesamt 100
-rw-r--r-- 1 kaputtnik users 337 28. Sep 02:10 campvis
-rw-r--r-- 1 kaputtnik users 47087 27. Nov 21:28 test.wgf
-rw-r--r-- 1 kaputtnik users 47283 27. Nov 21:30 ''$'\260''t'$'\323\336''EV.wgf'

Really critical because such a file couldn't be easily deleted.

I can avoid the problem by doing the following change. I used trunk 8508.

=== modified file 'src/wui/game_main_menu_save_game.cc'
--- src/wui/game_main_menu_save_game.cc 2017-11-05 19:59:33 +0000
+++ src/wui/game_main_menu_save_game.cc 2017-11-28 20:50:38 +0000
@@ -132,8 +132,8 @@
        ok_.set_enabled(load_or_save_.table().selections().size() == 1);
        load_or_save_.delete_button()->set_enabled(load_or_save_.has_selection());
        if (load_or_save_.has_selection()) {
- const SavegameData& gamedata = *load_or_save_.entry_selected();
- filename_editbox_.set_text(FileSystem::filename_without_ext(gamedata.filename.c_str()));
+ std::unique_ptr<SavegameData> gamedata = load_or_save_.entry_selected();
+ filename_editbox_.set_text(FileSystem::filename_without_ext(gamedata->filename.c_str()));
        }
 }

#6 looks good to me.

I'm quite busy now to test this. Maybe the next weekend. If someone have more time...

Since this is critical, I'll make a branch. Thanks for your investigation!

Jukka, it would be nice if you could provide a branch if you think you fixed a bug. So it is easier to test by just downloading and compiling, at least for me ;-)

Thanks for your work!

Fixed in build20-rc1