Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

pybind/rados: avoid call free() on invalid pointer #15159

Merged
merged 1 commit into from
May 19, 2017
Merged

pybind/rados: avoid call free() on invalid pointer #15159

merged 1 commit into from
May 19, 2017

Conversation

LiumxNL
Copy link
Contributor

@LiumxNL LiumxNL commented May 18, 2017

Signed-off-by: Mingxin Liu mingxin@xsky.com

@LiumxNL
pybind/rados: avoid call free() on invalid pointer
6dc842d 
Signed-off-by: Mingxin Liu <mingxin@xsky.com>
@LiumxNL
Copy link
Contributor Author

LiumxNL commented May 18, 2017 

INFO:tasks.workunit.client.0.mira006.stderr:/home/ubuntu/cephtest/workunit.client.0/cephtool/test.sh:1793: test_mon_ping: ceph ping 'mon.*' 
2017-03-07T12:17:53.211 INFO:tasks.workunit.client.0.mira006.stderr:2017-03-07 12:17:55.126686 7fbaabfff700 0 -- 10.255.0.210:0/1029460743 >> 10.255.0.210:6790/0 conn(0x7fbaac10eef0 :-1 s=STATE_OPEN_TAG_ACK pgs=924 cs=1 l=1).read_until injecting socket failure 
2017-03-07T12:17:53.212 INFO:tasks.workunit.client.0.mira006.stderr:*** Error in `python': munmap_chunk(): invalid pointer: 0x00007fbacb1ec5f4 ***

jdurgin
jdurgin approved these changes May 18, 2017
View reviewed changes
Copy link
Member

@jdurgin jdurgin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch!

@yuriw yuriw merged commit 5c203e8 into ceph:master May 19, 2017
@yuriw
Copy link
Contributor

yuriw commented May 19, 2017

http://pulpito.ceph.com/yuriw-2017-05-18_23:49:52-rados-wip-yuri-testing_2017_5_19-distro-basic-smithi/

http://pulpito.ceph.com/yuriw-2017-05-19_03:47:57-rados-wip-yuri-testing_2017_5_19---basic-smithi/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdurgin jdurgin jdurgin approved these changes

Assignees
No one assigned
Labels
bug-fix needs-qa pybind wip-yuri-testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LiumxNL @yuriw @jdurgin