Date: Mon, 15 Dec 2008 01:44:23 +0000 (UTC) From: Sam Leffler <sam@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r186108 - head/sbin/ifconfig Message-ID: <200812150144.mBF1iN6v080602@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: sam Date: Mon Dec 15 01:44:23 2008 New Revision: 186108 URL: http://svn.freebsd.org/changeset/base/186108 Log: back out unintended change Modified: head/sbin/ifconfig/ifvlan.c Modified: head/sbin/ifconfig/ifvlan.c ============================================================================== --- head/sbin/ifconfig/ifvlan.c Mon Dec 15 01:26:33 2008 (r186107) +++ head/sbin/ifconfig/ifvlan.c Mon Dec 15 01:44:23 2008 (r186108) @@ -119,9 +119,10 @@ vlan_set(int s, struct ifreq *ifr) } } -static void -getvlantag(const char *val) +static +DECL_CMD_FUNC(setvlantag, val, d) { + struct vlanreq vreq; u_long ul; char *endp; @@ -132,31 +133,11 @@ getvlantag(const char *val) /* check if the value can be represented in vlr_tag */ if (params.vlr_tag != ul) errx(1, "value for vlan out of range"); -} - -static -DECL_CMD_FUNC(setvlantag_clone, val, d) -{ - getvlantag(val); - clone_setcallback(vlan_create); -} - -static -DECL_CMD_FUNC(setvlantag, val, d) -{ - struct vlanreq vreq; - - getvlantag(val); - if (getvlan(s, &ifr, &vreq) == -1) - errx(1, "no existing vlan"); - vlan_set(s, &ifr); -} -static -DECL_CMD_FUNC(setvlandev_clone, val, d) -{ - strlcpy(params.vlr_parent, val, sizeof(params.vlr_parent)); - clone_setcallback(vlan_create); + if (getvlan(s, &ifr, &vreq) != -1) + vlan_set(s, &ifr); + else + clone_setcallback(vlan_create); } static @@ -165,9 +146,11 @@ DECL_CMD_FUNC(setvlandev, val, d) struct vlanreq vreq; strlcpy(params.vlr_parent, val, sizeof(params.vlr_parent)); + if (getvlan(s, &ifr, &vreq) != -1) - errx(1, "no existing vlan"); - vlan_set(s, &ifr); + vlan_set(s, &ifr); + else + clone_setcallback(vlan_create); } static @@ -189,8 +172,8 @@ DECL_CMD_FUNC(unsetvlandev, val, d) } static struct cmd vlan_cmds[] = { - DEF_CLONE_CMD_ARG("vlan", setvlantag_clone), - DEF_CLONE_CMD_ARG("vlandev", setvlandev_clone), + DEF_CLONE_CMD_ARG("vlan", setvlantag), + DEF_CLONE_CMD_ARG("vlandev", setvlandev), /* NB: non-clone cmds */ DEF_CMD_ARG("vlan", setvlantag), DEF_CMD_ARG("vlandev", setvlandev),
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200812150144.mBF1iN6v080602>