Learn more about these different git repos.
Other Git URLs
193 CID 11852: Dereference before null check (REVERSE_INULL) [select issue] CID 11853: Dereference before null check (REVERSE_INULL) [select issue] CID 11851: Dereference before null check (REVERSE_INULL)check_after_deref: Null-checking "entry" suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 194 if (entry != NULL && 195 service != NULL && 196 enabled != NULL) {
I suggested the NULL checks during code review but reading the code now more carefully the worst case that can happen is that all pointers will point to "line" so maybe the checks can be removed. Please test with malformed lines (two out of three valid keywords etc).
Fields changed
coverity: 11851 => 11851,11852
coverity: 11851,11852 => 11851,11852,11853
patch: 0 => 1 status: new => assigned
A Coverity bug can be moved to 1.10 beta directly w/o waiting on the triage.
milestone: NEEDS_TRIAGE => SSSD 1.10 beta resolution: => fixed status: assigned => closed
rhbz: => 0
Metadata Update from @jhrozek: - Issue assigned to okos - Issue set to the milestone: SSSD 1.10 beta
SSSD is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in SSSD's github repository.
This issue has been cloned to Github and is available here: - https://github.com/SSSD/sssd/issues/3013
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.
Login to comment on this ticket.