GlobalNOC / FlowSpaceFirewall

FlowSpace Firewall Application a floodlight based controller allowing multiple controllers to talk to a single switch, but can not interact with each others flow space (hence FlowSpace Firewall)
http://globalnoc.iu.edu/sdn/fsfw.html
Apache License 2.0
14 stars 14 forks source link

OFQueueGetRequest messages not handled properly #115

Closed gth828r closed 9 years ago

gth828r commented 9 years ago

Looking at the code, it appears that ofp_queue_get_config_requests get handled in OFControllerChannelHandler.java by the method processOFQueueGetRequest. I see two issues with the handler:

I am seeing this in the 1.0.5-dev branch. This is non-critical for GENI, because we haven't exposed queues to our experimenters.

The main issue is that the response will violate the principle of least surprise if someone does send a queue get config message, and it will interact poorly with some OF libraries that handle symmetric messages by keying off of the XID.

ajragusa commented 9 years ago

We'll take care of this, its pretty easy to fix.

ajragusa commented 9 years ago

in the globalnoc system this is issue 11110

ajragusa commented 9 years ago

Brayton is looking at this

ajragusa commented 9 years ago

Pull request #122 should take care of this

gth828r commented 9 years ago

I think that the issue described in globalnoc issue 11110 was addressed, but that only described half of the issue mentioned here. The other issue was that OFQueueGetConfigRequests should be responded to by an OFQueueGetConfigReply, according to section 5.3.4 of the OF 1.0 spec. Can you all address that as well? I think it should just be a one-line change.

ajragusa commented 9 years ago

Ok we will take a look at that

ajragusa commented 9 years ago

Ok Pull request #130 should take care of the rest of that