[net-next patch] tun: change tun_get_iff() prototype.

Subsystems: networking drivers, the rest, tun/tap driver

STALE5044d

2 messages, 2 authors, 2012-11-23 · open the first message on its own page

[net-next patch] tun: change tun_get_iff() prototype.

From: Rami Rosen <hidden>
Date: 2012-11-23 13:59:05

This patch changes tun_get_iff() prototype to return void as it never fails.

Signed-off-by: Rami Rosen <redacted>
---
 drivers/net/tun.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/net/tun.c b/drivers/net/tun.c
index b01e8c0..3bd9932 100644
--- a/drivers/net/tun.c
+++ b/drivers/net/tun.c
@@ -1662,7 +1662,7 @@ static int tun_set_iff(struct net *net, struct file *file, struct ifreq *ifr)
 	return err;
 }
 
-static int tun_get_iff(struct net *net, struct tun_struct *tun,
+static void tun_get_iff(struct net *net, struct tun_struct *tun,
 		       struct ifreq *ifr)
 {
 	tun_debug(KERN_INFO, tun, "tun_get_iff\n");
@@ -1671,7 +1671,6 @@ static int tun_get_iff(struct net *net, struct tun_struct *tun,
 
 	ifr->ifr_flags = tun_flags(tun);
 
-	return 0;
 }
 
 /* This is like a cut-down ethtool ops, except done via tun fd so no
@@ -1847,9 +1846,7 @@ static long __tun_chr_ioctl(struct file *file, unsigned int cmd,
 	ret = 0;
 	switch (cmd) {
 	case TUNGETIFF:
-		ret = tun_get_iff(current->nsproxy->net_ns, tun, &ifr);
-		if (ret)
-			break;
+		tun_get_iff(current->nsproxy->net_ns, tun, &ifr);
 
 		if (copy_to_user(argp, &ifr, ifreq_len))
 			ret = -EFAULT;
-- 
1.7.11.7

Re: [net-next patch] tun: change tun_get_iff() prototype.

From: David Miller <davem@davemloft.net>
Date: 2012-11-23 19:25:25

From: Rami Rosen <redacted>
Date: Fri, 23 Nov 2012 15:58:10 +0200
This patch changes tun_get_iff() prototype to return void as it never fails.

Signed-off-by: Rami Rosen <redacted>
Applied, thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help