Are we guaranteed that cp->lval can't match this block twice in one iteration over the constraints ?
If not, we need to free(execd_config->log_facility); before assigning the new xstrdup() to it.