X-Git-Url: http://git.ozlabs.org/?p=ccan;a=blobdiff_plain;f=tools%2Ftools.c;h=b78702d03a6d1ff26df2a1d412f3bb2965d40b67;hp=2da37234e2dd034ea8d5dd0f108eadcb8caf86bf;hb=adc90816df194167a588a943ae503a37fec3fb6a;hpb=3612661714e86333ceacca7314959a5ed938dc6a diff --git a/tools/tools.c b/tools/tools.c index 2da37234..b78702d0 100644 --- a/tools/tools.c +++ b/tools/tools.c @@ -1,108 +1,274 @@ -#include -#include +#include +#include +#include +#include +#include +#include +#include +#include +#include #include #include +#include +#include +#include #include #include #include #include -#include +#include +#include +#include #include "tools.h" -static char *tmpdir = NULL; -static unsigned int count; +static const char *tmpdir = NULL; +bool tools_verbose = false; -char *talloc_basename(const void *ctx, const char *dir) -{ - char *p = strrchr(dir, '/'); +/* Ten minutes. */ +const unsigned int default_timeout_ms = 10 * 60 * 1000; - if (!p) - return (char *)dir; - return talloc_strdup(ctx, p+1); +static void killme(int sig) +{ + kill(-getpid(), SIGKILL); } -char *talloc_dirname(const void *ctx, const char *dir) +char *run_with_timeout(const void *ctx, const char *cmd, + bool *ok, unsigned *timeout_ms) { - char *p = strrchr(dir, '/'); + pid_t pid; + int p[2]; + struct rbuf in; + int status, ms; + struct timeabs start; - if (!p) - return talloc_strdup(ctx, "."); - return talloc_strndup(ctx, dir, p - dir); -} + *ok = false; + if (pipe(p) != 0) + return tal_fmt(ctx, "Failed to create pipe: %s", + strerror(errno)); -char *talloc_getcwd(const void *ctx) -{ - unsigned int len; - char *cwd; - - /* *This* is why people hate C. */ - len = 32; - cwd = talloc_array(ctx, char, len); - while (!getcwd(cwd, len)) { - if (errno != ERANGE) { - talloc_free(cwd); - return NULL; - } - cwd = talloc_realloc(ctx, cwd, char, len *= 2); + if (tools_verbose) + printf("Running: %s\n", cmd); + + /* Always flush buffers before fork! */ + fflush(stdout); + start = time_now(); + pid = fork(); + if (pid == -1) { + close_noerr(p[0]); + close_noerr(p[1]); + return tal_fmt(ctx, "Failed to fork: %s", strerror(errno)); } - return cwd; + + if (pid == 0) { + struct itimerval itim; + + if (dup2(p[1], STDOUT_FILENO) != STDOUT_FILENO + || dup2(p[1], STDERR_FILENO) != STDERR_FILENO + || close(p[0]) != 0 + || close(STDIN_FILENO) != 0 + || open("/dev/null", O_RDONLY) != STDIN_FILENO) + exit(128); + + signal(SIGALRM, killme); + itim.it_interval.tv_sec = itim.it_interval.tv_usec = 0; + itim.it_value = timespec_to_timeval(time_from_msec(*timeout_ms).ts); + setitimer(ITIMER_REAL, &itim, NULL); + + status = system(cmd); + if (WIFEXITED(status)) + exit(WEXITSTATUS(status)); + /* Here's a hint... */ + exit(128 + WTERMSIG(status)); + } + + close(p[1]); + rbuf_init(&in, p[0], tal_arr(ctx, char, 4096), 4096); + if (!rbuf_read_str(&in, 0, do_tal_realloc) && errno) + in.buf = tal_free(in.buf); + + /* This shouldn't fail... */ + if (waitpid(pid, &status, 0) != pid) + err(1, "Failed to wait for child"); + + ms = time_to_msec(time_between(time_now(), start)); + if (ms > *timeout_ms) + *timeout_ms = 0; + else + *timeout_ms -= ms; + close(p[0]); + if (tools_verbose) { + printf("%s", in.buf); + printf("Finished: %u ms, %s %u\n", ms, + WIFEXITED(status) ? "exit status" : "killed by signal", + WIFEXITED(status) ? WEXITSTATUS(status) + : WTERMSIG(status)); + } + *ok = (WIFEXITED(status) && WEXITSTATUS(status) == 0); + return in.buf; } -/* Returns output if command fails. */ -char *run_command(const void *ctx, const char *fmt, ...) +/* Tals *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; - FILE *pipe; + char *cmd; + bool ok; + unsigned int default_time = default_timeout_ms; + + if (!time_ms) + time_ms = &default_time; + else if (*time_ms == 0) { + *output = tal_strdup(ctx, "\n== TIMED OUT ==\n"); + return false; + } va_start(ap, fmt); - cmd = talloc_vasprintf(ctx, fmt, ap); + cmd = tal_vfmt(ctx, fmt, ap); va_end(ap); - /* Ensure stderr gets to us too. */ - cmd = talloc_asprintf_append(cmd, " 2>&1"); - - pipe = popen(cmd, "r"); - if (!pipe) - return talloc_asprintf(ctx, "Failed to run '%s'", cmd); - - contents = grab_fd(cmd, fileno(pipe), NULL); - if (pclose(pipe) != 0) - return talloc_asprintf(ctx, "Running '%s':\n%s", - cmd, contents); - - talloc_free(cmd); - return NULL; + *output = run_with_timeout(ctx, cmd, &ok, time_ms); + if (ok) + return true; + if (!*output) + err(1, "Problem running child"); + if (*time_ms == 0) + *output = tal_strcat(ctx, take(*output), "\n== TIMED OUT ==\n"); + return false; } -static int unlink_all(char *dir) +static void unlink_all(const char *dir) { char cmd[strlen(dir) + sizeof("rm -rf ")]; sprintf(cmd, "rm -rf %s", dir); -// system(cmd); - return 0; + if (tools_verbose) + printf("Running: %s\n", cmd); + if (system(cmd) != 0) + warn("Could not remove temporary work in %s", dir); +} + +static pid_t *afree; +static void free_autofree(void) +{ + if (*afree == getpid()) + tal_free(afree); +} + +tal_t *autofree(void) +{ + if (!afree) { + afree = tal(NULL, pid_t); + *afree = getpid(); + atexit(free_autofree); + } + return afree; } -char *temp_file(const void *ctx, const char *extension) +const char *temp_dir(void) { /* For first call, create dir. */ while (!tmpdir) { tmpdir = getenv("TMPDIR"); if (!tmpdir) tmpdir = "/tmp"; - tmpdir = talloc_asprintf(talloc_autofree_context(), - "%s/ccanlint-%u.%lu", - tmpdir, getpid(), random()); + tmpdir = tal_fmt(autofree(), "%s/ccanlint-%u.%lu", + tmpdir, getpid(), random()); if (mkdir(tmpdir, 0700) != 0) { if (errno == EEXIST) { - talloc_free(tmpdir); + tal_free(tmpdir); tmpdir = NULL; continue; } err(1, "mkdir %s failed", tmpdir); } - talloc_set_destructor(tmpdir, unlink_all); + tal_add_destructor(tmpdir, unlink_all); + if (tools_verbose) + printf("Created temporary directory %s\n", tmpdir); } + return tmpdir; +} + +void keep_temp_dir(void) +{ + tal_del_destructor(temp_dir(), unlink_all); +} + +char *temp_file(const void *ctx, const char *extension, const char *srcname) +{ + char *f, *base, *suffix; + struct stat st; + unsigned int count = 0; + + base = path_join(ctx, temp_dir(), take(path_basename(ctx, srcname))); + /* Trim extension. */ + base[path_ext_off(base)] = '\0'; + suffix = tal_strdup(ctx, extension); + + do { + f = tal_strcat(ctx, base, suffix); + suffix = tal_fmt(base, "-%u%s", ++count, extension); + } while (lstat(f, &st) == 0); + + if (tools_verbose) + printf("Creating file %s\n", f); + + tal_free(base); + return f; +} + +bool move_file(const char *oldname, const char *newname) +{ + char *contents; + int fd; + bool ret; - return talloc_asprintf(ctx, "%s/%u%s", tmpdir, count++, extension); + if (tools_verbose) + printf("Moving file %s to %s: ", oldname, newname); + + /* Simple case: rename works. */ + if (rename(oldname, newname) == 0) { + if (tools_verbose) + printf("rename worked\n"); + return true; + } + + /* Try copy and delete: not atomic! */ + contents = grab_file(NULL, oldname); + if (!contents) { + if (tools_verbose) + printf("read failed: %s\n", strerror(errno)); + return false; + } + + fd = open(newname, O_WRONLY|O_CREAT|O_TRUNC, 0666); + if (fd < 0) { + if (tools_verbose) + printf("output open failed: %s\n", strerror(errno)); + ret = false; + goto free; + } + + ret = write_all(fd, contents, tal_count(contents)-1); + if (close(fd) != 0) + ret = false; + + if (ret) { + if (tools_verbose) + printf("copy worked\n"); + unlink(oldname); + } else { + if (tools_verbose) + printf("write/close failed\n"); + unlink(newname); + } + +free: + tal_free(contents); + return ret; +} + +void *do_tal_realloc(void *p, size_t size) +{ + tal_resize((char **)&p, size); + return p; }