<div dir="auto"><div><div data-smartmail="gmail_signature"><br></div><div class="gmail_quote"><div dir="ltr">On Tue, Jan 8, 2019, 21:57 Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi!<br>
> For system(rhel7.6, s390x) without __NR_recvmmsg supported, run<br>
> cve-2016-7117 result in timeout and killed by LTP framework. The<br>
> root reason is tst_syscall break with cleanup() function calling<br>
> in this trace path:<br>
> <br>
>   tst_syscall(__NR_recvmmsg, ...)<br>
>     tst_brk()<br>
>       cleanup()<br>
>         tst_fzsync_pair_cleanup()<br>
>           SAFE_PTHREAD_JOIN(pair->thread_b, NULL);<br>
> <br>
> cve-2016-7117 hung at here to wait for thread_b send_and_close() finishing.<br>
> But thread_b fall into infinite loop because of tst_fzsync_wait_b without<br>
> an extra condition to exit. Eventually, test get timeout error like:<br>
> <br>
>   cve-2016-7117.c:145: CONF: syscall(-1) __NR_recvmmsg not supported<br>
>   Test timeouted, sending SIGKILL!<br>
>   tst_test.c:1125: INFO: If you are running on slow machine, try exporting LTP_TIMEOUT_MUL > 1<br>
>   tst_test.c:1126: BROK: Test killed! (timeout?)<br>
<br>
Looks like the complete solution will be more complex, so what about we<br>
do a simple solution that would make it to the release?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Sure, I'm OK to delay the solution.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
We can change the cve-2016-7117 to check if __NR_recvmmsg() is supported<br>
in the test setup(), then we can avoid this problem to begin with.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Feel free to fix in that, thanks!</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Li Wang </div></div>