Re: [LTP] [PATCH 1/2] lib: initialize lib|main_pid to zero in the setup_ipc

[Cyril Hrubis <chrubis@suse.cz>]

Hi!
> I see, and you're right we have to set lib_pid before any real setup.
> 
> Does this change below make sense (based on the v2 I just sent)?
> 
> --- a/lib/tst_test.c
> +++ b/lib/tst_test.c
> @@ -144,6 +144,8 @@ static void setup_ipc(void)
>         }
> 
>         memset(results, 0 , size);
> +
> +       results->lib_pid = getpid();
>  }
> 
>  static void cleanup_ipc(void)
> @@ -1933,7 +1935,6 @@ void tst_run_tcases(int argc, char *argv[],
> struct tst_test *self)
> 
>         do_setup(argc, argv);
> 
> -       results->lib_pid = getpid();
>         tst_enable_oom_protection(results->lib_pid);
> 
>         SAFE_SIGNAL(SIGALRM, alarm_handler);

Yes, this should fix it.

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp