diff options
| author | Yan Zhen <[email protected]> | 2024-08-29 09:55:09 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-08-30 18:11:45 +0000 |
| commit | b26b64493343659cce8bbffa358bf39e4f68bdec (patch) | |
| tree | e2d6278daefceea519188abab802af9e66eda7b0 | |
| parent | nfp: Convert to use ERR_CAST() (diff) | |
| download | kernel-b26b64493343659cce8bbffa358bf39e4f68bdec.tar.gz kernel-b26b64493343659cce8bbffa358bf39e4f68bdec.zip | |
net: openvswitch: Use ERR_CAST() to return
Using ERR_CAST() is more reasonable and safer, When it is necessary
to convert the type of an error pointer and return it.
Signed-off-by: Yan Zhen <[email protected]>
Acked-by: Eelco Chaudron <[email protected]>
Reviewed-by: Aaron Conole <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
| -rw-r--r-- | net/openvswitch/flow_netlink.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/openvswitch/flow_netlink.c b/net/openvswitch/flow_netlink.c index c92bdc4dfe19..729ef582a3a8 100644 --- a/net/openvswitch/flow_netlink.c +++ b/net/openvswitch/flow_netlink.c @@ -2491,7 +2491,7 @@ static struct nlattr *reserve_sfa_size(struct sw_flow_actions **sfa, acts = nla_alloc_flow_actions(new_acts_size); if (IS_ERR(acts)) - return (void *)acts; + return ERR_CAST(acts); memcpy(acts->actions, (*sfa)->actions, (*sfa)->actions_len); acts->actions_len = (*sfa)->actions_len; |