Bug 1483795 - Crashes when nl-cache is enabled [NEEDINFO]
Summary: Crashes when nl-cache is enabled
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: All
OS: All
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
Depends On:
TreeView+ depends on / blocked
Reported: 2017-08-22 01:39 UTC by Zhang Huan
Modified: 2017-08-22 06:21 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2017-08-22 06:21:07 UTC
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
amukherj: needinfo? (pgurusid)

Attachments (Terms of Use)

Description Zhang Huan 2017-08-22 01:39:11 UTC
Description of problem:
When nl-cache is enabled, glusterfs crash while activating graph.

Version-Release number of selected component (if applicable):

How reproducible:
Every time

Steps to Reproduce:
1. enable nl-cache
2. run glusterfs

Actual results:
glusterfs client crash

Expected results:
glusterfs run without error

Additional info:
This problem is in glusterfs_ctx_tw_get(). When ctx->tw is initialized to call glusterfs_ctx_tw_get, will call GF_REF_GET to add refcount and return the existing ctx->tw pointer. However, return of GF_REF_GET is an integer of ref counter, but incorrectly use as pointer to return.

Comment 1 Zhang Huan 2017-08-22 02:47:21 UTC
I don't have permission to submit it to gerrit. Signing in with GitHub account gave me a result of "forbidden". Thus I put my fix here FYI.


Comment 2 Atin Mukherjee 2017-08-22 05:13:54 UTC
Poornima - Could you have a look at the patch provided by Zhang and port it in gerrit?

Comment 3 Zhang Huan 2017-08-22 06:03:56 UTC
Oops, it is my mistake that this is NOT an issue in master. Sorry for the inconvenience. I will close it.

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