[tor-dev] [Patch] common/procmon.c

[Gisle Vanem <gvanem@xxxxxxxxxxxx>]

Hi list!

I've been staring at this file for some time trying to find the elusive
compile-error (MingW-gcc v4.50) on Win_XP). The bug was a missing 
comma (!) and a typo ('err_msg' at line 277 changed to 'err_msg').

I've also changed the format for 'err_code' at line 293 into a "%ld" to suppress
a warning. How did this go unnoticed for ~1 month?

--- Git-latest\src\common\procmon.c       Thu May 26 19:30:24 2011
+++ common/procmon.c   Tue May 31 22:54:14 2011
@@ -252,7 +252,7 @@
    if (!GetExitCodeProcess(procmon->hproc, &exit_code)) {
      char *errmsg = format_win32_error(GetLastError());
      log_warn(procmon->log_domain, "Error \"%s\" occurred while polling "
-               "handle for monitored process %d; assuming it's dead."
+               "handle for monitored process %d; assuming it's dead.",
               errmsg, procmon->pid);
      tor_free(errmsg);
      its_dead_jim = 1;
@@ -287,12 +287,12 @@

      if (!its_dead_jim)
        log_info(procmon->log_domain, "Failed to open handle to monitored "
-                 "process %d, and error code %d (%s) is not 'invalid "
+                 "process %d, and error code %lu (%s) is not 'invalid "
                 "parameter' -- assuming the process is still alive.",
                 procmon->pid,
-                 err_code, err_msg);
+                 err_code, errmsg);

-      tor_free(err_msg);
+      tor_free(errmsg);
    }
  }
#else

-------------

Sorry, no git patch. I'm a git newbie.

Gisle V.


_______________________________________________
tor-dev mailing list
tor-dev@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-dev