aboutsummaryrefslogtreecommitdiff
path: root/gtk.c
diff options
context:
space:
mode:
authorSimon Tatham <anakin@pobox.com>2018-11-06 18:33:21 +0000
committerSimon Tatham <anakin@pobox.com>2018-11-06 18:37:23 +0000
commitbaed0e3eec513aa148afaea950b43a4f8b8d2734 (patch)
tree50fcc2de3f08ffb977f0c676ffa0667a4002224d /gtk.c
parentf281644b0be91917b7f9f43d7496140173d7ed2e (diff)
downloadpuzzles-baed0e3eec513aa148afaea950b43a4f8b8d2734.zip
puzzles-baed0e3eec513aa148afaea950b43a4f8b8d2734.tar.gz
puzzles-baed0e3eec513aa148afaea950b43a4f8b8d2734.tar.bz2
puzzles-baed0e3eec513aa148afaea950b43a4f8b8d2734.tar.xz
Fix a misuse of errno.
In menu_save_event, we checked ctx.error to see if an errno value had been left in it by the savefile_write callback, but if so, then we were passing the _current_ value of errno to strerror() in place of the saved value in ctx.error. This may well have been benign, but I spotted it in an eyeball review just now and thought I'd better fix it before it bit anyone.
Diffstat (limited to 'gtk.c')
-rw-r--r--gtk.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/gtk.c b/gtk.c
index c3115b5..20a6c0a 100644
--- a/gtk.c
+++ b/gtk.c
@@ -2287,7 +2287,7 @@ static void menu_save_event(GtkMenuItem *menuitem, gpointer data)
if (ctx.error) {
char boxmsg[512];
sprintf(boxmsg, "Error writing save file: %.400s",
- strerror(errno));
+ strerror(ctx.error));
error_box(fe->window, boxmsg);
goto free_and_return;
}