From ee705c1dcbf4576fc15ac5ed46426c859f08e686 Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Mon, 17 Sep 2012 20:07:49 +0000 Subject: * pinfo.cc (pinfo::init): Detect potential race where short block has been retrieved but PID_EXECED flag is not set. --- winsup/cygwin/pinfo.cc | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'winsup/cygwin/pinfo.cc') diff --git a/winsup/cygwin/pinfo.cc b/winsup/cygwin/pinfo.cc index e8d156e04..de3620fdb 100644 --- a/winsup/cygwin/pinfo.cc +++ b/winsup/cygwin/pinfo.cc @@ -299,6 +299,15 @@ pinfo::init (pid_t n, DWORD flag, HANDLE h0) bool created = shloc != SH_JUSTOPEN; + /* Detect situation where a transitional memory block is being retrieved. + If the block has been allocated with PINFO_REDIR_SIZE but not yet + updated with a PID_EXECED state then we'll retry. */ + MEMORY_BASIC_INFORMATION mbi; + if (!created && procinfo->exists () + && VirtualQuery (procinfo, &mbi, sizeof (mbi)) + && mbi.RegionSize < sizeof (_pinfo)) + goto loop; + if (!created && createit && (procinfo->process_state & PID_REAPED)) { memset (procinfo, 0, sizeof (*procinfo)); -- cgit v1.2.3