>
> So it looks all clean and works.
> That said, I would add a couple of tests on strdup you have:
>  - newnameserver->conf.file = strdup("/etc/resolv.conf");
>  - newnameserver->id = strdup(address);
> and of course, I would do the LIST_ADDQ after those checks.
>

Good catch, done.


> Just fix this and you get my Ack :)
> And thanks for your patience and your retransmits.
>

No problem, thanks for reviewing! Hopefully you guys get a break soon. Les
vacances se rapprochent.. :)

Ben

Attachment: 0001-MINOR-config-Implement-parse-resolv-conf-directive.patch
Description: Binary data

Reply via email to