[openib-general] Re: Bug in recv_handler() -sa_query.c file?

[Roland Dreier]

    Pradeep> Now mad_hdr.status field is declared as __be16. So,
    Pradeep> should the check be

    Pradeep> (mad_recv_wc->recv_buf.mad->mad_hdr.status & 0xff)

    Pradeep> before we return EINVAL?

I don't see why.  For one thing, that would be an endianness bug,
since as you say, the status field is in big-endian order, so the test
would be different depending on whether the host is big- or little-endian.

Also, all 16 bits of the status field should be zero if the request
succeeds.  What value do you see in the status field in the failed
response?  What SM are you using?

 - R.