Re: [PATCH] net: e1000: check transmit descriptor field values

2021-02-19 Thread Peter Maydell
On Fri, 19 Feb 2021 at 03:06, Alexander Bulekov wrote: > > On 210210 2022, P J P wrote: > > From: Prasad J Pandit > > > > While processing transmit (tx) descriptors in process_tx_desc() > > various descriptor fields are not checked properly. This may lead > > to infinite loop like issue. Add chec

Re: [PATCH] net: e1000: check transmit descriptor field values

2021-02-19 Thread P J P
+-- On Thu, 18 Feb 2021, Alexander Bulekov wrote --+ | Is this a DMA re-entracy/stack-overflow issue? IIRC the plan was to have | some sort of wider fix for these issues. There are bunch of these reports | floating around at this point, I believe. * It is similar to the eepro100 one. Thank you.

Re: [PATCH] net: e1000: check transmit descriptor field values

2021-02-18 Thread Alexander Bulekov
On 210210 2022, P J P wrote: > From: Prasad J Pandit > > While processing transmit (tx) descriptors in process_tx_desc() > various descriptor fields are not checked properly. This may lead > to infinite loop like issue. Add checks to avoid them. > +CC Peter Maydell Is this a DMA re-entracy/sta

Re: [PATCH] net: e1000: check transmit descriptor field values

2021-02-18 Thread Jason Wang
On 2021/2/18 3:47 下午, P J P wrote: Hello Jason, +-- On Thu, 18 Feb 2021, Jason Wang wrote --+ | On 2021/2/10 下午10:52, P J P wrote: | > From: Prasad J Pandit | > | > While processing transmit (tx) descriptors in process_tx_desc() | > various descriptor fields are not checked properly. This

Re: [PATCH] net: e1000: check transmit descriptor field values

2021-02-17 Thread P J P
Hello Jason, +-- On Thu, 18 Feb 2021, Jason Wang wrote --+ | On 2021/2/10 下午10:52, P J P wrote: | > From: Prasad J Pandit | > | > While processing transmit (tx) descriptors in process_tx_desc() | > various descriptor fields are not checked properly. This may lead | > to infinite loop like issue

Re: [PATCH] net: e1000: check transmit descriptor field values

2021-02-17 Thread Jason Wang
On 2021/2/10 下午10:52, P J P wrote: From: Prasad J Pandit While processing transmit (tx) descriptors in process_tx_desc() various descriptor fields are not checked properly. This may lead to infinite loop like issue. Add checks to avoid them. Reported-by: Alexander Bulekov Reported-by: Cheol

[PATCH] net: e1000: check transmit descriptor field values

2021-02-10 Thread P J P
From: Prasad J Pandit While processing transmit (tx) descriptors in process_tx_desc() various descriptor fields are not checked properly. This may lead to infinite loop like issue. Add checks to avoid them. Reported-by: Alexander Bulekov Reported-by: Cheolwoo Myung Reported-by: Ruhr-University