diff mbox

Fix assertion failure on test timeout

Message ID mvm7f7117my.fsf@hawking.suse.de
State Committed
Commit 67f779f1ead0bf38d1ca0e882e20c7ce0c03c204
Headers

Commit Message

Andreas Schwab Dec. 15, 2016, 4:25 p.m. UTC 
  * support/support_test_main.c (support_test_main): Don't shadow
	file-local variable test_pid.

Comments

Florian Weimer Dec. 15, 2016, 6:34 p.m. UTC | #1 
On 12/15/2016 05:25 PM, Andreas Schwab wrote:
> 	* support/support_test_main.c (support_test_main): Don't shadow
> 	file-local variable test_pid.
>
> diff --git a/support/support_test_main.c b/support/support_test_main.c
> index e185281eaf..0582230aec 100644
> --- a/support/support_test_main.c
> +++ b/support/support_test_main.c
> @@ -307,7 +307,7 @@ support_test_main (int argc, char **argv, const struct test_config *config)
>       - set up the timer
>       - fork and execute the function.  */
>
> -  pid_t test_pid = fork ();
> +  test_pid = fork ();
>    if (test_pid == 0)
>      {
>        /* This is the child.  */

Thanks, looks good to me, and sorry about that.

Florian
diff mbox

Patch

diff --git a/support/support_test_main.c b/support/support_test_main.c
index e185281eaf..0582230aec 100644
--- a/support/support_test_main.c
+++ b/support/support_test_main.c
@@ -307,7 +307,7 @@  support_test_main (int argc, char **argv, const struct test_config *config)
      - set up the timer
      - fork and execute the function.  */
 
-  pid_t test_pid = fork ();
+  test_pid = fork ();
   if (test_pid == 0)
     {
       /* This is the child.  */