Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

neutron2snabb updates related to L2TPv3 config features #283

Merged
merged 3 commits into from
Oct 22, 2014

Conversation

eugeneia
Copy link
Member

@eugeneia eugeneia commented Oct 9, 2014

Updates neutron2snabb for new requirements. See bottom of Neutron API Extensions.

Also updates ipv6:pton' to use inet_pton(3).

error("invalid ipv6 address "..p.." "..v)
end
i = i+1
local in_addr = ffi.new("uint8_t[4]")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Array should be 16 bytes for ipv6

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.Good cach, fixed.

@eugeneia eugeneia force-pushed the neutron2snabb-updates branch 4 times, most recently from 64342c3 to 65f4363 Compare October 10, 2014 17:53

-- Convert hexadecimal str to string of n bytes padded with null bytes at
-- the beginning. N defaults to `hexstr:len() / 2'.
function undump (str, n)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proposal.

@eugeneia eugeneia force-pushed the neutron2snabb-updates branch 2 times, most recently from a229166 to f712ad9 Compare October 14, 2014 12:14
)
config.local_cookie = lib.hexundump(config.local_cookie, 8)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using hexundump. Throw error if part of input would be ignored (e.g. >16 hex digits.).

Note: hexundump pads from the right, as opposed to what we talked about. Doesn't matter though right?

@eugeneia eugeneia force-pushed the neutron2snabb-updates branch from f712ad9 to 4207e2c Compare October 15, 2014 15:13
lukego added a commit that referenced this pull request Oct 22, 2014
neutron2snabb updates related to L2TPv3 config features
@lukego lukego merged commit d0248ed into snabbco:master Oct 22, 2014
@eugeneia eugeneia deleted the neutron2snabb-updates branch May 20, 2015 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants