+2010-11-30 [colin] 3.7.7cvs8
+
+ * src/action.c
+ Fix bug #2317, 'Action results in "Bad file descriptor"'
+ Introduced in 3.7.6cvs57. In fact the error has been there
+ since ages, and only new thing is the verification of the
+ various functions return code. Let the stuff work as it
+ does, as it seems to do the right thing since ages.
+
2010-11-28 [colin] 3.7.7cvs7
* src/gtk/quicksearch.c
( cvs diff -u -r 1.274.2.318 -r 1.274.2.319 src/mainwindow.c; ) > 3.7.7cvs5.patchset
( cvs diff -u -r 1.204.2.197 -r 1.204.2.198 src/prefs_common.c; ) > 3.7.7cvs6.patchset
( cvs diff -u -r 1.1.2.98 -r 1.1.2.99 src/gtk/quicksearch.c; ) > 3.7.7cvs7.patchset
+( cvs diff -u -r 1.12.2.62 -r 1.12.2.63 src/action.c; ) > 3.7.7cvs8.patchset
r |= close(chld_out[1]);
r |= close(fileno(stderr));
- r |= dup (chld_err[1]);
+ r |= dup (chld_err[1]); /* why does that fail ? */
r |= close(chld_err[0]);
r |= close(chld_err[1]);
if (r != 0)
- g_warning("%s(%d)", strerror(errno), errno);
+ debug_print("%s(%d)", strerror(errno), errno);
}
cmdline[0] = "sh";
if (sync)
r = write(chld_status[1], "1\n", 2);
if (r != 0)
- g_warning("%s(%d)", strerror(errno), errno);
+ debug_print("%s(%d)", strerror(errno), errno);
perror("fork");
_exit(1);
} else { /* Child */
r |= close(chld_status[1]);
if (r != 0)
- g_warning("%s(%d)", strerror(errno), errno);
+ debug_print("%s(%d)", strerror(errno), errno);
}
_exit(0);
}
r = close(chld_in[1]);
child_info->chld_in = -1; /* No more input */
if (r != 0)
- g_warning("%s(%d)", strerror(errno), errno);
+ debug_print("%s(%d)", strerror(errno), errno);
}
return child_info;
r = close(child_info->chld_in);
if (r != 0)
- g_warning("%s(%d)", strerror(errno), errno);
+ debug_print("%s(%d)", strerror(errno), errno);
child_info->chld_in = -1;
debug_print("Input to grand child sent.\n");
}