Re: [PATCH 2/2] IPC: Call lib function only when it's possible

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

 



Reviewed-by: Steven Dake <sdake@xxxxxxxxxx>

On 08/09/2012 04:50 AM, Jan Friesse wrote:
> send_ok was incorrectly tested as boolean, even it's errno type
> variable.
> 
> Signed-off-by: Jan Friesse <jfriesse@xxxxxxxxxx>
> ---
>  exec/ipc_glue.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/exec/ipc_glue.c b/exec/ipc_glue.c
> index 23123fa..7986ece 100644
> --- a/exec/ipc_glue.c
> +++ b/exec/ipc_glue.c
> @@ -636,7 +636,7 @@ static int32_t cs_ipcs_msg_process(qb_ipcs_connection_t *c,
>  		res = -ENOBUFS;
>  	}
>  
> -	if (send_ok) {
> +	if (send_ok >= 0) {
>  		corosync_service[service]->lib_engine[request_pt->id].lib_handler_fn(c, request_pt);
>  		res = 0;
>  	}
> 

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


[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux