[Patch] Fix exit code of the sensors tool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hallo Andre,

On Tue, 27 Jan 2009 10:00:43 +0100, Andre Prendel wrote:
> This patch fixes the exit code of the sensors tool in error case
> (e.g. No sensors found).
> 
> Error is set correctly, but at exit label exit() returns res instead
> of error.

Good catch, thanks for reporting. Not sure how I missed that.

> --- lm-sensors-dev/prog/sensors/main.c	2009-01-26 22:49:09.000000000 +0100
> +++ my-lm-sensors/prog/sensors/main.c	2009-01-26 22:19:06.000000000 +0100
> @@ -243,7 +243,7 @@
>  
>  int main(int argc, char *argv[])
>  {
> -	int c, res, i, error, do_bus_list;
> +	int c, res, i, error = 0, do_bus_list;
>  	const char *config_file_name = NULL;
>  
>  	struct option long_opts[] =  {
> @@ -344,5 +344,5 @@
>  
>  exit:
>  	sensors_cleanup();
> -	exit(res);
> +	exit(error);
>  }

Applied, thanks.

-- 
Jean Delvare




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux