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

os/filestore: fix clang static check warn "use-after-free“ #12581

Merged
merged 1 commit into from Dec 24, 2016

Conversation

Projects
None yet
3 participants
@Liuchang0812
Contributor

Liuchang0812 commented Dec 20, 2016

http://people.redhat.com/bhubbard/scan-build/report-7b926b.html#EndPath

last is a member of fiemap, free filemap after using last.

Signed-off-by: liuchang0812 liuchang0812@gmail.com

os/filestore: fix clang static check warn use-after-free
Signed-off-by: liuchang0812 <liuchang0812@gmail.com>

@liewegas liewegas merged commit 03e3a22 into ceph:master Dec 24, 2016

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
@badone

This comment has been minimized.

Contributor

badone commented Dec 27, 2016

@Liuchang0812 please check for existing PRs when submitting. #12499 pre-dates your PR by several days.

@Liuchang0812

This comment has been minimized.

Contributor

Liuchang0812 commented Dec 27, 2016

@badone very sorry for that, I will check for existing PRs carefully next time.

@Liuchang0812 Liuchang0812 deleted the Liuchang0812:fix-clang-warn-filestore branch Dec 27, 2016

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