This is same style as used by a function above:

    int process_copy_output(process_info_t *p, int fd) {
      DWORD read;
      char buf[1024];
      char *line, *start;

It seems like another inconsistent rule? (/cc @piscisaureus)