> + * Do not call this function on a ofproto_port obtained from > + * ofproto_port_dump_next(): that function retains ownership of the data in the > + * ofproto_port. */ "an ofproto". Otherwise looks good. Ethan