Ticket was cloned from Red Hat Bugzilla (product Red Hat Enterprise Linux 7): Bug 1431877
Description of problem: Many covscan errors were detected from 389-ds-base-1.3.6.1-1.el7, these need to be addressed before releasing a build.
Metadata Update from @mreynolds: - Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1431877
<img alt="0001-Issue-49169-FIx-covscan-errors.patch" src="/389-ds-base/issue/raw/files/cf6a5bd9440c7d74aac370c116e0c2f84f8c2bd84a2e9f6ba20a0380a6386c3d-0001-Issue-49169-FIx-covscan-errors.patch" />
Metadata Update from @mreynolds: - Custom field reviewstatus adjusted to review - Custom field type adjusted to defect
Metadata Update from @mreynolds: - Issue assigned to mreynolds
The patch LGTM.
Metadata Update from @nkinder: - Custom field reviewstatus adjusted to ack (was: review)
On issue is the free(cur) in map.c of sds. Should be sds_free(cur).
Applied Williams suggestion:
770fcf4..c75126b master -> master
Metadata Update from @mreynolds: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
commit c75126b broke tests, 95 failed, 140 errors. https://fedorapeople.org/~vashirov/share/report-gitf786639.html
Note that the report is for the latest master, but I bisected it to c75126b
Fixed. Problem was with the change to vattr.c - which I undid.
f786639..314e9ec master -> master
Thanks Mark! Looks good now: http://fedorapeople.org/~vashirov/share/report-git314e9ec.html
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/2228
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: fixed)
Login to comment on this ticket.