diff options
| author | Sumit Bose <sbose@redhat.com> | 2009-05-20 12:08:13 +0200 |
|---|---|---|
| committer | Ronnie Sahlberg <ronniesahlberg@gmail.com> | 2009-05-21 11:22:21 +1000 |
| commit | 2fcedf6dac9dd275adbb0017e53547614df29d93 (patch) | |
| tree | 94a290c1514cd2446139e31a6403466018355f73 /ctdb/ib/ibwrapper.c | |
| parent | 11988fc77a06c50c251acec1dc8cc2c275f68310 (diff) | |
| download | samba-2fcedf6dac9dd275adbb0017e53547614df29d93.tar.gz samba-2fcedf6dac9dd275adbb0017e53547614df29d93.tar.xz samba-2fcedf6dac9dd275adbb0017e53547614df29d93.zip | |
add missing checks on so far ignored return values
Most of these were found during a review by Jim Meyering <meyering@redhat.com>
(This used to be ctdb commit 3aee5ee1deb4a19be3bd3a4ce3abbe09de763344)
Diffstat (limited to 'ctdb/ib/ibwrapper.c')
| -rw-r--r-- | ctdb/ib/ibwrapper.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ctdb/ib/ibwrapper.c b/ctdb/ib/ibwrapper.c index 5a166e5353..6a557cbb4d 100644 --- a/ctdb/ib/ibwrapper.c +++ b/ctdb/ib/ibwrapper.c @@ -852,7 +852,7 @@ static int ibw_wc_recv(struct ibw_conn *conn, struct ibv_wc *wc) } if (part->to_read==0) { - pctx->receive_func(conn, part->buf, part->len); + if(pctx->receive_func(conn, part->buf, part->len)) goto error; part->len = 0; /* tells not having partial data (any more) */ if (ibw_wc_mem_threshold(pconn, part, pctx->opts.recv_threshold)) goto error; @@ -867,7 +867,7 @@ static int ibw_wc_recv(struct ibw_conn *conn, struct ibv_wc *wc) /* mostly awaited case: */ if (msglen<=remain) { - pctx->receive_func(conn, p, msglen); + if(pctx->receive_func(conn, p, msglen)) goto error; p += msglen; remain -= msglen; } else { |