Re: [PATCH 2/3] tests/qtest/rtc-test: Remove pointless NULL check

[Thomas Huth <thuth@redhat.com>]

On 03/05/2021 18.55, Peter Maydell wrote:
> In rtc-test.c we know that s is non-NULL because qtest_start()
> will return a non-NULL value, and we assume this when we
> pass s to qtest_irq_intercept_in(). So we can drop the
> initial assignment of NULL and the "if (s)" condition at
> the end of the function.
> 
> Fixes: Coverity CID 1432353
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>   tests/qtest/rtc-test.c | 6 ++----
>   1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/qtest/rtc-test.c b/tests/qtest/rtc-test.c
> index 402ce2c6090..8126ab1bdb8 100644
> --- a/tests/qtest/rtc-test.c
> +++ b/tests/qtest/rtc-test.c
> @@ -686,7 +686,7 @@ static void periodic_timer(void)
>   
>   int main(int argc, char **argv)
>   {
> -    QTestState *s = NULL;
> +    QTestState *s;
>       int ret;
>   
>       g_test_init(&argc, &argv, NULL);
> @@ -712,9 +712,7 @@ int main(int argc, char **argv)
>   
>       ret = g_test_run();
>   
> -    if (s) {
> -        qtest_quit(s);
> -    }
> +    qtest_quit(s);
>   
>       return ret;
>   }
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>