Bug 1297373 - [write-behind] : Write/Append to a full volume causes fuse client to crash
Summary: [write-behind] : Write/Append to a full volume causes fuse client to crash
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: write-behind
Version: mainline
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On: 1297004
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-11 10:42 UTC by Raghavendra G
Modified: 2016-06-16 13:54 UTC (History)
6 users (show)

Fixed In Version: glusterfs-3.8rc2
Doc Type: Bug Fix
Doc Text:
Clone Of: 1297004
Environment:
Last Closed: 2016-06-16 13:54:13 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Vijay Bellur 2016-01-11 10:51:23 UTC
REVIEW: http://review.gluster.org/13213 (performance/write-behind: fix memory corruption) posted (#1) for review on master by Raghavendra G (rgowdapp)

Comment 2 Vijay Bellur 2016-01-12 06:15:45 UTC
REVIEW: http://review.gluster.org/13213 (performance/write-behind: fix memory corruption) posted (#2) for review on master by Raghavendra G (rgowdapp)

Comment 3 Vijay Bellur 2016-01-13 12:14:02 UTC
REVIEW: http://review.gluster.org/13213 (performance/write-behind: fix memory corruption) posted (#3) for review on master by Raghavendra G (rgowdapp)

Comment 4 Vijay Bellur 2016-01-13 13:44:49 UTC
REVIEW: http://review.gluster.org/13213 (performance/write-behind: fix memory corruption) posted (#4) for review on master by Raghavendra G (rgowdapp)

Comment 5 Vijay Bellur 2016-01-14 05:42:17 UTC
COMMIT: http://review.gluster.org/13213 committed in master by Raghavendra G (rgowdapp) 
------
commit 1b9174d35dc113f97fd5cec811633853cef269f3
Author: Raghavendra G <rgowdapp>
Date:   Mon Jan 11 16:17:29 2016 +0530

    performance/write-behind: fix memory corruption
    
    1. while handling short writes, __wb_fulfill_short_write would've freed
       current request before moving on to next in the list if request is not
       big enough to accomodate completed number of bytes. So, make sure to
       save next member before invoking __wb_fulfill_short_write on current
       request. Also handle the case where request is exactly the size of
       remaining completed number of bytes.
    
    2. When write request is unwound because there is a conflicting failed
       liability, make sure its deleted from tempted list. Otherwise there
       will be two unwinds (one as part handling a failed request in
       wb_do_winds and another in wb_do_unwinds).
    
    Change-Id: Id1b54430796b19b956d24b8d99ab0cdd5140e4f6
    BUG: 1297373
    Signed-off-by: Raghavendra G <rgowdapp>
    Reviewed-on: http://review.gluster.org/13213
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Raghavendra Talur <rtalur>

Comment 6 Niels de Vos 2016-06-16 13:54:13 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.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [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://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


Note You need to log in before you can comment on or make changes to this bug.