<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 26, 2020 at 4:43 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">To fix unreported results error:<br>
tst_test.c:1036: BROK: Test haven't reported results!<br>
<br>
which also prevents to display kernel commit fix and CVE.<br>
<br>
Fixes: 8ac26843e ("syscalls/request_key04: new test for request_key()<br>
permission check bug")<br>
<br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
Hi,<br>
<br>
obvious fix which I'd normally merge without sending to ML.<br>
But there are 19 tests which use tst_res(TBROK, and at least some of<br>
them are affected by this (i.e. don't unconditionally report any result<br>
before tst_res(TBROK), so we might want to fix it different way.<br></blockquote><div><div class="gmail_default" style="font-size:small">+1 for this fix.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">By the way, do you think it makes sense to add build check parameters for tst_res() too? Just like what we do for tst_brk() in commit 0738e3753c51.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Any thoughts?</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Kind regards,<br>
Petr<br>
<br>
 testcases/kernel/syscalls/request_key/request_key04.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/testcases/kernel/syscalls/request_key/request_key04.c b/testcases/kernel/syscalls/request_key/request_key04.c<br>
index da036bf3a..c125f4261 100644<br>
--- a/testcases/kernel/syscalls/request_key/request_key04.c<br>
+++ b/testcases/kernel/syscalls/request_key/request_key04.c<br>
@@ -65,7 +65,7 @@ static void do_test(void)<br>
        if (TST_ERR == EACCES) {<br>
                tst_res(TPASS, "request_key() failed with EACCES as expected");<br>
        } else {<br>
-               tst_res(TBROK | TTERRNO,<br>
+               tst_res(TFAIL | TTERRNO,<br>
                        "request_key() failed with unexpected error code");<br>
        }<br>
 }<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>