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

Netfilter plugin improvements #576

Closed

Conversation

gcmoreira
Copy link
Contributor

@gcmoreira gcmoreira commented Jan 3, 2019

  • Added LKM lookup, showing the kernel module name to which the hook belongs to.
  • If the module is part of the kernel text, it also resolves the symbol to that specific address. It is showed between square brackets, ie: [selinux_ipv4_forward]
  • All kernel existing protocols were added (unless until kernel v4.20). It now allows to identify for instance IPv6, ARP, BRIDGE (ebtables), etc.
  • Removed hardcoded sizes.
  • Added function to the Linux common API to find LKM module addresses, similar to the one in mac implementation but it also resolve kernel symbols.

See and compare the output:
Old netfilter plugin output
New netfilter plugin output

 * Added LKM lookup, showing the kernel module name to which the hook belongs to.
 * If the module is part of the kernel text, it also resolves the symbol to that specific address. It is showed between square brackets, ie: [selinux_ipv4_forward]
 * All kernel exisiting protocols were added (unless until kernel v4.20). It now allows to identify for instance IPv6, ARP, BRIDGE (ebtables), etc.
 * Fixed wrong number of hooks and protos. Removed hardcoded sizes.
 * Added function to the Linux common API to find LKM module addresses, similar to the one in mac implementation but it also resolve kernel symbols.
@gcmoreira
Copy link
Contributor Author

The changes proposed in the branch are superseded by this other pull request

@gcmoreira gcmoreira closed this Jan 18, 2019
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.

1 participant