Skip to content

Fix: xml: Fix segfault in TRACKING_CHANGES() #448

Merged
merged 1 commit into from Feb 28, 2014

2 participants

@gao-yan
Cluster Labs member
gao-yan commented Feb 28, 2014

No description provided.

@beekhof
Cluster Labs member
beekhof commented Feb 28, 2014

Curious to know where/how you hit this...

@gao-yan
Cluster Labs member
gao-yan commented Feb 28, 2014

An user reported that. When he was using mgmtd, which was doing lrmd_handshake():

#0 0x00007f4a4f315ab3 in crm_xml_add (node=node@entry=0x230c930, name=name@entry=0x7f4a4ffe1cf8 "t", value=value@entry=0x7f4a4ffe214e "lrmd") at xml.c:1760
1760 if(TRACKING_CHANGES(node)) {
(gdb) bt
#0 0x00007f4a4f315ab3 in crm_xml_add (node=node@entry=0x230c930, name=name@entry=0x7f4a4ffe1cf8 "t", value=value@entry=0x7f4a4ffe214e "lrmd") at xml.c:1760

No idea if it happens in other cases.

@gao-yan
Cluster Labs member
gao-yan commented Feb 28, 2014

#1 0x00007f4a4ffe08fe in lrmd_handshake (lrmd=0x230b590, name=0x7f4a51280e59 "mgmtd") at lrmd_client.c:876

@beekhof beekhof merged commit 28fba1c into ClusterLabs:master Feb 28, 2014

1 check passed

Details default The Travis CI build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.