[PATCH] nagios: Changed return code on exit call

Andreas Ericsson ae at op5.se
Thu Nov 29 11:06:43 CET 2012


On 11/28/2012 08:15 PM, Ricardo Jose Maraschini wrote:
> 
> From: Ricardo Maraschini <ricardo.maraschini at gmail.com>
> 
> Changed exit return code from ERROR to EXIT_FAILURE in order to avoid
> warnings on compilers that have strict type-value checking rules.
> 

Applied. Thanks. And thanks for providing such an excellently formatted
patch.

> Signed-off-by: Ricardo Maraschini <ricardo.maraschini at opservices.com.br>
> 
> ---
>   base/nagios.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/base/nagios.c b/base/nagios.c
> index e711bb5..a5603bc 100644
> --- a/base/nagios.c
> +++ b/base/nagios.c
> @@ -478,7 +478,7 @@ int main(int argc, char **argv, char **env) {
>   				logit(NSLOG_PROCESS_INFO | NSLOG_RUNTIME_ERROR | NSLOG_CONFIG_ERROR, TRUE, "Failed to drop privileges.  Aborting.");
>   
>   				cleanup();
> -				exit(ERROR);
> +				exit(EXIT_FAILURE);
>   				}
>   
>   #ifdef USE_EVENT_BROKER
> 


-- 
Andreas Ericsson                   andreas.ericsson at op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.

------------------------------------------------------------------------------
Keep yourself connected to Go Parallel: 
VERIFY Test and improve your parallel project with help from experts 
and peers. http://goparallel.sourceforge.net




More information about the Developers mailing list