surprised coverity caught this.. :)

good for merge

On Sun, 2010-05-16 at 18:40 +1000, Angus Salkeld wrote:
> Signed-off-by: Angus Salkeld <asalk...@redhat.com>
> ---
>  lib/sam.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/sam.c b/lib/sam.c
> index 0ed77ef..a3d1cd0 100644
> --- a/lib/sam.c
> +++ b/lib/sam.c
> @@ -1011,8 +1011,9 @@ static void *hc_callback_thread (void *unused_param)
>               poll_error = poll (&pfds, 1, tmp_time_interval);
>  
>               if (poll_error == 0) {
> -                     sam_hc_send ();
> -                     counter++;
> +                     if (sam_hc_send () == CS_OK) {
> +                             counter++;
> +                     }
>  
>                       if (counter >= 4) {
>                               if (sam_internal_data.hc_callback () != 0) {

_______________________________________________
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to