Eh, silly mistake. Thanks. Ethan On Fri, Aug 12, 2011 at 12:06, Ben Pfaff <blp at nicira.com> wrote: > In bundle_check(), this: >> + error = nxm_dst_check(nab->dst, ofs, n_bits, flow) || error; > will assign 'error' either 0 or 1, not 0 or an OpenFlow error code. > > Otherwise this looks good to me. >