On 07/12/2016 01:15 PM, Pavel Březina wrote:
On 07/12/2016 12:34 PM, Michal Židek wrote:
> state->ipa_ctx->dyndns_ctx->last_refresh = time(NULL);
LGTM but maybe we should place the check before this line?
Not sure... I only added checks for the line with strcmp
(which is where it segfaulted). If I moved
it up where you suggest, there would be the question why
I do not check for example ipa_ctx or ipa_ctx->dyndns_ctx.
But if you prefer having it there I can do it, but will
also add a comment that the checks are relevant for the
strcmp line.
Michal