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

nvl leak in zfs/fm.c #2158

Closed
thegreatgazoo opened this issue Mar 3, 2014 · 2 comments
Closed

nvl leak in zfs/fm.c #2158

thegreatgazoo opened this issue Mar 3, 2014 · 2 comments
Labels
Component: ZED ZFS Event Daemon
Milestone
0.6.4

Comments

@thegreatgazoo
Copy link

A couple of memory leaks could happen in some error handling code branches, as mentioned at:
#2085 (diff)
@behlendorf behlendorf added this to the 0.6.3 milestone Mar 20, 2014
@behlendorf behlendorf added the Bug label Mar 20, 2014
@behlendorf
Copy link
Contributor

Thanks we'll review this in the context of the other event work and get it in at the same time.

@behlendorf behlendorf added the zed label Apr 2, 2014
@behlendorf
Copy link
Contributor

@thegreatgazoo I've just merged the outstanding zed code to master. If you could refresh your patch against the latest master code and open a pull request we can get it reviewed and merged cleanly now. Thanks!

behlendorf pushed a commit to behlendorf/zfs that referenced this issue May 1, 2014
@huangheintel @behlendorf
Fixed memory leaks in zevent handling
808d2bf 
Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#2158
@behlendorf behlendorf mentioned this issue May 1, 2014
Closed
@behlendorf behlendorf modified the milestones: 0.6.4, 0.6.3 May 1, 2014
behlendorf pushed a commit to behlendorf/zfs that referenced this issue Aug 13, 2014
@huangheintel @behlendorf
Fixed memory leaks in zevent handling
4871d65 
Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#2158
FransUrbo pushed a commit to FransUrbo/zfs that referenced this issue Aug 17, 2014
@huangheintel @FransUrbo
Fixed memory leaks in zevent handling
53457d9 
Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#2158
FransUrbo pushed a commit to FransUrbo/zfs that referenced this issue Aug 19, 2014
@huangheintel @FransUrbo
=> openzfs#2291: behlendorf/issue-2158 - Fixed memory leaks in zevent…
3d9a659 
… handling.

Fixed memory leaks in zevent handling

Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#2158
DeHackEd pushed a commit to DeHackEd/zfs that referenced this issue Sep 18, 2014
@huangheintel
Fixed memory leaks in zevent handling
39f6568 
Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#2158
@behlendorf behlendorf mentioned this issue Nov 13, 2014
Closed
ryao pushed a commit to ryao/zfs that referenced this issue Nov 29, 2014
@huangheintel @ryao
Fixed memory leaks in zevent handling
c497512 
Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#2158
behlendorf pushed a commit to behlendorf/zfs that referenced this issue Jun 9, 2015
@huangheintel @behlendorf
Fixed memory leaks in zevent handling
26db2bd 
Some nvlist_t could be leaked in error handling paths.
Also make sure cb argument to zfs_zevent_post() cannnot
be NULL.

Signed-off-by: Isaac Huang <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#2158
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: ZED ZFS Event Daemon
Projects
None yet
Milestone
0.6.4
Development

No branches or pull requests
2 participants
@behlendorf @thegreatgazoo