-
Notifications
You must be signed in to change notification settings - Fork 6k
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.
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
ceph-fuse: fix double free of args #7015
Conversation
@chipitsine could you
thanks. |
Can one of the admins verify this patch? |
ok to test |
Checking src/ceph_fuse.cc... [src/ceph_fuse.cc:55]: (error) Memory pointed to by 'argv' is freed twice. [src/ceph_fuse.cc:55]: (error) Deallocating a deallocated pointer: argv Signed-off-by: Ilya Shipitsin <chipitsine@gmail.com>
I added both prefix and Signed-off-by |
@ceph-jenkins test this please |
test this please |
1 similar comment
test this please |
the test failure is not relevant. there are multiple errors like
seems the monitor was not completely shutdown. and i am looking at the
failure. it only happens with py3. and is reproducible locally. addressed by #7025 |
lgtm. tested locally. |
@alfredodeza this jenkins failure is a false negative, probably due to a previously failed test. |
ceph-fuse: fix double free of args Reviewed-by: Kefu Chai <kchai@redhat.com>
Checking src/ceph_fuse.cc...
[src/ceph_fuse.cc:55]: (error) Memory pointed to by 'argv' is freed twice.
[src/ceph_fuse.cc:55]: (error) Deallocating a deallocated pointer: argv