Re: [Qemu-devel] [PULL 14/55] ipmi: Add a local BMC simulation

[Paolo Bonzini]

On 22/12/2015 17:53, Michael S. Tsirkin wrote:
> +{
> +    if ((netfn & 1) || (netfn > MAX_NETFNS) || (s->netfns[netfn / 2])) {

Off by one.  The check should be >=, not >.

Paolo

> +        return -1;
> +    }