URL: https://github.com/SSSD/sssd/pull/5750
Title: #5750: fix compilation warnings

elkoniu commented:
"""
> About 2nd patch... Can't we just change signature of `sss_getenv()` to `... 
> const char **_value`? (CC @elkoniu)

I think the problem has philosophical nature. We are allocating data and 
modifying pointer inside function so in fact it is not `const` even if we 
consider output data pointed by it as `const`. Another issue is deallocation. I 
think when talloc will be releasing the data with `free()` it will complain 
about const pointer.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5750#issuecomment-900202615
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to