Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 881062

Summary: Brick processes crash when an attempt to mount a volume is made by an unauthorized client
Product: [Community] GlusterFS Reporter: Krutika Dhananjay <kdhananj>
Component: protocolAssignee: Raghavendra Bhat <rabhat>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: high    
Version: mainlineCC: amarts, gluster-bugs
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.4.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 902188 (view as bug list) Environment:
Last Closed: 2013-07-24 17:34:18 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:
Embargoed:
Bug Depends On:    
Bug Blocks: 902188    
Attachments:
Description Flags
valgrind log file of one of the brick processes none

Description Krutika Dhananjay 2012-11-28 14:37:18 UTC
Created attachment 653566 [details]
valgrind log file of one of the brick processes

Description of problem:

When an unauthorized client tries to mount a volume, the brick processes crash.

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


How reproducible:
Always

Steps to Reproduce:
1. Create a volume.
2. Start the volume.
3. Set 'auth.allow' option to an arbitrary ip address a.b.c.d (say).
4. Mount the volume from machine p.q.r.s (say).
  
Actual results:

1. Mount fails (as expected).
2. Brick processes crash.

Expected results:

Brick processes must NOT crash.

Additional info:

Attached valgrind log file of one of the brick processes that crashed.

Comment 1 Krutika Dhananjay 2012-12-19 11:30:10 UTC
Posted for review.
http://review.gluster.org/#change,4337

Comment 2 Vijay Bellur 2012-12-19 21:19:11 UTC
CHANGE: http://review.gluster.org/4337 (protocol/server: Do not access key after GF_FREE in _delete_auth_opt()) merged in master by Anand Avati (avati)

Comment 3 Krutika Dhananjay 2012-12-20 02:12:58 UTC
Reassigning the bug to Raghavendra Bhat who knows the remaining part of the fix.

Comment 4 Vijay Bellur 2013-01-16 17:52:47 UTC
CHANGE: http://review.gluster.org/4373 (protocol/server: do server_connection_put only if bound_xl is NULL) merged in master by Vijay Bellur (vbellur)