diff options
| author |   Tom Parkin <tparkin@katalix.com> | 2020-07-28 18:20:30 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-07-30 16:45:31 -0700 |
| commit | 628703f59dcc832a0bd04b4fa41d856e5df98112 (patch) | |
| tree | a2b3fc3c205ecf055a80ef21e829cb1e7d5304fd /net/l2tp/l2tp_netlink.c | |
| parent | l2tp: don't export tunnel and session free functions (diff) | |
| download | linux-dev-628703f59dcc832a0bd04b4fa41d856e5df98112.tar.xz linux-dev-628703f59dcc832a0bd04b4fa41d856e5df98112.zip | |
l2tp: return void from l2tp_session_delete
l2tp_session_delete is used to schedule a session instance for deletion.
The function itself always returns zero, and none of its direct callers
check its return value, so have the function return void.
This change de-facto changes the l2tp netlink session_delete callback
prototype since all pseudowires currently use l2tp_session_delete for
their implementation of that operation.
Signed-off-by: Tom Parkin <tparkin@katalix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/l2tp/l2tp_netlink.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/l2tp/l2tp_netlink.c b/net/l2tp/l2tp_netlink.c index 35716a6e1e2c..def78eebca4c 100644 --- a/net/l2tp/l2tp_netlink.c +++ b/net/l2tp/l2tp_netlink.c @@ -670,7 +670,7 @@ static int l2tp_nl_cmd_session_delete(struct sk_buff *skb, struct genl_info *inf pw_type = session->pwtype; if (pw_type < __L2TP_PWTYPE_MAX) if (l2tp_nl_cmd_ops[pw_type] && l2tp_nl_cmd_ops[pw_type]->session_delete) - ret = (*l2tp_nl_cmd_ops[pw_type]->session_delete)(session); + l2tp_nl_cmd_ops[pw_type]->session_delete(session); l2tp_session_dec_refcount(session); |