[Gluster-devel] fd not being refed in syncop_opendir_cbk

Anand Avati anand.avati at gmail.com
Wed Jan 23 18:30:19 UTC 2013


On Wed, Jan 23, 2013 at 3:08 AM, Raghavendra Bhat <rabhat at redhat.com> wrote:

>
> Hi,
>
> syncop_opendir_cbk does not ref the fd while returning unlike
> syncop_open_cbk which results in crash due to fd being destroyed early and
> then being accessed (bug 885008). A patch has been submitted for review. (
> http://review.gluster.org/#change,4282). Please review it and provide
> inputs.
>
>
Doing the extra fd_ref() in syncop_open[dir]_cbk seems to be uglier than
thought. I see that it is already resulting in fd ref leak in gfapi. It is
safer to remove the extra ref/unref and make open[dir]() return only
success or failure with op_ret.

Avati
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://supercolony.gluster.org/pipermail/gluster-devel/attachments/20130123/321b428e/attachment-0001.html>


More information about the Gluster-devel mailing list