mbox series

[v2,0/3] fix miss input validation

Message ID 1617332330-25936-1-git-send-email-humin29@huawei.com (mailing list archive)
Headers
Series fix miss input validation |

Message

humin (Q) April 2, 2021, 2:58 a.m. UTC
  The validity verification of input parameters should be performed at
API layer, not in the PMD.

This set of patches fix miss input validation.
---
v2:
* Fixed mixes in ethdev header file and ethdev .c file.

Chengchang Tang (3):
  ethdev: fix miss input validation in module EEPROM dump API
  ethdev: fix miss input validation when access reg info
  ethdev: fix miss input validation when access EEPROM info

 drivers/net/e1000/igb_ethdev.c          |  3 ---
 drivers/net/hns3/hns3_regs.c            |  5 -----
 drivers/net/i40e/i40e_ethdev.c          |  3 ---
 drivers/net/ixgbe/ixgbe_ethdev.c        |  3 ---
 drivers/net/mlx5/linux/mlx5_ethdev_os.c |  4 ++--
 drivers/net/octeontx2/otx2_ethdev_ops.c |  3 +--
 lib/librte_ethdev/rte_ethdev.c          | 10 ++++++++++
 lib/librte_ethdev/rte_ethdev.h          |  5 +++++
 8 files changed, 18 insertions(+), 18 deletions(-)
  

Comments

Ferruh Yigit April 6, 2021, 7:53 a.m. UTC | #1
On 4/2/2021 3:58 AM, Min Hu (Connor) wrote:
> The validity verification of input parameters should be performed at
> API layer, not in the PMD.
> 
> This set of patches fix miss input validation.
> ---
> v2:
> * Fixed mixes in ethdev header file and ethdev .c file.
> 
> Chengchang Tang (3):
>    ethdev: fix miss input validation in module EEPROM dump API
>    ethdev: fix miss input validation when access reg info
>    ethdev: fix miss input validation when access EEPROM info
> 

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
  
Ferruh Yigit April 7, 2021, 10:28 p.m. UTC | #2
On 4/6/2021 8:53 AM, Ferruh Yigit wrote:
> On 4/2/2021 3:58 AM, Min Hu (Connor) wrote:
>> The validity verification of input parameters should be performed at
>> API layer, not in the PMD.
>>
>> This set of patches fix miss input validation.
>> ---
>> v2:
>> * Fixed mixes in ethdev header file and ethdev .c file.
>>
>> Chengchang Tang (3):
>>    ethdev: fix miss input validation in module EEPROM dump API
>>    ethdev: fix miss input validation when access reg info
>>    ethdev: fix miss input validation when access EEPROM info
>>
> 
> For series,
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Series applied to dpdk-next-net/main, thanks.