Hello,
I've updated the patch to fix resolvconf support.
It fixes a logic error in the patched dns_add(), and chmod()s the right
file(s) in the new domain_add_file(), so /etc/resolv.conf won't
become o-r / others un-readable again.
Also, each new FOO_file() function is above the old FOO() function,
to avoid the FOO_file() function being implicitly declared.
It still does not call resolvconf -d IFACE.inet6 anywhere, and I haven't
patched the other two ports with resolvconf support code yet. But this
seems to run on my machine better than 1.0.0.
Regards, Fabian
diff --git a/Misc/Portable.h b/Misc/Portable.h
index 8c43b80..9b89a0d 100644
--- a/Misc/Portable.h
+++ b/Misc/Portable.h
@@ -151,6 +151,7 @@ struct link_state_notify_t
#define SRVCONF_FILE "/etc/dibbler/server.conf"
#define RELCONF_FILE "/etc/dibbler/relay.conf"
#define RESOLVCONF_FILE "/etc/resolv.conf"
+#define PRIV_RESOLVCONF_FILE "/var/lib/dibbler/resolv.conf"
#define NTPCONF_FILE "/etc/ntp.conf"
#define RADVD_FILE "/etc/dibbler/radvd.conf"
#define CLNTPID_FILE "/var/lib/dibbler/client.pid"
diff --git a/Misc/Portable.h.in b/Misc/Portable.h.in
index d8bc4a9..a2fe9e6 100644
--- a/Misc/Portable.h.in
+++ b/Misc/Portable.h.in
@@ -151,6 +151,7 @@ struct link_state_notify_t
#define SRVCONF_FILE "/etc/dibbler/server.conf"
#define RELCONF_FILE "/etc/dibbler/relay.conf"
#define RESOLVCONF_FILE "/etc/resolv.conf"
+#define PRIV_RESOLVCONF_FILE "/var/lib/dibbler/resolv.conf"
#define NTPCONF_FILE "/etc/ntp.conf"
#define RADVD_FILE "/etc/dibbler/radvd.conf"
#define CLNTPID_FILE "/var/lib/dibbler/client.pid"
diff --git a/Port-linux/lowlevel-options-linux.c b/Port-linux/lowlevel-options-linux.c
index 4000290..eb0c50b 100644
--- a/Port-linux/lowlevel-options-linux.c
+++ b/Port-linux/lowlevel-options-linux.c
@@ -9,6 +9,7 @@
#define _BSD_SOURCE
#define _POSIX_SOURCE
+#define _GNU_SOURCE
#include <stdio.h>
#include <unistd.h>
@@ -42,19 +43,24 @@ extern char * Message;
* the pipe needs to be closed by the caller
*
* @param arg1 first command line argument passed to resolvconf
- * @param arg2 second command line argument passed to resolvconf
+ * @param ifname interface name for which to run resolvconf
*
* @return file handler (pipe to resolvconf process) or NULL
*/
-FILE *resolvconf_open(const char *arg1, const char *arg2)
+FILE *resolvconf_open(const char *arg1, const char *ifname)
{
pid_t child;
int pipefd[2];
+ char * ifname_;
if (access(RESOLVCONF, X_OK) != 0)
return NULL;
- if (pipe(pipefd) != 0)
+ if (!asprintf(&ifname_, "%s.inet6", ifname))
return NULL;
+ if (pipe(pipefd) != 0) {
+ free(ifname_);
+ return NULL;
+ }
switch(child = fork()) {
case 0: /* child */
close(pipefd[1]);
@@ -63,19 +69,35 @@ FILE *resolvconf_open(const char *arg1, const char *arg2)
close(pipefd[0]);
/* double fork so init reaps the child */
if (!fork()) { /* child */
- execl(RESOLVCONF, RESOLVCONF, arg1, arg2, (char *)NULL);
+ execl(RESOLVCONF, RESOLVCONF, arg1, ifname_, (char *)NULL);
} /* All other cases are meaningless here */
exit(EXIT_FAILURE);
break;
case EXIT_FAILURE: /* error */
+ free(ifname_);
return NULL;
break;
}
/* parent */
+ free(ifname_);
close(pipefd[0]);
waitpid(child, NULL, 0);
return fdopen(pipefd[1], "w");
}
+
+int resolvconf_feed(FILE * pipe, const char * file) {
+ FILE * f2 = NULL;
+ unsigned int c;
+
+ if (!(f2=fopen(file, "r")))
+ return LOWLEVEL_ERROR_FILE;
+
+ while ((c = fgetc(f2)) != EOF) {
+ fputc(c, pipe);
+ }
+ fclose(f2);
+ return LOWLEVEL_NO_ERROR;
+}
#endif
/* in iproute.c, borrowed from iproute2 */
@@ -230,18 +252,12 @@ int cfg_file_del(const char *file, const char *keyword, const char *value) {
-1 - unable to open temp. file
-2 - unable to open resolv.conf file
*/
-int dns_add(const char * ifname, int ifaceid, const char * addrPlain) {
+int dns_add_file(const char * ifname, int ifaceid, const char * addrPlain, const char * file) {
FILE * f = NULL;
unsigned char c;
-#ifdef MOD_RESOLVCONF
- /* try to use resolvconf */
- f=resolvconf_open("-a", ifname);
-#endif
-
- /* if resolvconf is not available, fallback to normal file append */
- if (!f && !(f=fopen(RESOLVCONF_FILE, "a+")) ) {
- return LOWLEVEL_ERROR_FILE;
+ if (!(f=fopen(file, "a+")) ) {
+ return LOWLEVEL_ERROR_FILE;
}
fseek(f, -1, SEEK_END);
@@ -257,46 +273,95 @@ int dns_add(const char * ifname, int ifaceid, const char * addrPlain) {
return LOWLEVEL_NO_ERROR;
}
+
+/*
+ * results 0 - ok
+ -1 - unable to open temp. file
+ -2 - unable to open resolv.conf file
+ */
+int dns_add(const char * ifname, int ifaceid, const char * addrPlain) {
+ FILE * f = NULL;
+ char * file;
+ int ret;
+
+#ifdef MOD_RESOLVCONF
+ /* try to use resolvconf */
+ f=resolvconf_open("-a", ifname);
+ if (f) {
+ if (!asprintf(&file, PRIV_RESOLVCONF_FILE ".%s", ifname)) {
+ fclose(f);
+ return LOWLEVEL_ERROR_UNSPEC;
+ }
+
+ /* edit private resolv.conf-like file */
+ if ((ret = dns_add_file(ifname, ifaceid, addrPlain, file))) {
+ free(file);
+ fclose(f);
+ return ret;
+ }
+
+ /* feed that file to resolvconf */
+ ret = resolvconf_feed(f, file);
+ free(file);
+ fclose(f);
+ return ret;
+ }
+
+#endif
+
+ /* if resolvconf is not available, fallback to normal file append */
+ return dns_add_file(ifname, ifaceid, addrPlain, RESOLVCONF_FILE);
+}
+
int dns_del(const char * ifname, int ifaceid, const char *addrPlain) {
#ifdef MOD_RESOLVCONF
- FILE *f = NULL;
- /* try to use resolvconf to remove config */
- if ((f=resolvconf_open("-d", ifname))) {
+ FILE * f = NULL;
+ char * file;
+ int ret;
+ /* try to use resolvconf to update config */
+ if ((f=resolvconf_open("-a", ifname))) {
+ if (!asprintf(&file, PRIV_RESOLVCONF_FILE ".%s", ifname)) {
+ fclose(f);
+ return LOWLEVEL_ERROR_UNSPEC;
+ }
+
+ /* edit private resolv.conf-like file */
+ if ((ret = cfg_file_del(file, "nameserver", addrPlain))) {
+ free(file);
+ fclose(f);
+ return ret;
+ }
+
+ /* feed that file to resolvconf */
+ ret = resolvconf_feed(f, file);
+ free(file);
fclose(f);
- return LOWLEVEL_NO_ERROR;
+ return ret;
}
#endif
return cfg_file_del(RESOLVCONF_FILE, "nameserver", addrPlain);
}