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.

Sign up for GitHub

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

Jump to bottom

Refcount redirection close #106

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Refcount redirection close #106

wants to merge 3 commits into from

Commits on Nov 8, 2021

  1. Cleanup warnings

    @kvojacheck
    kvojacheck committed Nov 8, 2021
    Configuration menu
    Copy the full SHA
    a30a1f4 View commit details
    Browse the repository at this point in the history

  2. Add ability to pass process ID to OnClose() 

    Refactor CUsbDkFilterStrategy::OnClose() in a way that the process id
can be passed from the beginning of the call-chain.
This is just a technical change needed for the next commit.
    @kvojacheck
    kvojacheck committed Nov 8, 2021
    Configuration menu
    Copy the full SHA
    a58993c View commit details
    Browse the repository at this point in the history

  3. Stop redirection upon closing the last reference 

    Add reference counting, since the previously used 'PID-based' method
might not work reliably: for example, if the program doing redirection
is forcefully closed from the task manager, the USB redirection could
remain stuck.
After this commit, we are always stopping redirection if the reference
count drops to zero.
    @kvojacheck
    kvojacheck committed Nov 8, 2021
    Configuration menu
    Copy the full SHA
    bc6cae1 View commit details
    Browse the repository at this point in the history