Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 21 Apr 2012 06:29:44 +0000 (UTC)
From:      Hiroki Sato <hrs@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-user@freebsd.org
Subject:   svn commit: r234532 - user/hrs/ipv6/sys/dev/usb
Message-ID:  <201204210629.q3L6Ti2a075138@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: hrs
Date: Sat Apr 21 06:29:44 2012
New Revision: 234532
URL: http://svn.freebsd.org/changeset/base/234532

Log:
  - Fix ifname matching.
  - Return non-zero status when unit < 0.

Modified:
  user/hrs/ipv6/sys/dev/usb/usb_pf.c

Modified: user/hrs/ipv6/sys/dev/usb/usb_pf.c
==============================================================================
--- user/hrs/ipv6/sys/dev/usb/usb_pf.c	Sat Apr 21 06:10:18 2012	(r234531)
+++ user/hrs/ipv6/sys/dev/usb/usb_pf.c	Sat Apr 21 06:29:44 2012	(r234532)
@@ -129,7 +129,7 @@ usbpf_ifname2ubus(const char *ifname)
 	int unit;
 	int error;
 
-	if (strncmp(ifname, USBUSNAME, sizeof(USBUSNAME)) <= 0)
+	if (strncmp(ifname, USBUSNAME, sizeof(USBUSNAME) - 1) != 0)
 		return (NULL);
 	error = ifc_name2unit(ifname, &unit);
 	if (error || unit < 0)
@@ -167,8 +167,10 @@ usbpf_clone_create(struct if_clone *ifc,
 	struct usb_bus *ubus;
 
 	error = ifc_name2unit(name, &unit);
-	if (error || unit < 0)
+	if (error)
 		return (error);
+	if (unit < 0)
+		return (EINVAL);
 
 	ubus = usbpf_ifname2ubus(name);
 	if (ubus == NULL)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201204210629.q3L6Ti2a075138>