mcedit leaks file handler if safe save set
Andrew V. Samoilov
kai at cmail.ru
Fri Dec 21 09:31:43 UTC 2001
Hi, Pavel!
There is unpleasant bug in mc's builtin editor since 4.5.55
found by Andrew W. Nosenko <awn at bcs.zp.ua>.
mc_mkstemps use pure open system call to create temporary file
and thus this file handle must be closed by pure close system call.
As far as I understand mc_mkstemps was written for local fs, but:
* it is used in edit/editcmd.c/edit_save_file() to generate filename
for safe save.
* without temporary hack below mc leaks file handlers and
this/these file(s) cannot be unlinked or executed.
BTW, Andrew W. Nosenko reported undelfs is not large file compatible.
There are atol is used in some places for ino_t.
Best regards,
Andrew.
P.S. Andrew, my previous solution was wrong. Roll it back, please,
and use this patch temporarily, please.
P.P.S. I am going on buziness trip for a week, so I probably will be
offline after 16.00 GMT today.
--- editcmd.c Mon Nov 26 13:20:56 2001
+++ /home/sav/src/mc/edit/editcmd.c Fri Dec 21 11:06:36 2001
@@ -201,20 +201,27 @@ int edit_save_file (WEdit * edit, const
if (this_save_mode > 0) {
char *savedir, *slashpos, *saveprefix;
- savedir = (char *) strdup (".");
- slashpos = strrchr (filename, '/');
+ slashpos = strrchr (filename, PATH_SEP);
if (slashpos) {
- free (savedir);
savedir = (char *) strdup (filename);
savedir[slashpos - filename + 1] = '\0';
- }
+ } else
+ savedir = (char *) strdup (".");
saveprefix = concat_dir_and_file (savedir, "cooledit");
free (savedir);
fd = mc_mkstemps(&savename, saveprefix, NULL);
g_free (saveprefix);
if (!savename)
return 0;
+/*
+ * FIXME: mc_mkstemps use pure open system call to create temporary file...
+ * This file handle must be close()d, but there is next line in edit.h:
+ * #define close mc_close
+ * So this hack needed.
+ */
+#undef close
close (fd);
+#define close mc_close
} else
savename = g_strdup (filename);
More information about the mc-devel
mailing list