Message ID | 20170530010259.48823-1-yskoh@mellanox.com (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Ferruh Yigit |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id A8BD82A5B; Tue, 30 May 2017 03:03:22 +0200 (CEST) Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30073.outbound.protection.outlook.com [40.107.3.73]) by dpdk.org (Postfix) with ESMTP id AD6A12647; Tue, 30 May 2017 03:03:20 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=GSaAnrh9GlwpdVIiDTY8/HO3wEC0Zz5wH/A7T+g30Os=; b=pPP/l+UFQ5DZBkLckC3wvSjiEAwZJ8wUrFSQNlLHn/pQVzKldRJsYyClbHYfuwQ04kg8SzzzZyZAhz1P6HTaXiyj3Gfox7GjCfUeNOu3x4bLUp/O3koLVzCz47Bajl3YfP4D88BKwaQ6ZODAllqBNRGwvi+CT2iEvCBuhS6u7IQ= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none; intel.com; dmarc=none action=none header.from=mellanox.com; Received: from mellanox.com (12.250.235.110) by DB6PR0501MB2039.eurprd05.prod.outlook.com (2603:10a6:4:6::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1124.9; Tue, 30 May 2017 01:03:15 +0000 From: Yongseok Koh <yskoh@mellanox.com> To: <ferruh.yigit@intel.com> CC: <dev@dpdk.org>, <adrien.mazarguil@6wind.com>, <nelio.laranjeiro@6wind.com>, Yongseok Koh <yskoh@mellanox.com>, <stable@dpdk.org> Date: Mon, 29 May 2017 18:02:59 -0700 Message-ID: <20170530010259.48823-1-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [12.250.235.110] X-ClientProxiedBy: CY4PR1001CA0008.namprd10.prod.outlook.com (2603:10b6:910:3d::21) To DB6PR0501MB2039.eurprd05.prod.outlook.com (2603:10a6:4:6::21) X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB6PR0501MB2039: X-MS-Office365-Filtering-Correlation-Id: c77bfb70-5868-4093-883c-08d4a6f7a80e X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081)(201703131423075)(201703031133081); SRVR:DB6PR0501MB2039; X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 3:3yYi5fCXbQ6ADHkLGtFj0RhCgtZhOZNcEpewxRQbgM0S9fFh9BlOXdSf8UBRA15MDds7E3uAWVH9GjkGrUcfqzAlr7uSTwDOdUriaQNHcHxGTANBGfVT31LBCNJOvN/t89TPb2f0tMmdjCrL7Y4si3Atoa2qVObpr79Fia1l9aJYX80BA9WTiJOpG2FUcrtPQltFJf4FNnezpz4vaxbrMuBD4p5UkNyQ6wueb3Re6TzUkR3m+t/CR0HZab7IIG+7YNSU3S4wVF/vIs+kF+cmgPhrpeN/YTq+n2WNB7qqwZTdxn2bu4AqNpJYVje4iClkjaP8dpvkHJRO8SPYMKVP4w6qX+eR4k7M9SDrI+5qZlo=; 25:xVHT20VBGOSbPjzH3q+Qo+Ueoh0/qkop9+FSfVEUsceiIf+88VYSEsUI7BK3LnQIsNsfG1c4C0sBYL0TVFN9wSCHFW0qkexNGT11HpvYdxwLlURgC1gStXBSg6JYkfdhoHnvcOqhNhkINhQfpygNqGyBy4DttJwavKiLn65lcl6epQSUfj7xEO/3C7mWKRZ5yLoaxWy2Lhuao4HleEP44ZsnYB0WbM8RjamO+KXCDt4Wes4abCgWS8YZ4wioNNl83Vp96ZGcTfAQnnQxLGXUbzArTgJP2RGoDrqju6xneAREkoQHEz5R7Xnbn0SrlEyvHG4r2n2y+FFUJ8630ufwvoFVyb8JkN+RAzLCB0yplvznvp1fhbKF260NR+ol2aXl/fQ6P5u1SRtj3b3esInNIxTFFXHEutV7M8FuWpcpz4LhpifLpirsIFXh4TttU/30cfJJdrKCCzWILlIV5535gH+ybPePk9GMy4M0RbzXrFc= X-LD-Processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 31:YrCCaXX09v8cvAeeNDyJYbWkcb2r5D550aCyaZb5NdUZi1eOYAR26EVXBlk8+OWYlu2RJl5AwapTO1sYj8uJr3HWK/dGBglX9afmFOn+HXE6cb6prUCS98IWcYodyAXY0lZv9aGXqQoTf/rV8gnQG/LtrWM39/iCVhQj2s6oWaI9iheDoFfqaeMKleVcQa6//YKVRWQvcKpT3fPrGd4rqWRUDEoVCBiWEowUt+mIcOk=; 20:3Xb4wIcJk2KslfH9ConWe39eGGzaZ7/YOZMyUWf7EEYBpAXgsUtOll0LOD4BrnTRsr0ItAJ6TMSY1F+kHf9KmL7qsy9Rcjr5LbAJn0NzPx2c69bJKtYF/V5rMTH4B/Kmu2Pm87wX4MzI1cJvq0+XfdOKSWeN3ZT6MypFItjrWo9ZpEnlZiW6aAj2V1LdONrqpQNNK++tATd3lZLF9Qbxm1cPQuF+G+mkJ18CJ896UkW0QIDEkfe7SbUpbs1i43dh4o4idX9qs9+NSJcpNIA8Of/HIqKqdaIN3YQXvQ1J9af1f5blfU2ACx/d1Le4aaV1oQwkGE5KE8Vu28OArLBbxL05pv+OEgVvsqqbIHECp3Pc4a96ozOzctJ4NKwT33SRdcCg8o2Z72Cy2Me6nER5yWp/hL6Mr9DujfhkVH8Cr54xqDhQWzJmOTC67XVO620yUV3ibFXaX5yCq3fkL9qARjnwvLYlMjZwEIrgb9Tjsfkeu6NQH2Fg+iRGEgW9bCLH X-Microsoft-Antispam-PRVS: <DB6PR0501MB20399969903331D87F311FFDC3F00@DB6PR0501MB2039.eurprd05.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700073)(100105000095)(100000701073)(100105300095)(100000702073)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(100000703073)(100105400095)(93006095)(93001095)(6055026)(6041248)(20161123555025)(20161123560025)(20161123562025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(6072148)(100000704073)(100105200095)(100000705073)(100105500095); SRVR:DB6PR0501MB2039; BCL:0; PCL:0; RULEID:(100000800073)(100110000095)(100000801073)(100110300095)(100000802073)(100110100095)(100000803073)(100110400095)(100000804073)(100110200095)(100000805073)(100110500095); SRVR:DB6PR0501MB2039; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DB6PR0501MB2039; 4:J7RXEaTzXHWRafzIyDykEQJTUIprVRl4ZwrR5ht9?= UmSWo7OlBzpI/amD13izTkttR2UqaoOi5jkIlTpQGxIzjEsQizBqq5rxF8PXBw0ML84QOQZrgfWaM0IzTnokfr3Nb3ZXXG2P55i/ZdHo5aEJw/Z9IvFVHzFufrx1liJSytIY4x/uR9NpaWlDWYRPqxqQtxUCAgxJ1+WxPx8tcs8B5/C9i3rppzmfyeKHW+SUZ45uAsqzpqDIcWX/oaaQOUrR7NpgzLV6nxef/gd9dufKqxRvXsZPLG0f+CCS5vEQ09IewpWfHTRVClCx+ZH1h244+hAP2Id9/pjR2z+rP/UNs3LIy/OCJR0HxBS38itArG1f/+nZpJMbh7tfrIXqkRtazEbleHhPQuwowZ0CqKOX0DGGY1O/bmxB6JhvgCN9zxn9rbkMmNj4NeQ5eThrlYfBdMbVh8T4wnHAPcRjlFlxJokreN+75A6kiXjH6d38eYBTdZlojCR5FbrnLHDsDMCwAwcTMU2LujATVx20FH2+QHgtZADfYzqXGt2jrVb/mk1z4oEEGkajnj+P/r9R7DAmZZKsSBeuTBiDohAJ2SnO3Pa+Pcn7IQ/bjotkRLekHG9WOUSc5Ra0tK5q2LVU9OeFsLknhdpwW3C8LxT4UfLPqCGXxS/qUMIpSt4zFxRn2bxcexCe4d8lB0igKYsGhYfv6x9M+QWcosM9WZU/pfDCk5UawBAT1ftIEqwI2AYS7B7tnRUzj+Q4PEVsaJxkTwdMaQy5aZ6+YT9bqkMl7+0PXFotyzmN7Gy8nW/9J/e7fIGDLCblI6n89DaDcX00YswD2VEsVnpTbcMZ7HoSADInNlfxdTB5QGMLIzcXroTARNlNUKLoNMfP4UkOk1y1qFT9tzJf5jgbs7u7maY31unCfpDTQVQB32OHSDszSrBPFZceNiVygoSKVKOxE8XlPX60njBkeAxY7aWzw6tuTkOkCHkegEdnAuqz8wNzMaLzROoCXHCQarXAgZXPpzRpPEm4joLOZlyA6eGvwdojuTmKMaSFG7mLtgafuhrbyUymK9mC5Kegbd8Ay16zDDIagl2JUByVKnE78US/4sICv93COwI9a0ZX+LIbvEh8Ey7ZVh8/8Zjm9dET/e1RHG3m3kTuaqOkTHiEWX/cSlINGuFKsg== X-Forefront-PRVS: 032334F434 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(39410400002)(39400400002)(39850400002)(39860400002)(39840400002)(39450400003)(478600001)(6666003)(36756003)(2351001)(54906002)(42186005)(55016002)(33646002)(6916009)(189998001)(50986999)(38730400002)(110136004)(33026002)(53936002)(5660300001)(305945005)(3846002)(25786009)(1076002)(2906002)(50466002)(47776003)(48376002)(81166006)(8676002)(66066001)(7736002)(5003940100001)(86362001)(50226002)(6116002); DIR:OUT; SFP:1101; SCL:1; SRVR:DB6PR0501MB2039; H:mellanox.com; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DB6PR0501MB2039; 23:f5vmISiW6nA738UHsPHiSibjEOurbbbBcgwjq2W?= ZjDgRXRaitC58j+4Zn6UYwk/MCGEJanm3N2Mj35u3Tamr4Y3UgiZDJt/+PrCxeZizLSs/dMARcpg8XEFnMOOJY+eI2hcXqemcUfsgdBlQGQwQZ2VhOTOYrlOsUuXBZ5KkCDCmYkhHI4xCNC1sCD0cMo/0amzbRjaSeFt/erdR35ogsdh4Ndv1bWWs+dvm20d0hKjK1nDoNkcyA4aWYxlSx0+Cw9zqA0i6DQdC4RcPdJRiKf/ZY/trU7ndnNQJivDzhNLiFRRoPSWIQNUlN6UV0ETWfw9Ht7V4TpjsJnsTHfmP44p0Q91W3CM9PkTEYjsMemmVwvYapxA+a5gYq/NfyO5fBIAt5ieSo/6rkvFd2YhCZ5tZS9Xq9RbSWd/wOSWFtJUmxiGcCaqDNgeL8vMK5rqp6FasKI1hteEjq7kiK7jOrc+cVbBBEBD9i8f+kRs97So2M2eJAOmCMlvC6lg+ADYpvhAQa1aZmIxff8qXWMPhaBUvHkNYgFqvvMc/5dVv5Q0ANpB502slQUuj+96r/N6TQK2PuEngVYB+e/uG0+4mu8ViWRm2nJ6HofAMX9QBmB77kZ/L9fQKGbucCV4ggRRVgah438MH7ulN4b7vBZeAEXfK2dDAvxOFSgUZo14gI3aoaTe1x0u3gkERyjnSHgWtDdfndEBgMUFXkELWv0ifkWp4Hfjk/5A2CQqsdAxGxw8ao97cxhg1qs4LrxKgwIBcY6C4ytS20gQTOPfwRYiUwi6lpPXmWZmbpLkPYZUXwzPSkf5UakvxathyR/TC1Lmqulyn/Zk9FONfZSPso61O2NSbp254LrDqczL/HURRvZG1fvgBWOIKS5IrMlen92EVEEUdneRPLncBZmhFaiMjj679kcFrYqF0zoDZ4Yl5+HAvm7XvT7+gwoDBjAShs3JgmllNdEKbCbA0ltl7iqL2nLQOBJBLh+fBzKuPpr7yjPpwF5NRaqMhZ0G3i78OWkLazi4Sp3yZuo20XcDrdIYoiw== X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 6:HXIUVQ2f23BSxxcsUr8TKiOrNZFz8H0xz3nmoW8l+yK9RQyBC/ZC6EJfel7uEhQOq2/Q7LMrZx4etGVhInYiFV2faS5hHNSj1/GecFvyPraHB3DYbSe+5vnfjr2o5yBkIH/D8Tfwgqv8+4AcbZY7quv0E2TZU6D6dcaTh/JuO0lJ6VaHQePcoX4ByH/E/5KNxT6KWkiXN97+eKzxmHJSzr5nfiDklgRPFpwG9+e2JgT9xb0/3BXpIcDkJ9+rUcPiTgb2AXEZnDJPZfhFp6ournZ4uAb2VhGN58PPcRuD/+HtTf+deUbD4+O3G4AQLiM0wcHH2ZAxIHORFqOiXfqHLKaj9FFvmS9pHrA12jdNRbOy9Rbvv3zPFBbXRaTdYcOL0DLkn8WROka3M3NdwxxlaKgVR3W+HMDP9PY7CCSOuvMpVzXdd/pc1bKidrMg0teqeq5HyxEmoexR7ngXpHJMZXRtSUb7v02G3iH3+tcABJL/FUsN7fgnTSGgWAifCDv5xgHx/9tO6SSCZFQZplZKMVyGqEEKxlU3R4ol9reLQ4a+cdSEsHv4gMKNOS3nauXbGbqfuEJZjuNHO4i9cKfTdA== X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 5:rI6EnI7GDSVYQV0/8AI/9QSbt91+1UncW6hoX9mB2K8I0/0SQb/DiuB42RYE5yKKUjJdCvXc6kIgmkloOoHfTd3CCC4UmNLTl3R642XQEwmNEjYo6zwy3dGZHvNLL1NywnECHlUr+1g3Q/cs1dkgVZi7bLIrJkB4InnT8STs2tG+msR0rQ+rBsO6uzaTPWl1R0uG+uyr/zszmf5vXsCuqZd9oiGcqPt8emQL8ZDooybvnhL3mboV4JogyMDtWrHvP1701euJszSEkMKpwLrwno+yLiVh2IE9IdhZEnfhHdaeTGr2VMJalBOJhJUtsl6gaeC3JNYwKOT4kNhAz2g7k8xvCCo5JJWhiLPzX0TQtumoZtqbe+/mXtDHXOCIuCGdTUvbjB2C2x63sWJ0RENoTvzN7kZksnEBxsWYSXhJFUSPnQ4U1yHBvUGqIJKAyP3G7310ggb4iaVDJ9/C9uZrJ1FEjXoGKZTCi16wh6MiKTsAkHV/ruGuly81ityxS28V; 24:P3VconZ4PC+oahd4+3xKGkPw3a82VMhtNaOCJ5ne87x4oVetlr9tShCWwuXFVAs/8FbbvEP7nO3LH7bLzxwwVk0RXLSqGTUC4rMBi/f2avE= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 7:EOuj+A+tYFRuG2p8w+2IRBuZ0YYuSN07+kcip4V5+EqPlZG3J1DPdnUDta+qX+pdv+2o0lpDpWdB81bBADwYWKWhVft5kTUnMFGfDXfqySwp8iDFVD9P51wJF43fLvrZHvj6LkNn05Fv5MIPGRhubNu4KLo4m/tFquuLR2rEB8LDbZjE2/kZ3u3XV8WmR3g9wYTO1nL5/boqtCVF4lryyqcmz6Utpi1Gh5cPWWQBrVhhVLjM/2JlT2hU05kALL+OLeH5zQvBP/AX3kZzTkfYsvDybp5uvdOMYoqy8vMNbQcSn8HzyFOWd8UY24y5u2Y+99iljbw0QxV1jsWHW0iZXQ== X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 May 2017 01:03:15.9302 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0501MB2039 Subject: [dpdk-dev] [PATCH] net/mlx5: fix wrong exception handling X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
ci/Intel-compilation | success | Compilation OK |
Commit Message
Yongseok Koh
May 30, 2017, 1:02 a.m. UTC
A sanity check is required in priv_fdir_disable(). If resizing Rx queue
fails, this can cause a crash by referencing a NULL pointer.
Cc: stable@dpdk.org
Fixes: 76f5c99e6840 ("mlx5: support flow director")
Fixes: 0cdddf4d0626 ("net/mlx5: split Rx queue structure")
Signed-off-by: Yongseok Koh <yskoh@mellanox.com>
---
drivers/net/mlx5/mlx5_fdir.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
Comments
On Mon, May 29, 2017 at 06:02:59PM -0700, Yongseok Koh wrote: > A sanity check is required in priv_fdir_disable(). If resizing Rx queue > fails, this can cause a crash by referencing a NULL pointer. > > Cc: stable@dpdk.org > Fixes: 76f5c99e6840 ("mlx5: support flow director") > Fixes: 0cdddf4d0626 ("net/mlx5: split Rx queue structure") > > Signed-off-by: Yongseok Koh <yskoh@mellanox.com> Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
On 5/30/2017 8:48 AM, Adrien Mazarguil wrote: > On Mon, May 29, 2017 at 06:02:59PM -0700, Yongseok Koh wrote: >> A sanity check is required in priv_fdir_disable(). If resizing Rx queue >> fails, this can cause a crash by referencing a NULL pointer. >> >> Cc: stable@dpdk.org >> Fixes: 76f5c99e6840 ("mlx5: support flow director") >> Fixes: 0cdddf4d0626 ("net/mlx5: split Rx queue structure") >> >> Signed-off-by: Yongseok Koh <yskoh@mellanox.com> > > Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com> Applied to dpdk-next-net/master, thanks.
diff --git a/drivers/net/mlx5/mlx5_fdir.c b/drivers/net/mlx5/mlx5_fdir.c index f80c58b4d..1cff41bab 100644 --- a/drivers/net/mlx5/mlx5_fdir.c +++ b/drivers/net/mlx5/mlx5_fdir.c @@ -733,9 +733,11 @@ priv_fdir_disable(struct priv *priv) /* Destroy flow director context in each RX queue. */ for (i = 0; (i != priv->rxqs_n); i++) { - struct rxq_ctrl *rxq_ctrl = - container_of((*priv->rxqs)[i], struct rxq_ctrl, rxq); + struct rxq_ctrl *rxq_ctrl; + if (!(*priv->rxqs)[i]) + continue; + rxq_ctrl = container_of((*priv->rxqs)[i], struct rxq_ctrl, rxq); if (!rxq_ctrl->fdir_queue) continue; priv_fdir_queue_destroy(priv, rxq_ctrl->fdir_queue);