Description of problem:

                if (!error) {
                        list_del(&node->list);
                        free(node);
                        /* XXXXXXXXXX There should be a continue here */
                }

                if (!comline.override_path) {
                        sleep(5);
                        continue;
                }

                /* Check for manual intervention */
                override = open_override(comline.override_path);

                if (check_override(override, node->name, 5) > 0) {
                        syslog(LOG_WARNING, "fence \"%s\" overridden by "
                               "administrator intervention", node->name);

                        list_del(&node->list);
                        free(node);
                }
                close_override(&override, comline.override_path);
        }

Version          : 5.1 beta
Reproducibility  : 100%
Actual results   : Use-after-free could potentially cause a crash in fenced
Expected results : No crash
Additional info  : Unclear whether this has caused actual problems.