URL:
https://github.com/SSSD/sssd/pull/94
Title: #94: Enable {socket,dbus}-activation for responders
pbrezina commented:
"""
> if (conn->last_request_time != NULL) {
> time_t *last_request_time = conn->last_request_time;
>
> *last_request_time = time(NULL);
> }
>
> This looks weird. Why did you not use simply conn->last_request_time =
> time(NULL)?
>
Hmmm. Got reminded the reason ... :-)
Take a look on the conn structure and you'll realize that last_request_time
is not a time_t, but a time_t *.
I meant: `*conn->last_request_time = time(NULL)` but it's my personal preference,
feel free to leave it this way.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/94#issuecomment-274467906