From mboxrd@z Thu Jan 1 00:00:00 1970 From: Erik Faye-Lund Subject: [PATCH v4 05/15] mingw: use real pid Date: Mon, 11 Oct 2010 23:50:19 +0200 Message-ID: <1286833829-5116-6-git-send-email-kusmabite@gmail.com> References: <1286833829-5116-1-git-send-email-kusmabite@gmail.com> Cc: msysgit@googlegroups.com, j6t@kdbg.org, avarab@gmail.com, sunshine@sunshineco.com To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Mon Oct 11 23:51:58 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1P5QHl-0006Bj-8S for gcvg-git-2@lo.gmane.org; Mon, 11 Oct 2010 23:51:57 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756569Ab0JKVvs (ORCPT ); Mon, 11 Oct 2010 17:51:48 -0400 Received: from mail-ew0-f46.google.com ([209.85.215.46]:36668 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756491Ab0JKVvr (ORCPT ); Mon, 11 Oct 2010 17:51:47 -0400 Received: by mail-ew0-f46.google.com with SMTP id 20so833577ewy.19 for ; Mon, 11 Oct 2010 14:51:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer:in-reply-to:references; bh=ntsbEOrBoEvRz0bZBy8CXUo5147W4ajnWeZqM3r12Xs=; b=a2qmEPqgYzmf2zesMQHuBPrhznDoZBwZ//N/pYUrSJXrAzCUqhIvbWuXipLo12hGx6 leEJ8eXfU5A2QDTFTKQN3rLNDBmq9iTipYYExClpQmNqkxN8eyycPyu/9Yj+s19AoQzZ 3zeRM5ocLLzj6LA6W59b7Y/FdkG4oi0Ape3YI= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=r3at4EY3sIOi5nIOglR1ui5/SiqSFrZqNdKWYMOLP1WJbmiISiv1/DdaxtowGM0BOn ylp5Y9ddNjzw8QefEuditDNpR8JAuuhDxyWVqirIMBXkyVPRGhFA14jrXm/uZUmi0b++ X522G3sWDPloFE1OSPEDvfk7IwX0RmO94+yJU= Received: by 10.213.112.207 with SMTP id x15mr2037505ebp.67.1286833906911; Mon, 11 Oct 2010 14:51:46 -0700 (PDT) Received: from localhost (cm-84.215.188.225.getinternet.no [84.215.188.225]) by mx.google.com with ESMTPS id v59sm11652173eeh.4.2010.10.11.14.51.45 (version=TLSv1/SSLv3 cipher=RC4-MD5); Mon, 11 Oct 2010 14:51:46 -0700 (PDT) X-Mailer: git-send-email 1.7.3.165.gdfe39.dirty In-Reply-To: <1286833829-5116-1-git-send-email-kusmabite@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: The Windows port have so far been using process handles in place of PID. However, this is not work consistent with what getpid returns. PIDs are system-global identifiers, but process handles are local to a process. Using PIDs instead of process handles allows, for instance, a user to kill a hung process with the Task Manager, something that would have been impossible with process handles. Change the code to use the real PID, and use OpenProcess to get a process-handle. Store the PID and the process handle in a table protected by a critical section, so we can safely close the process handle later. Signed-off-by: Erik Faye-Lund --- compat/mingw.c | 70 +++++++++++++++++++++++++++++++++++++++++++++++++++++++- compat/mingw.h | 10 ++----- 2 files changed, 72 insertions(+), 8 deletions(-) diff --git a/compat/mingw.c b/compat/mingw.c index 701a555..e2e3c54 100644 --- a/compat/mingw.c +++ b/compat/mingw.c @@ -702,6 +702,13 @@ static int env_compare(const void *a, const void *b) return strcasecmp(*ea, *eb); } +struct { + pid_t pid; + HANDLE proc; +} *pinfo; +static int num_pinfo; +CRITICAL_SECTION pinfo_cs; + static pid_t mingw_spawnve_fd(const char *cmd, const char **argv, char **env, const char *dir, int prepend_cmd, int fhin, int fhout, int fherr) @@ -794,7 +801,23 @@ static pid_t mingw_spawnve_fd(const char *cmd, const char **argv, char **env, return -1; } CloseHandle(pi.hThread); - return (pid_t)pi.hProcess; + + /* + * The process ID is the human-readable identifier of the process + * that we want to present in log and error messages. The handle + * is not useful for this purpose. But we cannot close it, either, + * because it is not possible to turn a process ID into a process + * handle after the process terminated. + * Keep the handle in a list for waitpid. + */ + EnterCriticalSection(&pinfo_cs); + num_pinfo++; + pinfo = xrealloc(pinfo, sizeof(*pinfo) * num_pinfo); + pinfo[num_pinfo - 1].pid = pi.dwProcessId; + pinfo[num_pinfo - 1].proc = pi.hProcess; + LeaveCriticalSection(&pinfo_cs); + + return (pid_t)pi.dwProcessId; } static pid_t mingw_spawnve(const char *cmd, const char **argv, char **env, @@ -1518,6 +1541,51 @@ char *getpass(const char *prompt) return strbuf_detach(&buf, NULL); } +pid_t waitpid(pid_t pid, int *status, unsigned options) +{ + HANDLE h = OpenProcess(SYNCHRONIZE | PROCESS_QUERY_INFORMATION, + FALSE, pid); + if (!h) { + errno = ECHILD; + return -1; + } + + if (options == 0) { + int i; + if (WaitForSingleObject(h, INFINITE) != WAIT_OBJECT_0) { + CloseHandle(h); + return 0; + } + + if (status) + GetExitCodeProcess(h, (LPDWORD)status); + + EnterCriticalSection(&pinfo_cs); + + for (i = 0; i < num_pinfo; ++i) + if (pinfo[i].pid == pid) + break; + + if (i < num_pinfo) { + CloseHandle(pinfo[i].proc); + memmove(pinfo + i, pinfo + i + 1, + sizeof(*pinfo) * (num_pinfo - i - 1)); + num_pinfo--; + pinfo = xrealloc(pinfo, + sizeof(*pinfo) * num_pinfo); + } + + LeaveCriticalSection(&pinfo_cs); + + CloseHandle(h); + return pid; + } + CloseHandle(h); + + errno = EINVAL; + return -1; +} + #ifndef NO_MINGW_REPLACE_READDIR /* MinGW readdir implementation to avoid extra lstats for Git */ struct mingw_DIR diff --git a/compat/mingw.h b/compat/mingw.h index a5bde82..7c4eeea 100644 --- a/compat/mingw.h +++ b/compat/mingw.h @@ -140,13 +140,7 @@ static inline int mingw_unlink(const char *pathname) } #define unlink mingw_unlink -static inline pid_t waitpid(pid_t pid, int *status, unsigned options) -{ - if (options == 0) - return _cwait(status, pid, 0); - errno = EINVAL; - return -1; -} +pid_t waitpid(pid_t pid, int *status, unsigned options); #ifndef NO_OPENSSL #include @@ -321,11 +315,13 @@ void free_environ(char **env); static int mingw_main(); \ int main(int argc, const char **argv) \ { \ + extern CRITICAL_SECTION pinfo_cs; \ _fmode = _O_BINARY; \ _setmode(_fileno(stdin), _O_BINARY); \ _setmode(_fileno(stdout), _O_BINARY); \ _setmode(_fileno(stderr), _O_BINARY); \ argv[0] = xstrdup(_pgmptr); \ + InitializeCriticalSection(&pinfo_cs); \ return mingw_main(argc, argv); \ } \ static int mingw_main(c,v) -- 1.7.3.1.199.g72340