File descriptor leak #31

Closed
kyprizel opened this Issue Dec 11, 2012 · 4 comments

Comments

Projects
None yet
3 participants
@kyprizel

There is a file descriptor leak in core.c: logger_api_vlogmsg.
Possible patch: https://gist.github.com/4258758

@kyprizel

This comment has been minimized.

Show comment Hide comment
@kyprizel

kyprizel Dec 11, 2012

Looks like my patch doesn't fix the problem.

Looks like my patch doesn't fix the problem.

@ironbee

This comment has been minimized.

Show comment Hide comment
@ironbee

ironbee Dec 11, 2012

Owner

On Tue, Dec 11, 2012 at 9:04 AM, Eldar Zaitov notifications@github.comwrote:

Looks like my patch doesn't fix the problem.

I'll try to look into the problem as soon as I can. Thanks for reporting
it!

-Nick

Owner

ironbee commented Dec 11, 2012

On Tue, Dec 11, 2012 at 9:04 AM, Eldar Zaitov notifications@github.comwrote:

Looks like my patch doesn't fix the problem.

I'll try to look into the problem as soon as I can. Thanks for reporting
it!

-Nick

@ghost ghost assigned nickleroy Dec 11, 2012

@ironbee

This comment has been minimized.

Show comment Hide comment
@ironbee

ironbee Dec 11, 2012

Owner

Eldar,

I'll try to look into the problem as soon as I can. Thanks for reporting
it!

I did look into this a bit. Unless this is a serious current problem
for you, I wouldn't spend too much time looking into this... The
relevant code is being significantly reworked in a branch that I hope
to merge in before 0.7.

-Nick

Owner

ironbee commented Dec 11, 2012

Eldar,

I'll try to look into the problem as soon as I can. Thanks for reporting
it!

I did look into this a bit. Unless this is a serious current problem
for you, I wouldn't spend too much time looking into this... The
relevant code is being significantly reworked in a branch that I hope
to merge in before 0.7.

-Nick

@kyprizel

This comment has been minimized.

Show comment Hide comment
@kyprizel

kyprizel Dec 12, 2012

no problem, thank you.

no problem, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment