* stable request: net: sched: Remove TCA_OPTIONS from policy
@ 2018-11-10 19:20 Thomas Deutschmann
2018-11-10 21:40 ` stable request: net: sched: Remove TCA_OPTIONS from policy,stable " David Miller
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Deutschmann @ 2018-11-10 19:20 UTC (permalink / raw)
To: stable@vger.kernel.org; +Cc: dsahern, davem@davemloft.net, pupilla, Greg KH
[-- Attachment #1.1: Type: text/plain, Size: 1881 bytes --]
Hi,
all stable kernels carrying
> From 8b4c3cdd9dd8290343ce959a132d3b334062c5b9 Mon Sep 17 00:00:00 2001
> From: David Ahern <dsahern@gmail.com>
> Date: Wed, 3 Oct 2018 15:05:36 -0700
> Subject: [PATCH] net: sched: Add policy validation for tc attributes
>
> A number of TC attributes are processed without proper validation
> (e.g., length checks). Add a tca policy for all input attributes and use
> when invoking nlmsg_parse.
>
> The 2 Fixes tags below cover the latest additions. The other attributes
> are a string (KIND), nested attribute (OPTIONS which does seem to have
> validation in most cases), for dumps only or a flag.
>
> Fixes: 5bc1701881e39 ("net: sched: introduce multichain support for filters")
> Fixes: d47a6b0e7c492 ("net: sched: introduce ingress/egress block index attributes for qdisc")
> Signed-off-by: David Ahern <dsahern@gmail.com>
> Signed-off-by: David S. Miller <davem@davemloft.net>
should backport
> From e72bde6b66299602087c8c2350d36a525e75d06e Mon Sep 17 00:00:00 2001
> From: David Ahern <dsahern@gmail.com>
> Date: Wed, 24 Oct 2018 08:32:49 -0700
> Subject: [PATCH] net: sched: Remove TCA_OPTIONS from policy
>
> Marco reported an error with hfsc:
> root@Calimero:~# tc qdisc add dev eth0 root handle 1:0 hfsc default 1
> Error: Attribute failed policy validation.
>
> Apparently a few implementations pass TCA_OPTIONS as a binary instead
> of nested attribute, so drop TCA_OPTIONS from the policy.
>
> Fixes: 8b4c3cdd9dd8 ("net: sched: Add policy validation for tc attributes")
> Reported-by: Marco Berizzi <pupilla@libero.it>
> Signed-off-by: David Ahern <dsahern@gmail.com>
> Signed-off-by: David S. Miller <davem@davemloft.net>
See https://lkml.org/lkml/2018/10/26/80.
--
Regards,
Thomas Deutschmann / Gentoo Linux Developer
C4DD 695F A713 8F24 2AA1 5638 5849 7EE5 1D5D 74A5
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 618 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: stable request: net: sched: Remove TCA_OPTIONS from policy,stable request: net: sched: Remove TCA_OPTIONS from policy
2018-11-10 19:20 stable request: net: sched: Remove TCA_OPTIONS from policy Thomas Deutschmann
@ 2018-11-10 21:40 ` David Miller
2018-11-11 20:50 ` Greg KH
0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2018-11-10 21:40 UTC (permalink / raw)
To: whissi; +Cc: stable, dsahern, pupilla, gregkh
From: Thomas Deutschmann <whissi@gentoo.org>
Date: Sat, 10 Nov 2018 20:20:01 +0100
> Hi,
>
> all stable kernels carrying
>
>> From 8b4c3cdd9dd8290343ce959a132d3b334062c5b9 Mon Sep 17 00:00:00 2001
>> From: David Ahern <dsahern@gmail.com>
>> Date: Wed, 3 Oct 2018 15:05:36 -0700
>> Subject: [PATCH] net: sched: Add policy validation for tc attributes
>>
>> A number of TC attributes are processed without proper validation
>> (e.g., length checks). Add a tca policy for all input attributes and use
>> when invoking nlmsg_parse.
>>
>> The 2 Fixes tags below cover the latest additions. The other attributes
>> are a string (KIND), nested attribute (OPTIONS which does seem to have
>> validation in most cases), for dumps only or a flag.
>>
>> Fixes: 5bc1701881e39 ("net: sched: introduce multichain support for filters")
>> Fixes: d47a6b0e7c492 ("net: sched: introduce ingress/egress block index attributes for qdisc")
>> Signed-off-by: David Ahern <dsahern@gmail.com>
>> Signed-off-by: David S. Miller <davem@davemloft.net>
>
> should backport
>
>> From e72bde6b66299602087c8c2350d36a525e75d06e Mon Sep 17 00:00:00 2001
>> From: David Ahern <dsahern@gmail.com>
>> Date: Wed, 24 Oct 2018 08:32:49 -0700
>> Subject: [PATCH] net: sched: Remove TCA_OPTIONS from policy
>>
>> Marco reported an error with hfsc:
>> root@Calimero:~# tc qdisc add dev eth0 root handle 1:0 hfsc default 1
>> Error: Attribute failed policy validation.
>>
>> Apparently a few implementations pass TCA_OPTIONS as a binary instead
>> of nested attribute, so drop TCA_OPTIONS from the policy.
>>
>> Fixes: 8b4c3cdd9dd8 ("net: sched: Add policy validation for tc attributes")
>> Reported-by: Marco Berizzi <pupilla@libero.it>
>> Signed-off-by: David Ahern <dsahern@gmail.com>
>> Signed-off-by: David S. Miller <davem@davemloft.net>
>
> See https://lkml.org/lkml/2018/10/26/80.
Indeed, -stable folks please add this latter patch on top.
Thank you.
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: stable request: net: sched: Remove TCA_OPTIONS from policy,stable request: net: sched: Remove TCA_OPTIONS from policy
2018-11-10 21:40 ` stable request: net: sched: Remove TCA_OPTIONS from policy,stable " David Miller
@ 2018-11-11 20:50 ` Greg KH
0 siblings, 0 replies; 3+ messages in thread
From: Greg KH @ 2018-11-11 20:50 UTC (permalink / raw)
To: David Miller; +Cc: whissi, stable, dsahern, pupilla
On Sat, Nov 10, 2018 at 01:40:03PM -0800, David Miller wrote:
> From: Thomas Deutschmann <whissi@gentoo.org>
> Date: Sat, 10 Nov 2018 20:20:01 +0100
>
> > Hi,
> >
> > all stable kernels carrying
> >
> >> From 8b4c3cdd9dd8290343ce959a132d3b334062c5b9 Mon Sep 17 00:00:00 2001
> >> From: David Ahern <dsahern@gmail.com>
> >> Date: Wed, 3 Oct 2018 15:05:36 -0700
> >> Subject: [PATCH] net: sched: Add policy validation for tc attributes
> >>
> >> A number of TC attributes are processed without proper validation
> >> (e.g., length checks). Add a tca policy for all input attributes and use
> >> when invoking nlmsg_parse.
> >>
> >> The 2 Fixes tags below cover the latest additions. The other attributes
> >> are a string (KIND), nested attribute (OPTIONS which does seem to have
> >> validation in most cases), for dumps only or a flag.
> >>
> >> Fixes: 5bc1701881e39 ("net: sched: introduce multichain support for filters")
> >> Fixes: d47a6b0e7c492 ("net: sched: introduce ingress/egress block index attributes for qdisc")
> >> Signed-off-by: David Ahern <dsahern@gmail.com>
> >> Signed-off-by: David S. Miller <davem@davemloft.net>
> >
> > should backport
> >
> >> From e72bde6b66299602087c8c2350d36a525e75d06e Mon Sep 17 00:00:00 2001
> >> From: David Ahern <dsahern@gmail.com>
> >> Date: Wed, 24 Oct 2018 08:32:49 -0700
> >> Subject: [PATCH] net: sched: Remove TCA_OPTIONS from policy
> >>
> >> Marco reported an error with hfsc:
> >> root@Calimero:~# tc qdisc add dev eth0 root handle 1:0 hfsc default 1
> >> Error: Attribute failed policy validation.
> >>
> >> Apparently a few implementations pass TCA_OPTIONS as a binary instead
> >> of nested attribute, so drop TCA_OPTIONS from the policy.
> >>
> >> Fixes: 8b4c3cdd9dd8 ("net: sched: Add policy validation for tc attributes")
> >> Reported-by: Marco Berizzi <pupilla@libero.it>
> >> Signed-off-by: David Ahern <dsahern@gmail.com>
> >> Signed-off-by: David S. Miller <davem@davemloft.net>
> >
> > See https://lkml.org/lkml/2018/10/26/80.
>
> Indeed, -stable folks please add this latter patch on top.
Now queued up, thanks.
greg k-h
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-11-12 6:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-10 19:20 stable request: net: sched: Remove TCA_OPTIONS from policy Thomas Deutschmann
2018-11-10 21:40 ` stable request: net: sched: Remove TCA_OPTIONS from policy,stable " David Miller
2018-11-11 20:50 ` Greg KH
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.