Description of problem: LVM2.2.02.174/libdm/libdm-report.c:3415]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses. Source code is if ((r = handler(rh, rh->selection->mem, field_num, DM_REPORT_RESERVED_GET_DYNAMIC_VALUE, rvw->matched_name, &tmp_value) <= 0)) { Maybe better code if ((r = handler(rh, rh->selection->mem, field_num, DM_REPORT_RESERVED_GET_DYNAMIC_VALUE, rvw->matched_name, &tmp_value)) <= 0) { Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
Thanks for noticing - fix will be pushed upstream soon.
Fixed with upstream commit: https://www.redhat.com/archives/lvm-devel/2017-September/msg00026.html