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

Process Id #71

Closed
ghost opened this issue Sep 15, 2016 · 7 comments
Closed

Process Id #71

ghost opened this issue Sep 15, 2016 · 7 comments

Comments

@ghost
Copy link

ghost commented Sep 15, 2016

To get the process id it's currently required to execute the very expensive GetTcpTable2 (or similar) function. However, the process id already seems to be available in WFP (https://msdn.microsoft.com/en-us/library/windows/hardware/ff552397(v=vs.85).aspx). Maybe this could be added to PWINDIVERT_ADDRESS?

PS: Thanks for the great lib!

@TechnikEmpire
Copy link

Fyi I found a documentation link showing that it's possible to use a string absolute path to one or more binaries for filtering matching, supported back in Vista. So it should be possible to do that as well. When I have time I'd like to take a crack at one of these features and just do a PR.

@basil00
Copy link
Owner

basil00 commented Sep 16, 2016

Although that data-structure contains a processId field, the process ID is not actually available at the WFP network layer (e.g. FWPM_LAYER_OUTBOUND_IPPACKET_V4, etc.) at which WinDivert runs (i.e. the processId field will just be zero).

@basil00 basil00 closed this as completed Sep 16, 2016
@ghost
Copy link
Author

ghost commented Sep 16, 2016

Thanks for the replies 👍
Please also check here @basil00, there are several ways mentioned to get processId from that layer.

@basil00
Copy link
Owner

basil00 commented Sep 17, 2016

Please also check here @basil00, there are several ways mentioned to get processId from that layer.

No, as the link mentions you still need to get the PID from the ALE layer, and WinDivert does not support this layer for now.

@ghost
Copy link
Author

ghost commented Sep 22, 2016

I see, you are indeed correct, thanks again @basil00!

@dfct
Copy link

dfct commented Feb 9, 2017

@basil00, do you have any plans to support the ALE layer some day? A project of mine has me looking at it now. Cheers

@basil00
Copy link
Owner

basil00 commented Feb 10, 2017

Perhaps one day, but development on WinDivert is currently dormant until #53 is resolved.

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

No branches or pull requests

3 participants