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

TCP Reassembly #11

Open
de-cix-role opened this issue Oct 12, 2016 · 3 comments
Open

TCP Reassembly #11

de-cix-role opened this issue Oct 12, 2016 · 3 comments
Assignees

Comments

@de-cix-role
Copy link
Collaborator

(new)cdietzel@flowbox:/jbod3/users/cdietzel/blackholing/32_accept$ zcat /some.pcap.gz | pbgpp.py - --filter-message-type UPDATE --filter-next-hop 80.81.193.66
[Message.py:92 - __parse() ] unpack requires a string argument of length 10
[Message.py:92 - __parse() ] unpack requires a string argument of length 10
[Message.py:92 - __parse() ] unpack requires a string argument of length 10

@thannaske
Copy link
Contributor

Error occurs in Message.py in parsing method. Therefore the error message is not related to the UPDATE filter but to a possible bug in the parsing method.

@thannaske thannaske self-assigned this Oct 13, 2016
@thannaske thannaske changed the title UPDATE filter seems to be broken TCP Reassembly Oct 19, 2016
@thannaske
Copy link
Contributor

Error occurs while trying to parse large TCP packets that are containing several BGP messages. Performing a TCP reassembly that is able to rebuild the segmented payloads into a single one is the solution for this kind of problem.

@kurtrwall
Copy link

First off, I consider this library to be a holy grail and I applaud the folks that work on this project.

Is there anyone that can tell me the impact that this limitation has? Do large packets come through frequently enough for this issue to warrant re-adding the "high priority" label? Would love to help work out the issues with it. Any help and/or attention on this is appreciated. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants