NACK: [PATCH] acpi: s3/s4: initialize error to zero

[Colin Ian King]

On 04/07/2019 17:33, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Static analysis is warning that error is not initialized; it is
> actually set indirectly but lets be overly cautious and initialize
> it anyhow to squish the static analysis warnings.
> 
> Addresses-Coverity: ("Unintialized scalar variable")
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/s4/s4.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/acpi/s4/s4.c b/src/acpi/s4/s4.c
> index 6de1dd91..34b6f7b5 100644
> --- a/src/acpi/s4/s4.c
> +++ b/src/acpi/s4/s4.c
> @@ -68,7 +68,7 @@ static int s4_init(fwts_framework *fw)
>  static void s4_check_log(fwts_framework *fw,
>  	fwts_list *klog, int *errors, int *oopses, int *warn_ons)
>  {
> -	int error;
> +	int error = 0;
>  	int oops;
>  	int warn_on;
>  
> 
NACK, I made a mistake. Sending V2.