#587 unexpected status retuned as success in fail_over.c
Closed: Fixed None Opened 13 years ago by jhrozek.

Found by clang:

989 default:
990 DEBUG(1, ("Unexpected status %d for a SRV server\n", status))do { if (1 <= debug_level) 
991 ret = EIO;
992 break;
993 }
994 
995 ret = EOK;

The break on line 989 should really say goto done so that EIO is returned.

See http://jhrozek.fedorapeople.org/sssd-clang-llvm/report-3luCQF.html#EndPath for details.


Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.3.0
owner: somebody => jhrozek

Fixed by 34e9335

fixedin: => 1.3.0
resolution: => fixed
status: new => closed

Fields changed

rhbz: => 0

Metadata Update from @jhrozek:
- Issue assigned to jhrozek
- Issue set to the milestone: SSSD 1.3.0

7 years ago

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/1629

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata