opennetworkinglab / flowvisor

FlowVisor - A network hypervisor
Other
164 stars 67 forks source link

NullPointerExceptions in UpdateFlowSpace #249

Closed nikr closed 10 years ago

nikr commented 11 years ago

When UpdateFlowSpace is called without a value for the priority a NullPointerException is thrown.

the problem can be reproduced by sending e.g. the JSON message

{
"method":"update-flowspace",
"id":"update-flowspace887",
"params":[
  { 
  "name":"flowspace_foo",
  "dpid":"00:00:00:00:00:00:00:05",
  "match":{"in_port":2}
  }
],
"jsonrpc":"2.0"
}

to FlowVisor's JSON RPC.

A patch to remedy this is here

alshabib commented 11 years ago

Thanks for the patch and well spotted issue. It'll be applied soon and we'll let you know.

Ali Al-Shabibi (sent from handheld)

On 26 juin 2013, at 22:06, Niklas Rehfeld notifications@github.com wrote:

When UpdateFlowSpace is called without a value for the priority a NullPointerException is thrown.

the problem can be reproduced by sending e.g. the JSON message

{ "method":"update-flowspace", "id":"update-flowspace887", "params":[ { "name":"flowspace_foo", "dpid":"00:00:00:00:00:00:00:05", "match":{"in_port":2} } ], "jsonrpc":"2.0" } to FlowVisor's JSON RPC.

A patch to remedy this is here

— Reply to this email directly or view it on GitHub.

nikr commented 10 years ago

fixed in 30fbc06