diff options
| author |   Chris Luke <chrisy@flirble.org> | 2016-06-02 11:00:41 -0400 |
|---|---|---|
| committer |   Chris Luke <chris_luke@cable.comcast.com> | 2016-06-03 00:58:17 +0000 |
| commit | b98a3a87a968d9675103e16b6d4b3949955288c0 (patch) | |
| tree | aa52156a5e95a61dcd97ba2bc57abdd1f3f13896 /src/plugins/unittest/interface_test.c | |
| parent | 1f752a3f8ff420aa7d67904c2bf29ca3f2ea10b7 (diff) | |
VPP-91 fix sr tunnel add_del collision checkv16.06-rc2
The add_del function was not properly checking if a tunnel already
existed; instead it was checking if the given tunnel name existed.
If no tunnel name was given it flat out refused to add a tunnel
even though that is optional.
Cleanup the add/del parameter validation to "do what I expect" it
to do:
When adding a tunnel:
- If a "name" is given, it must not exist.
- The "key" is always checked, and must not exist.
When deleting a tunnel:
- If the "name" is given, and it exists, then use it.
- If the "name" is not given, use the "key".
- If the "name" and the "key" are given, then both must point to the
same thing.
Change-Id: I9b48ae0203f9664cf8af0f7dc49bf480ddec10d5
Signed-off-by: Chris Luke <chrisy@flirble.org>
(cherry picked from commit e54436005341800f76a584299ef8bf99e8d66227)
Diffstat (limited to 'src/plugins/unittest/interface_test.c')
0 files changed, 0 insertions, 0 deletions