mbox series

[SRU,J,0/1,F,0/3] CVE-2024-26830

Message ID 20240708152124.14807-1-bethany.jamison@canonical.com
Headers show
Series CVE-2024-26830 | expand

Message

Bethany Jamison July 8, 2024, 3:21 p.m. UTC
[Impact]

i40e: Do not allow untrusted VF to remove administratively set MAC

Currently when PF administratively sets VF's MAC address and the VF
is put down (VF tries to delete all MACs) then the MAC is removed
from MAC filters and primary VF MAC is zeroed.

Do not allow untrusted VF to remove primary MAC when it was set
administratively by PF.

[Fix]

Noble:	not-affected
Jammy:	Backport - I ignored #defines next to the fix area, shouldn't
	affect the fix changes
Focal:	Backport - same as Jammy, Clean cherry-picks from prereqs
Bionic:	needed
Xenial:	needed
Trusty: not going to be fixed by us

[Test Case]

Compile and boot tested

[Where problems could occur]

This fix affects those who use the Intel i40e driver, an issue with 
this fix would be visible to the user via unexpected system behavior.

Ivan Vecera (1):
  i40e: Do not allow untrusted VF to remove administratively set MAC

 .../ethernet/intel/i40e/i40e_virtchnl_pf.c    | 38 ++++++++++++++++---
 1 file changed, 33 insertions(+), 5 deletions(-)

Comments

Manuel Diewald July 8, 2024, 5:41 p.m. UTC | #1
On Mon, Jul 08, 2024 at 10:21:20AM -0500, Bethany Jamison wrote:
> [Impact]
> 
> i40e: Do not allow untrusted VF to remove administratively set MAC
> 
> Currently when PF administratively sets VF's MAC address and the VF
> is put down (VF tries to delete all MACs) then the MAC is removed
> from MAC filters and primary VF MAC is zeroed.
> 
> Do not allow untrusted VF to remove primary MAC when it was set
> administratively by PF.
> 
> [Fix]
> 
> Noble:	not-affected
> Jammy:	Backport - I ignored #defines next to the fix area, shouldn't
> 	affect the fix changes
> Focal:	Backport - same as Jammy, Clean cherry-picks from prereqs
> Bionic:	needed
> Xenial:	needed
> Trusty: not going to be fixed by us
> 
> [Test Case]
> 
> Compile and boot tested
> 
> [Where problems could occur]
> 
> This fix affects those who use the Intel i40e driver, an issue with 
> this fix would be visible to the user via unexpected system behavior.
> 
> Ivan Vecera (1):
>   i40e: Do not allow untrusted VF to remove administratively set MAC
> 
>  .../ethernet/intel/i40e/i40e_virtchnl_pf.c    | 38 ++++++++++++++++---
>  1 file changed, 33 insertions(+), 5 deletions(-)
> 
> -- 
> 2.34.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Acked-by: Manuel Diewald <manuel.diewald@canonical.com>
Noah Wager July 9, 2024, 2:12 a.m. UTC | #2
Acked-by: Noah Wager <noah.wager@canonical.com>

On Mon, Jul 08, 2024 at 10:21:20AM -0500, Bethany Jamison wrote:
> [Impact]
> 
> i40e: Do not allow untrusted VF to remove administratively set MAC
> 
> Currently when PF administratively sets VF's MAC address and the VF
> is put down (VF tries to delete all MACs) then the MAC is removed
> from MAC filters and primary VF MAC is zeroed.
> 
> Do not allow untrusted VF to remove primary MAC when it was set
> administratively by PF.
> 
> [Fix]
> 
> Noble:	not-affected
> Jammy:	Backport - I ignored #defines next to the fix area, shouldn't
> 	affect the fix changes
> Focal:	Backport - same as Jammy, Clean cherry-picks from prereqs
> Bionic:	needed
> Xenial:	needed
> Trusty: not going to be fixed by us
> 
> [Test Case]
> 
> Compile and boot tested
> 
> [Where problems could occur]
> 
> This fix affects those who use the Intel i40e driver, an issue with 
> this fix would be visible to the user via unexpected system behavior.
> 
> Ivan Vecera (1):
>   i40e: Do not allow untrusted VF to remove administratively set MAC
> 
>  .../ethernet/intel/i40e/i40e_virtchnl_pf.c    | 38 ++++++++++++++++---
>  1 file changed, 33 insertions(+), 5 deletions(-)
> 
> -- 
> 2.34.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Stefan Bader July 19, 2024, 9:06 a.m. UTC | #3
On 08.07.24 17:21, Bethany Jamison wrote:
> [Impact]
> 
> i40e: Do not allow untrusted VF to remove administratively set MAC
> 
> Currently when PF administratively sets VF's MAC address and the VF
> is put down (VF tries to delete all MACs) then the MAC is removed
> from MAC filters and primary VF MAC is zeroed.
> 
> Do not allow untrusted VF to remove primary MAC when it was set
> administratively by PF.
> 
> [Fix]
> 
> Noble:	not-affected
> Jammy:	Backport - I ignored #defines next to the fix area, shouldn't
> 	affect the fix changes
> Focal:	Backport - same as Jammy, Clean cherry-picks from prereqs
> Bionic:	needed
> Xenial:	needed
> Trusty: not going to be fixed by us
> 
> [Test Case]
> 
> Compile and boot tested
> 
> [Where problems could occur]
> 
> This fix affects those who use the Intel i40e driver, an issue with
> this fix would be visible to the user via unexpected system behavior.
> 
> Ivan Vecera (1):
>    i40e: Do not allow untrusted VF to remove administratively set MAC
> 
>   .../ethernet/intel/i40e/i40e_virtchnl_pf.c    | 38 ++++++++++++++++---
>   1 file changed, 33 insertions(+), 5 deletions(-)
> 

Applied to jammy,focal:linux/master-next. Thanks.

-Stefan