URL:
https://github.com/SSSD/sssd/pull/5270
Title: #5270: CONFDB: fixed compilation warning
sumit-bose commented:
"""
Hi,
I agree with the fix because currently there is imo really a chance that `enabled` is used
uninitialized.
But I think `confdb_get_domain_enabled()` should get some fixes as well, at least the
doc-comment which currently says
```
* @param enabled The output variable; it can not be NULL; if the
* domain is explicitely enabled, *enabled is equal to 1; if the
* domain is explicitely disabled, *enabled is equal to 0; any
* other case, *enabled is -1.
```
should be fixes since the `enabled` parameter here is a bool and `any other case ...` does
not work anymore.
If you don't want to touch `confdb_get_domain_enabled()` in this PR please open a
ticket to fix it.
bye,
Sumit
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5270#issuecomment-672633520