diff options
| author | Christopher Faylor <me@cgf.cx> | 2012-07-02 23:55:51 +0400 |
|---|---|---|
| committer | Christopher Faylor <me@cgf.cx> | 2012-07-02 23:55:51 +0400 |
| commit | e9d7f5b3a9c2cac10d720631b40f610be441daa9 (patch) | |
| tree | 5e77762f48e5ec3823eefff73cbc40468a3e7dc1 | |
| parent | fb348d22afc85a2dfba80c3059391e8bc276e9ca (diff) | |
* pinfo.cc (_pinfo::commune_request): Eliminate unneeded assignment found by
Clang.
| -rw-r--r-- | winsup/cygwin/ChangeLog | 5 | ||||
| -rw-r--r-- | winsup/cygwin/pinfo.cc | 2 |
2 files changed, 5 insertions, 2 deletions
diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 611ba0dc4..912063c06 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,5 +1,10 @@ 2012-07-02 Christopher Faylor <me.cygwin2012@cgf.cx> + * pinfo.cc (_pinfo::commune_request): Eliminate unneeded assignment + found by Clang. + +2012-07-02 Christopher Faylor <me.cygwin2012@cgf.cx> + * external.cc (fillout_pinfo): Return NULL rather than 0. (exit_process): Guard against NULL pointer dereference found by Clang. diff --git a/winsup/cygwin/pinfo.cc b/winsup/cygwin/pinfo.cc index c9a7b413e..eb4923e9b 100644 --- a/winsup/cygwin/pinfo.cc +++ b/winsup/cygwin/pinfo.cc @@ -655,7 +655,6 @@ _pinfo::commune_request (__uint32_t code, ...) HANDLE& hp = si._si_commune._si_process_handle; HANDLE& fromthem = si._si_commune._si_read_handle; HANDLE request_sync = NULL; - bool locked = false; res.s = NULL; res.n = 0; @@ -682,7 +681,6 @@ _pinfo::commune_request (__uint32_t code, ...) } va_end (args); - locked = true; char name_buf[MAX_PATH]; request_sync = CreateSemaphore (&sec_none_nih, 0, LONG_MAX, shared_name (name_buf, "commune", myself->pid)); |