From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 46BA043AB4; Fri, 9 Feb 2024 15:01:46 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DF81B42E64; Fri, 9 Feb 2024 15:01:45 +0100 (CET) Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-sn1nam02on2052.outbound.protection.outlook.com [40.107.96.52]) by mails.dpdk.org (Postfix) with ESMTP id BF23440697 for ; Fri, 9 Feb 2024 15:01:44 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GYCzqk3JKYir+nfeKDyjMyRw1Wtx7lwSQkKRLRQYKKcZ9f5I+AXvImzQPdp9waV+heS7g+N/YYNWWhOx82gXtojwArUCeXTLmqUEJr7fEeSyFvUxerYFzdc7jCjFXCJbRNMckbFh0QqDCdiyDrJS42J2P2qLMbThxT3qSTudfwL56hcBl9wyu1qDgW79/ZYfv+fP7eUNRr/I2JlBEujg4qTDiTYFjI78ejeqgj2uAcz8iqEjSdpy/yrEZK1cKZiUJg8YfvF1+N61Y3w6OG3HBPxe0xLCs7bcnrrLHZvEGe2ih2RuIdR6c93lWKUSEIkbehJRM0k8rRoIJOdDJ2g7ZQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=moDMJea4wxs6pNHpQHuj+wrsVLYtRSz1Pi39/EWEvUk=; b=lvc9Nx0P6r6NJB3AF1UNM0hOtPa+XtHHGqX0Vx/k7gW3NZC/iA+mtBNSbE/JDKMuoOd7TouncUA5vE988OfO1Qonj5obugpaLr38d2wnOy+huVHnNwwJTAE4fnBLxilpe1yBxiIKQ9lESeYzeSL9HFaQtTrUlT1aPa5iOJvxGMh+Z3X4zYq2YnQkTfDL+gnmtqHZZBTSksDYxg8LHRL4TeqKNhMvsQheFr8kcYcMf5VHqJ7SajhizjeqDIAPhnOmI/O99QHjI4iXN/5ZyKY+PrYgNFO00O+H1vZRo5kqCqH9rkQiM8J38nv1U6JQF7X4i2W65IrPCJslvs7lvry2Ag== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=moDMJea4wxs6pNHpQHuj+wrsVLYtRSz1Pi39/EWEvUk=; b=rUlo5ZL8wqJrqjBVXpWJZuvnWOYKDzJfqm22lnUIc5rnQup80Al8FOm+0L8o1oM/F7gxmwetim+KAROYEkDsD9bJjUXvXHR9HCz9BjY/5ScvXl0ufUpxV0scF/Z1mIHfKOS/fPIO2jUjZjYhRMMhjOzCwGiIUfOZdTb4glxoZrrEpoeU0gCKO9pYTcGH1FRhKnCvu/0AF60a7ZFgnTOU10iZLAEHY6fajsIMuqTqDT9fTDgaKvmTcQ5ruQJiYJqckS14KVtNCG3Mzi/h/xNPDTOOqf6pyngXmFtFOoeNvFBBD3Jz5vITPzdc4oI+y8xG80fLbIc9ZPSVPX6pmTrJ9A== Received: from DS7PR12MB6336.namprd12.prod.outlook.com (2603:10b6:8:93::8) by CH0PR12MB5372.namprd12.prod.outlook.com (2603:10b6:610:d7::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7270.21; Fri, 9 Feb 2024 14:01:41 +0000 Received: from DS7PR12MB6336.namprd12.prod.outlook.com ([fe80::72c1:f42f:adb2:cd2c]) by DS7PR12MB6336.namprd12.prod.outlook.com ([fe80::72c1:f42f:adb2:cd2c%7]) with mapi id 15.20.7270.012; Fri, 9 Feb 2024 14:01:41 +0000 From: Gregory Etelson To: Ferruh Yigit , Dariusz Sosnowski , "dev@dpdk.org" CC: Maayan Kashani , Ori Kam , Aman Singh , Yuying Zhang Subject: Re: [PATCH] app/testpmd: add size parameter to raw_encap action Thread-Topic: [PATCH] app/testpmd: add size parameter to raw_encap action Thread-Index: AQHaB95uCra9Gqm560uVCjkUBJySQrECf6CAgAAscYCAAANYgIAAAS2l Date: Fri, 9 Feb 2024 14:01:41 +0000 Message-ID: References: <20231026073047.300888-1-getelson@nvidia.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DS7PR12MB6336:EE_|CH0PR12MB5372:EE_ x-ms-office365-filtering-correlation-id: 98a737e8-4d2b-4d77-f353-08dc2977a439 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: wk0pCO2ZJ1BK4YeAWmNFc8JNSKWQfCdVYxbIhFtkRHY23oh/LaFDynL3JVgMMnEGkLvhNOAIPnGQUBOuRkfrGhmLcPS6LMs0QNNnPnqwCuvg0mPkwPi/JZpityr0g2PqMn2IW3B74muNgy/EhFFNNHt7tAWOqmIdmPAZX55vIosmoer2JZeei/bTAEbwAsagUf/W3ICMX7llDffmpu5xyNr/BUUiiYtmlsTSwdd10Vx7FJeZLysgmmrCF1uPWg9NnsQNjJw1hFBD3bZm0jt2R0+NcSq40ZlfZaSiZ5AgVtLJFFX8z0q0BsbvaydsRQqsycLpiz1WPBh9SNlGWhg35j4jmHhLfrvVB8nIwQ3ARYNYX/Kvg9wKem1vlilDYlsSm8+n7G+ZwO7QzLa66SQQm8CTgMFbMNuLpDU/hFslBM53MTilvD3ommFSgnuJkwSlBPL34UpC1wIP73xOUSureg8uOEH6Bh8trTxoEkzph8vKeVoJ2kAt3xwEixXKqWsPDUtM5Oo6niNz9vP/rmOx3q+UHpk9Bfd/Aa4/P1SVLCoJ80p5t9ck6IxFX/vrni/09qBdshuNN/lsEmPS34U6DZyTq3qD0guVOManlDGncHMnlrsHqNeIm7Bymu8KmnW3 x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DS7PR12MB6336.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(376002)(39860400002)(396003)(136003)(346002)(366004)(230273577357003)(230922051799003)(1800799012)(64100799003)(451199024)(186009)(55016003)(4326008)(86362001)(8676002)(8936002)(66556008)(66446008)(64756008)(33656002)(122000001)(38100700002)(19627405001)(41300700001)(2906002)(38070700009)(52536014)(83380400001)(76116006)(91956017)(66946007)(5660300002)(110136005)(316002)(54906003)(66476007)(7696005)(6506007)(478600001)(71200400001)(53546011)(9686003); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?MsD5tuL8OhKZ3BNyEKpC15sNe5vtFUl/jhXpyy7JrAPNqRtfGtc8VKfZDEup?= =?us-ascii?Q?j1H1dKHX3MSlmnZPpFv/AK50d2hUVVKRiPuI2VvDJFQ8+06rzEKUqmgFFF8Y?= =?us-ascii?Q?EjuPE7yroxcU68O9cA11oxPQYATVMHl/dT3s+jxJM5BgOzgeaHNauljpnhjr?= =?us-ascii?Q?h2N1J20kZtVdY+PD89fJKgiVIcTQjclqVL4xAW9EV08GCalR3WCvY+rw4sFU?= =?us-ascii?Q?G2OH4zC5jQ65pmhgiEdfzYs4pP4LuxhWVg1KvZF6kLEgcH27V8MD27wQBXcb?= =?us-ascii?Q?vmYRLVwDcVwxWa7QBM5uW3aUINSoDTTnhtTcRQiRiYE8byKNxOEFnLSOX88X?= =?us-ascii?Q?KdpYvp5ukihvCLx7PTsS8IDFcyZI0a53JgLeSQ1FfzEFpndjesgmrL+F4ukF?= =?us-ascii?Q?78dDvD3JS004q274UKEpPcGCLhghOlBXr61ocue6V6PPSKgcNHN2EFPPUjtW?= =?us-ascii?Q?ar4LpFEVvNGutwIhubHd6UiFGIRc1Eiugsg97JnzoLn4jawZTuBQgFL45rD1?= =?us-ascii?Q?aVAd8pu9avUEEyJ55DStvkJz4BqoIJlVLLfGbkxcM/KoQ0ukMkFjXl+/KpHI?= =?us-ascii?Q?F8VvMXQydYaqFuTUS9Bd+yT1QILqbIsfAd9mJRjC7jbxfc8q9Iu00VVjLqVP?= =?us-ascii?Q?f67Kgr1PdWnbxcvfbFjz29yAsUw+lXwzZtC+YIYnagB44+d+a4HNX1GnH8u5?= =?us-ascii?Q?Ts/gzvu4huevg6L3dx9984JOvjhLJ1GBQQ8yBhyJTBa3qRW+9YpTwzSHqkt/?= =?us-ascii?Q?fyRKU5j+nck7SFjw9oHeMbmVzLZSgtV8xBrTNH7vlwH4rf3wJCuqD3YuIflN?= =?us-ascii?Q?zizK6AkDVBY6TaBv40XuBmMRaTU22i9oOkR8TPQjxRk+f6P6deL+5JEKgVD6?= =?us-ascii?Q?VRwFCO1RhH5KQb/uc3yjTs5LHF9vP8CKyfT8LH8qctUvaOHSTvuCY5w0kMtr?= =?us-ascii?Q?BoXfKjgypUEvdr1U582sb5d5iNHiwUHVTSWn4siXsCX/uJDhWoveUwgQ6FP8?= =?us-ascii?Q?czNRPr94sRbi04sPVFV6OWbDd6Dqa2WSDx8bN8clVXkaHkjzQ5q7fnVAVJeg?= =?us-ascii?Q?oXdU6MTeMNl28Xp56ZDUFoz+tRC5peP64rk44C+gbm/XkJwdbnHgTtlL1R/r?= =?us-ascii?Q?KS7vBO+37Xj3hu2rCSo8NVIcFefC5hTxu+13WCb4AbiaED4MLD8QtkZZWsdt?= =?us-ascii?Q?L5CaFRMWC1/N8XAN7JANhZhv/kOcbBcJBGNE5CANuHLq8TRJdQaOjgITkBmV?= =?us-ascii?Q?QtSWa6zOhdyYaR93lxdIPYY3Rh1q7gWvHsl59nKu5EZWGjCanr2KTqndYdsS?= =?us-ascii?Q?BF009LEAV9mmeQMq1Z2QJTSn5vkSiseklIxYXg8SDuMtGMSQtysODEIS1JFq?= =?us-ascii?Q?VipyHMANOwxx8sLZWEOHe8lfjkCvZugq2TDQkG+2I2c8fhzxL+xHyYXNuIiQ?= =?us-ascii?Q?o8w1B9HkUkHPNRIm6TZJobkr1PmSDIsECOgm/lplrWHlpAVhiJBmzjV9TAnV?= =?us-ascii?Q?JSjFIvKLfowQgxsmUqAZLkTe2LP9ZAdbcw+9SDyv7qtwIXtJvF3QX1aB0bQt?= =?us-ascii?Q?mzcLVqx2NFaji8BDitNx7bsK3HatNqyhmC7oiDFUOmLje1RxaF2NzIaeLHVa?= =?us-ascii?Q?QOCXs5ft0wk8IAHI/zm01Hs=3D?= Content-Type: multipart/alternative; boundary="_000_DS7PR12MB6336E1DC9E1CDFFD16AC14A4A54B2DS7PR12MB6336namp_" MIME-Version: 1.0 X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DS7PR12MB6336.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 98a737e8-4d2b-4d77-f353-08dc2977a439 X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Feb 2024 14:01:41.2617 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: 90kBRt2sRxMKhvO4NfEZSITe5nunZKx5qm82IGy8QGRYsrPMKkjQ52i9t1fhz4gN8bptEfGWy0GRH2EE7zfkxw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR12MB5372 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org --_000_DS7PR12MB6336E1DC9E1CDFFD16AC14A4A54B2DS7PR12MB6336namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hello Ferruh, The patch did not fix an existing issue. It was proposed the code improvement. Let's drop it for now. I'll post a general solution in the next iteration. Regards, Gregory ________________________________ From: Ferruh Yigit Sent: Friday, February 9, 2024 15:55 To: Dariusz Sosnowski ; Gregory Etelson ; dev@dpdk.org Cc: Maayan Kashani ; Ori Kam ; Aman = Singh ; Yuying Zhang Subject: Re: [PATCH] app/testpmd: add size parameter to raw_encap action External email: Use caution opening links or attachments On 2/9/2024 1:43 PM, Dariusz Sosnowski wrote: >> -----Original Message----- >> From: Dariusz Sosnowski >> Sent: Friday, February 9, 2024 12:04 >> To: Gregory Etelson ; dev@dpdk.org >> Cc: Gregory Etelson ; Maayan Kashani >> ; Ori Kam ; Aman Singh >> ; Yuying Zhang >> Subject: RE: [PATCH] app/testpmd: add size parameter to raw_encap action >> >> Hi Gregory, >> >>> -----Original Message----- >>> From: Gregory Etelson >>> Sent: Thursday, October 26, 2023 09:31 >>> To: dev@dpdk.org >>> Cc: Gregory Etelson ; Maayan Kashani >>> ; Ori Kam ; Aman Singh >>> ; Yuying Zhang >>> Subject: [PATCH] app/testpmd: add size parameter to raw_encap action >>> >>> Testpmd always provides RAW_ENCAP flow action configuration with encap >>> buffer and the buffer size. >>> That implementation does not allow to create non-masked raw_encap >>> action in the template API actions template. >>> >>> The patch adds the `size` parameter to testpmd `raw_encap` action >>> configuration. >>> Testpmd can create non-masked raw-encap action template and specify >>> encap buffer during flow creation. >>> >>> Example: >>> >>> # total data size is 50 >>> testpmd> set raw_encap 0 \ >>> eth src is 11:22:33:44:55:66 dst is aa:bb:cc:dd:01:aa / \ >>> ipv4 src is 31.31.31.31 dst is 63.63.63.1 / udp src is 1 / \ >>> vxlan vni is 1 / end_set >>> >>> testpmd> flow actions_template 0 create ingress \ >>> actions_template_id 50 \ >>> template raw_encap size 50 / jump / end \ >>> mask raw_encap size 50 / jump / end \ >>> >>> tstpmd> flow queue 0 create 0 template_table 0 \ >>> pattern_template 0 actions_template 0 postpone no \ >>> pattern ... end \ >>> actions raw_encap index 0 / jump group 1 / end >>> >>> The new `size` parameter is mutually exclusive with the existing >>> `index` parameter. >>> >>> Signed-off-by: Gregory Etelson >> >> The following sequence of commands results in "Bad arguments" error, but= I >> think it should be accepted. >> >> testpmd> port stop all >> Stopping ports... >> Checking link statuses... >> Done >> testpmd> flow configure 0 queues_number 4 queues_size 64 >> Configure flows on port 0: number of queues 4 with 64 elements >> testpmd> port start all >> Port 0: B8:CE:F6:7B:D8:E0 >> Checking link statuses... >> Done >> testpmd> set raw_encap 0 eth src is 11:22:33:44:55:66 dst is >> testpmd> aa:bb:cc:dd:01:aa / ipv4 src is 31.31.31.31 dst is 63.63.63.1 / >> testpmd> udp src is 1 / vxlan vni is 1 / end_set flow actions_template 0 >> testpmd> create ingress actions_template_id 100 template raw_encap index >> testpmd> 0 / jump / end mask raw_encap index 0 / jump / end >> Bad arguments > > I bisected the tree, and it appears that this issue is not caused by this= commit, but it's an existing problem in testpmd. > The root cause of "Bad arguments" is the fact that when parsing function = for raw_encap index is called, > parse_int() is called with size =3D=3D 0, but raw_encap index size is siz= eof(size_t). > This causes a failure in validation introduced in commit 913b919906da ("a= pp/testpmd: add size validation to token parsers"). > > The same issue appears for other cases where parse_int() is called with s= ize =3D=3D 0 e.g., parsing RSS queues in RSS flow action. > I'll provide a patch for testpmd which addresses that. > Hi Dariusz, I merged that patch recently, and it is still not merged to main repo, so we can drop it from next-net if required, instead of having a fix on top of it. That patch is because of a defect that overwrites some memory, and side impact you mentioned was not expected. Can you please sync with Gregory about it first? --_000_DS7PR12MB6336E1DC9E1CDFFD16AC14A4A54B2DS7PR12MB6336namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable
Hello Ferruh,

The patch did not fix an existing issue.
It was proposed the code improvement.
Let's drop it for now.
I'll post a general solution in the next iteration.

Regards,
Gregory
From: Ferruh Yigit <ferr= uh.yigit@amd.com>
Sent: Friday, February 9, 2024 15:55
To: Dariusz Sosnowski <dsosnowski@nvidia.com>; Gregory Etelson= <getelson@nvidia.com>; dev@dpdk.org <dev@dpdk.org>
Cc: Maayan Kashani <mkashani@nvidia.com>; Ori Kam <orika@nv= idia.com>; Aman Singh <aman.deep.singh@intel.com>; Yuying Zhang &l= t;yuying.zhang@intel.com>
Subject: Re: [PATCH] app/testpmd: add size parameter to raw_encap ac= tion
 
External email: Use caution opening links or attac= hments


On 2/9/2024 1:43 PM, Dariusz Sosnowski wrote:
>> -----Original Message-----
>> From: Dariusz Sosnowski
>> Sent: Friday, February 9, 2024 12:04
>> To: Gregory Etelson <getelson@nvidia.com>; dev@dpdk.org
>> Cc: Gregory Etelson <getelson@nvidia.com>; Maayan Kashani >> <mkashani@nvidia.com>; Ori Kam <orika@nvidia.com>; Ama= n Singh
>> <aman.deep.singh@intel.com>; Yuying Zhang <yuying.zhang@i= ntel.com>
>> Subject: RE: [PATCH] app/testpmd: add size parameter to raw_encap = action
>>
>> Hi Gregory,
>>
>>> -----Original Message-----
>>> From: Gregory Etelson <getelson@nvidia.com>
>>> Sent: Thursday, October 26, 2023 09:31
>>> To: dev@dpdk.org
>>> Cc: Gregory Etelson <getelson@nvidia.com>; Maayan Kashan= i
>>> <mkashani@nvidia.com>; Ori Kam <orika@nvidia.com>;= Aman Singh
>>> <aman.deep.singh@intel.com>; Yuying Zhang <yuying.zha= ng@intel.com>
>>> Subject: [PATCH] app/testpmd: add size parameter to raw_encap = action
>>>
>>> Testpmd always provides RAW_ENCAP flow action configuration wi= th encap
>>> buffer and the buffer size.
>>> That implementation does not allow to create non-masked raw_en= cap
>>> action in the template API actions template.
>>>
>>> The patch adds the `size` parameter to testpmd `raw_encap` act= ion
>>> configuration.
>>> Testpmd can create non-masked raw-encap action template and sp= ecify
>>> encap buffer during flow creation.
>>>
>>> Example:
>>>
>>> # total data size is 50
>>> testpmd> set raw_encap 0 \
>>>          eth src = is 11:22:33:44:55:66 dst is aa:bb:cc:dd:01:aa / \
>>>          ipv4 src= is 31.31.31.31 dst is 63.63.63.1 / udp src is 1 / \
>>>          vxlan vn= i is 1 / end_set
>>>
>>> testpmd> flow actions_template 0 create ingress \
>>>          actions_= template_id 50 \
>>>          template= raw_encap size 50 / jump / end \
>>>          mask raw= _encap size 50 / jump / end \
>>>
>>> tstpmd> flow queue 0 create 0 template_table 0 \
>>>         pattern_templa= te 0 actions_template 0 postpone no \
>>>         pattern ... en= d \
>>>         actions raw_en= cap index 0 / jump group 1 / end
>>>
>>> The new `size` parameter is mutually exclusive with the existi= ng
>>> `index` parameter.
>>>
>>> Signed-off-by: Gregory Etelson <getelson@nvidia.com>
>>
>> The following sequence of commands results in "Bad arguments&= quot; error, but I
>> think it should be accepted.
>>
>> testpmd> port stop all
>> Stopping ports...
>> Checking link statuses...
>> Done
>> testpmd> flow configure 0 queues_number 4 queues_size 64
>> Configure flows on port 0: number of queues 4 with 64 elements
>> testpmd> port start all
>> Port 0: B8:CE:F6:7B:D8:E0
>> Checking link statuses...
>> Done
>> testpmd> set raw_encap 0 eth src is 11:22:33:44:55:66 dst is >> testpmd> aa:bb:cc:dd:01:aa / ipv4 src is 31.31.31.31 dst is 63.= 63.63.1 /
>> testpmd> udp src is 1 / vxlan vni is 1 / end_set flow actions_t= emplate 0
>> testpmd> create ingress actions_template_id 100 template raw_en= cap index
>> testpmd> 0 / jump / end mask raw_encap index 0 / jump / end
>> Bad arguments
>
> I bisected the tree, and it appears that this issue is not caused by t= his commit, but it's an existing problem in testpmd.
> The root cause of "Bad arguments" is the fact that when pars= ing function for raw_encap index is called,
> parse_int() is called with size =3D=3D 0, but raw_encap index size is = sizeof(size_t).
> This causes a failure in validation introduced in commit 913b919906da = ("app/testpmd: add size validation to token parsers").
>
> The same issue appears for other cases where parse_int() is called wit= h size =3D=3D 0 e.g., parsing RSS queues in RSS flow action.
> I'll provide a patch for testpmd which addresses that.
>

Hi Dariusz,

I merged that patch recently, and it is still not merged to main repo,
so we can drop it from next-net if required, instead of having a fix on
top of it.

That patch is because of a defect that overwrites some memory, and side
impact you mentioned was not expected.
Can you please sync with Gregory about it first?

--_000_DS7PR12MB6336E1DC9E1CDFFD16AC14A4A54B2DS7PR12MB6336namp_--