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

[Driver] Fix failure after first IRP captured #27

Merged
merged 5 commits into from
Mar 23, 2024
Merged

Conversation

hugsy
Copy link
Owner

@hugsy hugsy commented Mar 23, 2024

Fixed by 6b8e9c1

The bug was coming from an unused Status variable initialized to UNSUCCESSFUL that was (almost) always returned in CapturedIrp::CaptureIrpPreHook.

As a consequence the first IRP with input data would simply fail, resulting in the disabling the monitoring of the driver.

@hugsy hugsy added component:driver bug Something isn't working labels Mar 23, 2024
@hugsy hugsy merged commit c95be06 into main Mar 23, 2024
4 checks passed
@hugsy hugsy deleted the fix_single_irp_drop_bug branch March 23, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component:driver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant