* [patch net] ovs: fix dp check in ovs_dp_reset_user_features
@ 2014-02-14 10:42 Jiri Pirko
[not found] ` <1392374556-8375-1-git-send-email-jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Jiri Pirko @ 2014-02-14 10:42 UTC (permalink / raw)
To: netdev; +Cc: davem, jesse, dev, tgraf
This fixes crash when userspace does "ovs-dpctl add-dp dev" where dev is
existing non-dp netdevice.
Introduced by:
commit 44da5ae5fbea4686f667dc854e5ea16814e44c59
"openvswitch: Drop user features if old user space attempted to create datapath"
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
---
net/openvswitch/datapath.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c
index e9a48ba..e42340d 100644
--- a/net/openvswitch/datapath.c
+++ b/net/openvswitch/datapath.c
@@ -1174,7 +1174,7 @@ static void ovs_dp_reset_user_features(struct sk_buff *skb, struct genl_info *in
struct datapath *dp;
dp = lookup_datapath(sock_net(skb->sk), info->userhdr, info->attrs);
- if (!dp)
+ if (IS_ERR(dp))
return;
WARN(dp->user_features, "Dropping previously announced user features\n");
--
1.8.5.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [patch net] ovs: fix dp check in ovs_dp_reset_user_features
[not found] ` <1392374556-8375-1-git-send-email-jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
@ 2014-02-16 1:24 ` Jesse Gross
0 siblings, 0 replies; 2+ messages in thread
From: Jesse Gross @ 2014-02-16 1:24 UTC (permalink / raw)
To: Jiri Pirko; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev, David Miller
On Fri, Feb 14, 2014 at 2:42 AM, Jiri Pirko <jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org> wrote:
> This fixes crash when userspace does "ovs-dpctl add-dp dev" where dev is
> existing non-dp netdevice.
>
> Introduced by:
> commit 44da5ae5fbea4686f667dc854e5ea16814e44c59
> "openvswitch: Drop user features if old user space attempted to create datapath"
>
> Signed-off-by: Jiri Pirko <jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
Applied, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-02-16 1:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-14 10:42 [patch net] ovs: fix dp check in ovs_dp_reset_user_features Jiri Pirko
[not found] ` <1392374556-8375-1-git-send-email-jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
2014-02-16 1:24 ` Jesse Gross
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.