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

[Michael S. Tsirkin]

On Fri, Jan 08, 2016 at 10:05:44AM +0100, Paolo Bonzini wrote:
> 
> 
> 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

Corey, can you look into this please?

> > +        return -1;
> > +    }