Hi Andrei

OMG. Sorry for it. Here is the attached patch.

- Jonathan

On Thu, Jul 6, 2017 at 7:34 PM Andrei Borzenkov <[email protected]> wrote:

> On Thu, Jul 6, 2017 at 1:05 PM, Jonathan Kang <[email protected]>
> wrote:
> > The previous email was on hold because of the unsubscribed mail account.
> >
> > ---------- Forwarded message ---------
> > From: Jonathan Kang <[email protected]>
> > Date: Thu, Jul 6, 2017 at 4:34 PM
> > Subject: [PATCH] dhcp/dhclient: Improve "interface" statement parsing
> > To: [email protected] <[email protected]>
> >
> >
> > Hi
> >
> > This is a patch
>
> Are you sure you did not forget to attach patch?
>
> > improving "interface" statement parsing, which fixes
> > https://bugzilla.opensuse.org/show_bug.cgi?id=1047004.
> >
> > dhcp/dhclient: Improve "interface" statement parsing
> >
> > In commit d405cfd9089f9552969e6a3e1a1c4550fc3c1695, parsing "interface"
> > statement is introduced. But it leads to uncommplete parsing of the
> > "request" entry, if one of the lines in "request" entry is prefixed with
> > word "interface". For example, the default configuration of openSUSE
> > distribution:
> >
> > request subnet-mask, broadcast-address, routers,
> >         rfc3442-classless-static-routes,
> >         interface-mtu, host-name, domain-name, domain-search,
> >         domain-name-servers, nis-domain, nis-servers,
> >         nds-context, nds-servers, nds-tree-name,
> >         netbios-name-servers, netbios-dd-server,
> >         netbios-node-type, netbios-scope, ntp-servers;
> >
> > Fixes commit d405cfd9089f9552969e6a3e1a1c4550fc3c1695.
> >
> > _______________________________________________
> > networkmanager-list mailing list
> > [email protected]
> > https://mail.gnome.org/mailman/listinfo/networkmanager-list
> >
>
From c4b926c3b4acba572471e23b6727fa9537a21d7c Mon Sep 17 00:00:00 2001
From: Jonathan Kang <[email protected]>
Date: Thu, 6 Jul 2017 16:20:25 +0800
Subject: [PATCH] dhcp/dhclient: Improve "interface" statement parsing

In commit d405cfd9089f9552969e6a3e1a1c4550fc3c1695, parsing "interface"
statement is introduced. But it leads to uncommplete parsing of the
"request" entry, if one of the lines in "request" entry is prefixed with
word "interface". For example, the default configuration of openSUSE
distribution:

request subnet-mask, broadcast-address, routers,
	rfc3442-classless-static-routes,
	interface-mtu, host-name, domain-name, domain-search,
	domain-name-servers, nis-domain, nis-servers,
	nds-context, nds-servers, nds-tree-name,
	netbios-name-servers, netbios-dd-server,
	netbios-node-type, netbios-scope, ntp-servers;

Fixes commit d405cfd9089f9552969e6a3e1a1c4550fc3c1695.
---
 src/dhcp/nm-dhcp-dhclient-utils.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/dhcp/nm-dhcp-dhclient-utils.c b/src/dhcp/nm-dhcp-dhclient-utils.c
index 11f868e..b2f1d99 100644
--- a/src/dhcp/nm-dhcp-dhclient-utils.c
+++ b/src/dhcp/nm-dhcp-dhclient-utils.c
@@ -292,7 +292,8 @@ nm_dhcp_dhclient_create_config (const char *interface,
 				continue;
 
 			if (   !intf[0]
-			    && g_str_has_prefix (p, "interface")) {
+			    && g_str_has_prefix (p, "interface")
+			    && !in_req) {
 				if (read_interface (p, intf, sizeof (intf)))
 					continue;
 			}
-- 
2.9.4

_______________________________________________
networkmanager-list mailing list
[email protected]
https://mail.gnome.org/mailman/listinfo/networkmanager-list

Reply via email to