[CONTROLLER-684] Minimum values of SET_TP_SRC and SET_TP_DST are incorrectly set to 1. Created: 13/Aug/14 Updated: 05/May/15 Resolved: 05/May/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | adsal |
| Affects Version/s: | Helium |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Hideyuki Tai | Assignee: | Hideyuki Tai |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 1536 |
| Description |
|
Minimum values of SET_TP_SRC and SET_TP_DST actions are incorrectly set to 1 in SAL (Service Abstraction Layer). Therefore, when I tried to install a flow entry which contains an action "SET_TP_SRC=0", the following error message was output. ERROR o.o.controller.sal.action.Action - Invalid field value assignement. For type: setTpSrc Expected: [0x1-0xffff], Got: 0x0 Because the range of TCP port and UDP port includes 0, the minimum values of SET_TP_SRC and SET_TP_DST actions must be set to 0. What I used to reproduce the issue The latest version of the Base Edition on August 12th, 2014. $ git log -3 --oneline How to reproduce the issue 1. Start up the Base Edition. $ ./run.sh 2. Install a flow entry by using REST API of FlowProgrammer. For example, you can use the REST API by executing curl command on Linux as follows. $ curl --user "admin":"admin" -H "Content-type: application/json" -X PUT \ ,"ingressPort":"2","priority":"501","etherType":"0x800","protocol":"tcp","actions":["SET_TP_SRC=0"] }' You can see the following error message in the OSGi console right after you execute the above command. ERROR o.o.controller.sal.action.Action - Invalid field value assignement. For type: setTpSrc Expected: [0x1-0xffff], Got: 0x0 |
| Comments |
| Comment by Hideyuki Tai [ 13/Aug/14 ] |
|
I've verified that this issue is fixed by the following patch. |
| Comment by Carol Sanders [ 05/May/15 ] |
|
This bug is part of the project to Move all ADSAL associated component bugs to ADSAL. |