#include <ccan/noerr/noerr.h>
#include <ccan/read_write_all/read_write_all.h>
#include <ccan/noerr/noerr.h>
+#include <ccan/time/time.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <sys/time.h>
#include <err.h>
#include <unistd.h>
#include <assert.h>
+#include <signal.h>
#include "tools.h"
-static char *tmpdir = NULL;
+static const char *tmpdir = NULL;
bool tools_verbose = false;
/* Ten minutes. */
int p[2];
char *ret;
int status, ms;
- struct timeval start, end;
+ struct timeval start;
*ok = false;
if (pipe(p) != 0)
if (tools_verbose)
printf("Running: %s\n", cmd);
- gettimeofday(&start, NULL);
+ /* Always flush buffers before fork! */
+ fflush(stdout);
+ start = time_now();
pid = fork();
if (pid == -1) {
close_noerr(p[0]);
close_noerr(p[1]);
return talloc_asprintf(ctx, "Failed to fork: %s",
strerror(errno));
- return NULL;
}
if (pid == 0) {
signal(SIGALRM, killme);
itim.it_interval.tv_sec = itim.it_interval.tv_usec = 0;
- itim.it_value.tv_sec = *timeout_ms / 1000;
- itim.it_value.tv_usec = (*timeout_ms % 1000) * 1000;
+ itim.it_value = time_from_msec(*timeout_ms);
setitimer(ITIMER_REAL, &itim, NULL);
status = system(cmd);
if (waitpid(pid, &status, 0) != pid)
err(1, "Failed to wait for child");
- gettimeofday(&end, NULL);
- if (end.tv_usec < start.tv_usec) {
- end.tv_usec += 1000000;
- end.tv_sec--;
- }
- ms = (end.tv_sec - start.tv_sec) * 1000
- + (end.tv_usec - start.tv_usec) / 1000;
+ ms = time_to_msec(time_sub(time_now(), start));
if (ms > *timeout_ms)
*timeout_ms = 0;
else
*timeout_ms -= ms;
-
+ close(p[0]);
if (tools_verbose) {
printf("%s", ret);
printf("Finished: %u ms, %s %u\n", ms,
return ret;
}
-/* Returns output if command fails. */
-char *run_command(const void *ctx, unsigned int *time_ms, const char *fmt, ...)
+/* Tallocs *output off ctx; return false if command fails. */
+bool run_command(const void *ctx, unsigned int *time_ms, char **output,
+ const char *fmt, ...)
{
va_list ap;
- char *cmd, *contents;
+ char *cmd;
bool ok;
unsigned int default_time = default_timeout_ms;
if (!time_ms)
time_ms = &default_time;
- else if (*time_ms == 0)
- return talloc_strdup(ctx, "\n== TIMED OUT ==\n");
+ else if (*time_ms == 0) {
+ *output = talloc_strdup(ctx, "\n== TIMED OUT ==\n");
+ return false;
+ }
va_start(ap, fmt);
cmd = talloc_vasprintf(ctx, fmt, ap);
va_end(ap);
- contents = run_with_timeout(ctx, cmd, &ok, time_ms);
- if (ok) {
- talloc_free(contents);
- return NULL;
- }
-
- if (!contents)
+ *output = run_with_timeout(ctx, cmd, &ok, time_ms);
+ if (ok)
+ return true;
+ if (!*output)
err(1, "Problem running child");
if (*time_ms == 0)
- contents = talloc_asprintf_append(contents,
- "\n== TIMED OUT ==\n");
- return contents;
+ *output = talloc_asprintf_append(*output,
+ "\n== TIMED OUT ==\n");
+ return false;
}
-static int unlink_all(char *dir)
+static int unlink_all(const char *dir)
{
char cmd[strlen(dir) + sizeof("rm -rf ")];
sprintf(cmd, "rm -rf %s", dir);
return 0;
}
-char *temp_dir(const void *ctx)
+const char *temp_dir(const void *ctx)
{
/* For first call, create dir. */
while (!tmpdir) {
return tmpdir;
}
-char *maybe_temp_file(const void *ctx, const char *extension, bool keep,
- const char *srcname)
+int unlink_file_destructor(char *filename)
{
- size_t baselen;
+ unlink(filename);
+ return 0;
+}
+
+char *temp_file(const void *ctx, const char *extension, const char *srcname)
+{
+ unsigned baselen;
char *f, *suffix = talloc_strdup(ctx, "");
struct stat st;
unsigned int count = 0;
- if (!keep)
- srcname = talloc_basename(ctx, srcname);
- else
- assert(srcname[0] == '/');
-
+ srcname = talloc_basename(ctx, srcname);
if (strrchr(srcname, '.'))
baselen = strrchr(srcname, '.') - srcname;
else
do {
f = talloc_asprintf(ctx, "%s/%.*s%s%s",
- keep ? "" : temp_dir(ctx),
+ temp_dir(ctx),
baselen, srcname,
suffix, extension);
talloc_free(suffix);