Re: [PATCH 1/9] quorumtools: check for quorum_dispatch return code

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




Reviewed-by: Steven Dake <sdake@xxxxxxxxxx>

On 02/07/2012 03:22 AM, Fabio M. Di Nitto wrote:
> From: "Fabio M. Di Nitto" <fdinitto@xxxxxxxxxx>
> 
> Signed-off-by: Fabio M. Di Nitto <fdinitto@xxxxxxxxxx>
> ---
> :100644 100644 4be7fd1... f81fff7... M	tools/corosync-quorumtool.c
>  tools/corosync-quorumtool.c |    6 +++++-
>  1 files changed, 5 insertions(+), 1 deletions(-)
> 
> diff --git a/tools/corosync-quorumtool.c b/tools/corosync-quorumtool.c
> index 4be7fd1..f81fff7 100644
> --- a/tools/corosync-quorumtool.c
> +++ b/tools/corosync-quorumtool.c
> @@ -453,7 +453,11 @@ static int show_nodes(nodeid_format_t nodeid_format, name_format_t name_format)
>  
>  	g_called = 0;
>  	while (g_called == 0) {
> -		quorum_dispatch(q_handle, CS_DISPATCH_ONE);
> +		err = quorum_dispatch(q_handle, CS_DISPATCH_ONE);
> +		if (err != CS_OK) {
> +			fprintf(stderr, "quorum_dispatch FAILED: %d\n", err);
> +			goto err_exit;
> +		}
>  	}
>  
>  	display_nodes_data(nodeid_format, name_format);

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss


[Corosync Project]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Free Online Dating]     [Linux Kernel]     [Linux SCSI]     [XFree86]

Add to Google Powered by Linux