[PATCH net-next] gtp: Remove a bogus tab

Subsystems: gtp (gprs tunneling protocol), networking drivers, the rest

STALE1629d

3 messages, 3 authors, 2022-03-18 · open the first message on its own page

[PATCH net-next] gtp: Remove a bogus tab

From: Dan Carpenter <hidden>
Date: 2022-03-17 07:57:17

The "kfree_skb(skb_to_send);" is not supposed to be indented that far.

Signed-off-by: Dan Carpenter <redacted>
---
 drivers/net/gtp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index 756714d4ad92..a208e2b1a9af 100644
--- a/drivers/net/gtp.c
+++ b/drivers/net/gtp.c
@@ -1793,7 +1793,7 @@ static int gtp_genl_send_echo_req(struct sk_buff *skb, struct genl_info *info)
 	if (IS_ERR(rt)) {
 		netdev_dbg(gtp->dev, "no route for echo request to %pI4\n",
 			   &dst_ip);
-			   kfree_skb(skb_to_send);
+		kfree_skb(skb_to_send);
 		return -ENODEV;
 	}
 
-- 
2.20.1

Re: [PATCH net-next] gtp: Remove a bogus tab

From: patchwork-bot+netdevbpf@kernel.org
Date: 2022-03-17 23:50:21

Hello:

This patch was applied to netdev/net-next.git (master)
by Tony Nguyen [off-list ref]:

On Thu, 17 Mar 2022 10:56:42 +0300 you wrote:
The "kfree_skb(skb_to_send);" is not supposed to be indented that far.

Signed-off-by: Dan Carpenter <redacted>
---
 drivers/net/gtp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Here is the summary with links:
  - [net-next] gtp: Remove a bogus tab
    https://git.kernel.org/netdev/net-next/c/02f393381d14

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html

Re: [PATCH net-next] gtp: Remove a bogus tab

From: Jakub Kicinski <kuba@kernel.org>
Date: 2022-03-18 00:45:58

On Thu, 17 Mar 2022 23:50:12 +0000 patchwork-bot+netdevbpf@kernel.org
wrote:
Hello:

This patch was applied to netdev/net-next.git (master)
by Tony Nguyen [off-list ref]:

On Thu, 17 Mar 2022 10:56:42 +0300 you wrote:
quoted
The "kfree_skb(skb_to_send);" is not supposed to be indented that far.

Signed-off-by: Dan Carpenter <redacted>
---
 drivers/net/gtp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)  
Here is the summary with links:
  - [net-next] gtp: Remove a bogus tab
    https://git.kernel.org/netdev/net-next/c/02f393381d14
Not really, the patch that got applied was the version from Wojciech.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help