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

Fix cred leak in zpl_fallocate_common #5330

Merged
merged 1 commit into from
Oct 24, 2016
Merged

Conversation

tuxoko
Copy link
Contributor

@tuxoko tuxoko commented Oct 24, 2016

This is caught by kmemleak when running compress_004_pos

Signed-off-by: Chunwei Chen david.chen@osnexus.com

This is caught by kmemleak when running compress_004_pos

Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
@mention-bot
Copy link

@tuxoko, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dechamps, @behlendorf and @ryao to be potential reviewers.

Copy link
Contributor

@behlendorf behlendorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Absolutely right. LGTM.

@dweeezil
Copy link
Contributor

Oops... LGTM.

@behlendorf behlendorf merged commit 9fa4db4 into openzfs:master Oct 24, 2016
tuxoko added a commit to tuxoko/zfs that referenced this pull request Nov 11, 2016
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244 
Closes openzfs#5330
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244
Closes openzfs#5330
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244
Closes openzfs#5330
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244
Closes openzfs#5330
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244
Closes openzfs#5330
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244
Closes openzfs#5330
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes openzfs#5244
Closes openzfs#5330
Requires-builders: style
behlendorf pushed a commit that referenced this pull request Feb 3, 2017
This is caught by kmemleak when running compress_004_pos

Reviewed-by: Tim Chase <tim@chase2k.com>
Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
Closes #5244
Closes #5330
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.

4 participants