Two cases of dead storage in path_utils.c:
- in `directory_list()`, the value of `prune` in unconditionally overwritten after the first assignment: - http://jhrozek.fedorapeople.org/sssd-clang-llvm/report-NCECM2.html#EndPath - in `is_ancestor_path()`, the value of result is set in all the branches which also return as well as when no branch is selected. Maybe it would be nicer to rewrite the code with if..else if..else.. - http://jhrozek.fedorapeople.org/sssd-clang-llvm/report-gp1oRt.html#EndPath - in `split_path()`, although the value stored to `component_len` is used in the enclosing expression, the value is never actually read from `component_len` after that point: - http://jhrozek.fedorapeople.org/sssd-clang-llvm/report-rpgeQB.html#EndPath
Assigning to dpal as per his request.
Fields changed
priority: major => minor
milestone: NEEDS_TRIAGE => Tools 1.0
rhbz: => 0
blockedby: => blocking: => coverity: => feature_milestone: => milestone: Tools Backlog => Tools 2.0 patch: => 0
design: => design_review: => 0 fedora_test_page: => selected: => testsupdated: 0 => 1
patch: 0 => 1 testsupdated: 1 => 0
Metadata Update from @jhrozek: - Issue assigned to dpal - Issue set to the milestone: Tools 2.0
Seems to be fixed via e984e91
Metadata Update from @atikhonov: - Custom field component adjusted to None (was: Path Utils) - Custom field design_review reset (from 0) - Custom field patch adjusted to on (was: 1) - Custom field selected adjusted to None - Custom field testsupdated reset (from 0) - Custom field type adjusted to None (was: defect) - Custom field version adjusted to None (was: 1.2.1) - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
Login to comment on this ticket.