* fhandler_console.cc (fhandler_console::fixup_after_fork_exec): Make error
message more explicit. * pinfo.cc (_pinfo::commune_request): Don't lock process unless we're looking for fifos.
This commit is contained in:
		| @@ -1,3 +1,10 @@ | |||||||
|  | 2006-03-23  Christopher Faylor  <cgf@timesys.com> | ||||||
|  |  | ||||||
|  | 	* fhandler_console.cc (fhandler_console::fixup_after_fork_exec): Make | ||||||
|  | 	error message more explicit. | ||||||
|  | 	* pinfo.cc (_pinfo::commune_request): Don't lock process unless we're | ||||||
|  | 	looking for fifos. | ||||||
|  |  | ||||||
| 2006-03-23  Christopher Faylor  <cgf@timesys.com> | 2006-03-23  Christopher Faylor  <cgf@timesys.com> | ||||||
|  |  | ||||||
| 	* dcrt0.cc (child_info_spawn::handle_spawn): Don't initialize the | 	* dcrt0.cc (child_info_spawn::handle_spawn): Don't initialize the | ||||||
|   | |||||||
| @@ -1804,12 +1804,12 @@ fhandler_console::fixup_after_fork_exec (bool execing) | |||||||
|       bool sawerr = false; |       bool sawerr = false; | ||||||
|       if (!get_io_handle ()) |       if (!get_io_handle ()) | ||||||
| 	{ | 	{ | ||||||
| 	  system_printf ("error opening input console handle after fork/exec, errno %d, %E", get_errno ()); | 	  system_printf ("error opening input console handle for %s after fork/exec, errno %d, %E", get_name (), get_errno ()); | ||||||
| 	  sawerr = true; | 	  sawerr = true; | ||||||
| 	} | 	} | ||||||
|       if (!get_output_handle ()) |       if (!get_output_handle ()) | ||||||
| 	{ | 	{ | ||||||
| 	  system_printf ("error opening output console handle after fork/exec, errno %d, %E", get_errno ()); | 	  system_printf ("error opening output console handle for %s after fork/exec, errno %d, %E", get_name (), get_errno ()); | ||||||
| 	  sawerr = true; | 	  sawerr = true; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
|   | |||||||
| @@ -603,7 +603,6 @@ _pinfo::commune_request (__uint32_t code, ...) | |||||||
|     break; |     break; | ||||||
|     } |     } | ||||||
|  |  | ||||||
|   lock_process now (); |  | ||||||
|   locked = true; |   locked = true; | ||||||
|   char name_buf[CYG_MAX_PATH]; |   char name_buf[CYG_MAX_PATH]; | ||||||
|   request_sync = CreateSemaphore (&sec_none_nih, 0, LONG_MAX, |   request_sync = CreateSemaphore (&sec_none_nih, 0, LONG_MAX, | ||||||
| @@ -652,6 +651,7 @@ _pinfo::commune_request (__uint32_t code, ...) | |||||||
|       break; |       break; | ||||||
|     case PICOM_FIFO: |     case PICOM_FIFO: | ||||||
|       { |       { | ||||||
|  | 	lock_process now (); | ||||||
| 	DWORD x = ReadFile (fromthem, res.handles, sizeof (res.handles), &nr, NULL); | 	DWORD x = ReadFile (fromthem, res.handles, sizeof (res.handles), &nr, NULL); | ||||||
| 	if (!x || nr != sizeof (res.handles)) | 	if (!x || nr != sizeof (res.handles)) | ||||||
| 	  { | 	  { | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user