Skip to content

stability fixes#544

Merged
fklassen merged 2 commits intoappneta:Bugs_#536_#537from
GabrielGanne:stab
Mar 6, 2019
Merged

stability fixes#544
fklassen merged 2 commits intoappneta:Bugs_#536_#537from
GabrielGanne:stab

Conversation

@GabrielGanne
Copy link
Copy Markdown
Contributor

Hi,

Those are fix proposal for issues #536 and #537.

However, the #356 reproducer is in big endian for some reason, and when working on little endian, it fails to load because the read capture length is bigger than the maximum.
However both issues really look the same to me and the fixes are localized, so I'm confident about my proposal.
Please review.

Best regards

get_ipv6_next() returns NULL on malformed packets. If that happens
return the last proto that could be read.

This should fix issue appneta#537
get_ipv6_next() returns NULL on malformed packets. If that happens
return the last header that could be read.

This should fix issue appneta#536
@fklassen fklassen changed the base branch from master to Bugs_#536_#537 March 6, 2019 15:14
@fklassen
Copy link
Copy Markdown
Member

fklassen commented Mar 6, 2019

@GabrielGanne thanks for the PR. I am rebasing into a branch for test, documentations, changelog and versioning.

@fklassen fklassen merged commit 9ac9307 into appneta:Bugs_#536_#537 Mar 6, 2019
@GabrielGanne GabrielGanne deleted the stab branch October 14, 2024 11:50
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