> -----Original Message----- > From: Hal Rosenstock [mailto:hal@xxxxxxxxxxxxxxxxxx] > Sent: Wednesday, November 05, 2014 12:22 PM > To: Weiny, Ira > Cc: linux-rdma (linux-rdma@xxxxxxxxxxxxxxx); Dan Ben-Yosef > Subject: [PATCH infiniband-diags] saquery.c: Return proper status from > query_sa_cpi on bad result status > > > When result status was other than IB_SA_MAD_STATUS_SUCCESS, > 0 was being returned rather than EIO as intended. > > Signed-off-by: Hal Rosenstock <hal@xxxxxxxxxxxx> Thanks applied, Ira > --- > diff --git a/src/saquery.c b/src/saquery.c index 0f39064..1af3401 100644 > --- a/src/saquery.c > +++ b/src/saquery.c > @@ -1445,7 +1445,7 @@ static int query_sa_cpi(struct sa_handle *h, struct > query_params *query_params) > memcpy(&query_params->cpi, cpi, sizeof(query_params->cpi)); > Exit: > sa_free_result_mad(&result); > - return (0); > + return ret; > } > > static const struct query_cmd query_cmds[] = { -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
