* Re: [dpdk-dev] [PATCH] virtio: fix crash if CQ is not negotiated
[not found] <1432549252-8630-1-git-send-email-damarion@cisco.com>
@ 2015-05-26 14:59 ` Stephen Hemminger
2015-06-12 12:50 ` Thomas Monjalon
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2015-05-26 14:59 UTC (permalink / raw)
To: Damjan Marion; +Cc: dev
On Mon, 25 May 2015 12:20:52 +0200
Damjan Marion <damarion@cisco.com> wrote:
> Fix NULL dereference if virtio control queue is not negotiated.
>
> Signed-off-by: Damjan Marion <damarion@cisco.com>
This is good belt and suspenders thing to have, but did you see early
patches to check the feature bits and not call this code?
Acked-by: Stephen Hemminger <stephen@networkplumber.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [dpdk-dev] [PATCH] virtio: fix crash if CQ is not negotiated
2015-05-26 14:59 ` [dpdk-dev] [PATCH] virtio: fix crash if CQ is not negotiated Stephen Hemminger
@ 2015-06-12 12:50 ` Thomas Monjalon
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2015-06-12 12:50 UTC (permalink / raw)
To: Damjan Marion; +Cc: dev
> > Fix NULL dereference if virtio control queue is not negotiated.
> >
> > Signed-off-by: Damjan Marion <damarion@cisco.com>
>
> This is good belt and suspenders thing to have, but did you see early
> patches to check the feature bits and not call this code?
>
> Acked-by: Stephen Hemminger <stephen@networkplumber.org>
Applied, thanks
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-06-12 12:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1432549252-8630-1-git-send-email-damarion@cisco.com>
2015-05-26 14:59 ` [dpdk-dev] [PATCH] virtio: fix crash if CQ is not negotiated Stephen Hemminger
2015-06-12 12:50 ` Thomas Monjalon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).