libOpenflow icon indicating copy to clipboard operation
libOpenflow copied to clipboard

Request to fix a typo in flowmod.go

Open vanytsvetkov opened this issue 5 months ago • 0 comments

My apologies, however, are you sure that this code should look exactly like it does?

https://github.com/contiv/libOpenflow/blob/d967b14cc6889aa008ad564b5617abc272c9be15/openflow13/flowmod.go#L99-L102

Perhaps you wanted to write something like:

        binary.BigEndian.PutUint32(bytes[n:], f.OutPort) 
        n += 4 
        binary.BigEndian.PutUint32(bytes[n:], f.OutGroup) 
        n += 4 

Just check the source code of OpenvSwitch to be completely sure.

        ofm->out_port = ofputil_port_to_ofp11(fm->out_port);
        ofm->out_group = htonl(fm->out_group);

vanytsvetkov avatar Sep 09 '24 19:09 vanytsvetkov