Bug 1002577

Summary: Add brick operation is causing one of the smbd process in server to crash
Product: [Community] GlusterFS Reporter: Raghavendra Talur <rtalur>
Component: libgfapiAssignee: Raghavendra Talur <rtalur>
Status: CLOSED CURRENTRELEASE QA Contact: Sudhir D <sdharane>
Severity: urgent Docs Contact:
Priority: high    
Version: mainlineCC: crh, gluster-bugs, lmohanty, sbhaloth, sdharane, shaines, spalai
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.5.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1000545 Environment:
Last Closed: 2014-04-17 11:46:44 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On: 1000545    
Bug Blocks:    

Comment 1 Anand Avati 2013-08-29 13:37:25 UTC
REVIEW: http://review.gluster.org/5734 (api: Fill loc.path and loc.gfid before syncop_open.) posted (#1) for review on master by Raghavendra Talur (rtalur)

Comment 2 Anand Avati 2013-08-29 13:41:44 UTC
REVIEW: http://review.gluster.org/5734 (api: Fill loc.path and loc.gfid before syncop_open.) posted (#2) for review on master by Raghavendra Talur (rtalur)

Comment 3 Anand Avati 2013-08-29 20:53:09 UTC
COMMIT: http://review.gluster.org/5734 committed in master by Anand Avati (avati) 
------
commit 2ec4b22bb02f572f74157df5264b247095fece77
Author: Raghavendra Talur <rtalur>
Date:   Thu Aug 29 19:04:40 2013 +0530

    api: Fill loc.path and loc.gfid before syncop_open.
    
    syncop_open was crashing because of NULL dereference.
    This fixes that.
    
    Change-Id: I4bc48fac2a6c0b15c806cbbb4ae45c67891ab7ed
    BUG: 1002577
    Signed-off-by: Raghavendra Talur <rtalur>
    Reviewed-on: http://review.gluster.org/5734
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Anand Avati <avati>

Comment 4 Anand Avati 2013-09-10 00:53:10 UTC
REVIEW: http://review.gluster.org/5877 (api: Fill loc.path and loc.gfid before syncop_open.) posted (#1) for review on release-3.4 by Anand Avati (avati)

Comment 5 Anand Avati 2013-09-10 01:07:38 UTC
REVIEW: http://review.gluster.org/5877 (api: Fill loc.path and loc.gfid before syncop_open.) posted (#2) for review on release-3.4 by Anand Avati (avati)

Comment 6 Anand Avati 2013-09-10 08:20:56 UTC
COMMIT: http://review.gluster.org/5877 committed in release-3.4 by Vijay Bellur (vbellur) 
------
commit 236216cba153cb2c270e63698ecb073601287fd4
Author: Raghavendra Talur <rtalur>
Date:   Thu Aug 29 19:04:40 2013 +0530

    api: Fill loc.path and loc.gfid before syncop_open.
    
    syncop_open was crashing because of NULL dereference.
    This fixes that.
    
    Change-Id: I4bc48fac2a6c0b15c806cbbb4ae45c67891ab7ed
    BUG: 1002577
    Signed-off-by: Raghavendra Talur <rtalur>
    Reviewed-on: http://review.gluster.org/5877
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 7 Niels de Vos 2014-04-17 11:46:44 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.5.0, please reopen this bug report.

glusterfs-3.5.0 has been announced on the Gluster Developers mailinglist [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/6137
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user