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

CSI: fix data race in plugin manager #12553

Merged
merged 1 commit into from
Apr 12, 2022
Merged

CSI: fix data race in plugin manager #12553

merged 1 commit into from
Apr 12, 2022

Commits on Apr 12, 2022

  1. CSI: fix data race in plugin manager

    The plugin manager for CSI hands out instances of a plugin for callers
    that need to mount a volume. The `MounterForPlugin` method accesses
    the internal instances map without a lock, and can be called
    concurrently from outside the plugin manager's main run-loop.
    
    The original commit for the instances map included a warning that it
    needed to be accessed only from the main loop but that comment was
    unfortunately ignored shortly thereafter, so this bug has existed in
    the code for a couple years without being detected until we ran tests
    with `-race` in #12098. Lesson learned here: comments make for lousy
    enforcement of invariants!
    tgross committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    762acf8 View commit details
    Browse the repository at this point in the history