Bug 762180 (GLUSTER-448)

Summary: server_link_cbk refers to @inode when op_ret is -1
Product: [Community] GlusterFS Reporter: Anand Avati <aavati>
Component: protocolAssignee: Anand Avati <aavati>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: mainlineCC: chrisw, gluster-bugs, vijay
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Anand Avati 2009-12-06 07:30:20 UTC
PATCH: http://patches.gluster.com/patch/2577 in master (protocol/server: fix bad reference to @inode in server_link_cbk)

Comment 1 Anand Avati 2009-12-06 10:10:17 UTC
#0  0x00007f3b9971daae in server_link_cbk (frame=0x7f3b800d8c78, cookie=0x0, this=0xa6fd10, op_ret=-1, op_errno=17, inode=0x0, stbuf=0x0, 
    preparent=0x0, postparent=0x0) at ../../../../../xlators/protocol/server/src/server-protocol.c:1610
	hdr = (gf_hdr_common_t *) 0x7f3b80009770
	rsp = (gf_fop_link_rsp_t *) 0x7f3b800097dc
	state = (server_state_t *) 0x7f3b800c22d0
	gf_errno = 17
	hdrlen = 348
	link_inode = (inode_t *) 0x0
	__FUNCTION__ = "server_link_cbk"
#1  0x00007f3b9972834a in server_link_resume (frame=0x7f3b800d8c78, bound_xl=0xa6e160)
    at ../../../../../xlators/protocol/server/src/server-protocol.c:4763
	state = (server_state_t *) 0x7f3b800c22d0
	op_ret = -1
	op_errno = 17
#2  0x00007f3b9972ed8f in server_resolve_done (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:529
	state = (server_state_t *) 0x7f3b800c22d0
	bound_xl = (xlator_t *) 0xa6e160
#3  0x00007f3b9972ee80 in server_resolve_all (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:564
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	__FUNCTION__ = "server_resolve_all"
#4  0x00007f3b9972e990 in server_resolve_entry (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:372
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	resolve = (server_resolve_t *) 0x7f3b800c23c0
	ret = -1
	loc = (loc_t *) 0x7f3b800c2318
#5  0x00007f3b9972ecf6 in server_resolve (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:500
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	resolve = (server_resolve_t *) 0x7f3b800c23c0
#6  0x00007f3b9972ee5b in server_resolve_all (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:560
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	__FUNCTION__ = "server_resolve_all"
#7  0x00007f3b9972eb98 in server_resolve_inode (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:448
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	resolve = (server_resolve_t *) 0x7f3b800c2340
	ret = 0
	loc = (loc_t *) 0x7f3b800c22f0
#8  0x00007f3b9972ed0e in server_resolve (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:504
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	resolve = (server_resolve_t *) 0x7f3b800c2340
#9  0x00007f3b9972ee0c in server_resolve_all (frame=0x7f3b800d8c78) at ../../../../../xlators/protocol/server/src/server-resolve.c:553
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
	__FUNCTION__ = "server_resolve_all"
#10 0x00007f3b9972ef18 in resolve_and_resume (frame=0x7f3b800d8c78, fn=0x7f3b997280f9 <server_link_resume>)
    at ../../../../../xlators/protocol/server/src/server-resolve.c:586
	state = (server_state_t *) 0x7f3b800c22d0
	this = (xlator_t *) 0xa6fd10
#11 0x00007f3b99728516 in server_link (frame=0x7f3b800d8c78, this=0xa6e160, hdr=0xaf8f60, hdrlen=342, iobuf=0x0)
    at ../../../../../xlators/protocol/server/src/server-protocol.c:4797
	req = (gf_fop_link_req_t *) 0xaf8fcc
	state = (server_state_t *) 0x7f3b800c22d0
	oldpathlen = 81
	newpathlen = 81
	newbaselen = 40
#12 0x00007f3b9972c3de in protocol_server_interpret (this=0xa6fd10, trans=0xab1190, hdr_p=0xaf8f60 "", hdrlen=342, iobuf=0x0)
    at ../../../../../xlators/protocol/server/src/server-protocol.c:6278
	conn = (server_connection_t *) 0xab1ed0
	hdr = (gf_hdr_common_t *) 0xaf8f60
	bound_xl = (xlator_t *) 0xa6e160
	frame = (call_frame_t *) 0x7f3b800d8c78
	peerinfo = (peer_info_t *) 0xab1200
	type = 1
	op = 8
	ret = -1
	__FUNCTION__ = "protocol_server_interpret"
#13 0x00007f3b9972d57b in protocol_server_pollin (this=0xa6fd10, trans=0xab1190)
    at ../../../../../xlators/protocol/server/src/server-protocol.c:6681
	hdr = 0xaf8f60 ""
	hdrlen = 342
	ret = 0
	iobuf = (struct iobuf *) 0x0
#14 0x00007f3b9972d789 in notify (this=0xa6fd10, event=2, data=0xab1190)
    at ../../../../../xlators/protocol/server/src/server-protocol.c:6737
	ret = 0
	trans = (transport_t *) 0xab1190
	peerinfo = (peer_info_t *) 0xab1200
	myinfo = (peer_info_t *) 0xab12f0
	__FUNCTION__ = "notify"
#15 0x00007f3b9af0b3d5 in xlator_notify (xl=0xa6fd10, event=2, data=0xab1190) at ../../../libglusterfs/src/xlator.c:923
	old_THIS = (xlator_t *) 0x7f3b9b14b600
	ret = 0
#16 0x00007f3b9af19c3d in transport_peerproc (trans_data=0xab1190) at ../../../libglusterfs/src/transport.c:414
	trans = (transport_t *) 0xab1190
	msg = (struct transport_msg *) 0xb9d5b0
#17 0x00007f3b9aad8faa in start_thread () from /lib/libpthread.so.0

Comment 2 Anand Avati 2009-12-06 13:41:40 UTC
PATCH: http://patches.gluster.com/patch/2583 in master (protocol/server: more fixes in server_link_cbk)