Message ID | 151386204146.3724.1821581875608154672.stgit@localhost.localdomain |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show
Return-Path: <netdev-owner@vger.kernel.org> X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=<UNKNOWN>) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=virtuozzo.com header.i=@virtuozzo.com header.b="P9EkoRDQ"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3z2XD12G50z9s7m for <patchwork-incoming@ozlabs.org>; Fri, 22 Dec 2017 00:14:25 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751807AbdLUNOW (ORCPT <rfc822;patchwork-incoming@ozlabs.org>); Thu, 21 Dec 2017 08:14:22 -0500 Received: from mail-eopbgr10100.outbound.protection.outlook.com ([40.107.1.100]:13920 "EHLO EUR02-HE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751622AbdLUNOR (ORCPT <rfc822;netdev@vger.kernel.org>); Thu, 21 Dec 2017 08:14:17 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=virtuozzo.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=O/hozORWWXviJsB2rrQtHmQAsLP0Iqk8Ao9PavNeaeQ=; b=P9EkoRDQG89VoaCJnUWd6BKcVTt0Xi2KJS5yhViqi4cTJTTKNjhJXP6Q7AYcJtla8Ae8Sp22aevImp0rmQi22GTGHBKKMcZrTLmov5ui7bLHQuUtajtLvDLoO+SM5AOyCJZEmc4sH4x+iuYHKc2045fd1ctZtYkcjNdh/6aEYzA= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=ktkhai@virtuozzo.com; Received: from localhost.localdomain (195.214.232.6) by DB6PR0801MB1336.eurprd08.prod.outlook.com (2603:10a6:4:b::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.323.15; Thu, 21 Dec 2017 13:14:13 +0000 Subject: [PATCH 2/3] net: Add BUG_ON() to get_net() From: Kirill Tkhai <ktkhai@virtuozzo.com> To: netdev@vger.kernel.org, davem@davemloft.net Cc: eric.dumazet@gmail.com, ktkhai@virtuozzo.com, ebiederm@xmission.com Date: Thu, 21 Dec 2017 16:14:10 +0300 Message-ID: <151386204146.3724.1821581875608154672.stgit@localhost.localdomain> In-Reply-To: <151386201910.3724.7199367937841370542.stgit@localhost.localdomain> References: <151386201910.3724.7199367937841370542.stgit@localhost.localdomain> User-Agent: StGit/0.18 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [195.214.232.6] X-ClientProxiedBy: HE1PR0102CA0024.eurprd01.prod.exchangelabs.com (2603:10a6:7:14::37) To DB6PR0801MB1336.eurprd08.prod.outlook.com (2603:10a6:4:b::8) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 5aaf94a0-9166-4b29-19a1-08d54874bb73 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(5600026)(4604075)(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(2017052603307)(7153060); SRVR:DB6PR0801MB1336; X-Microsoft-Exchange-Diagnostics: 1; DB6PR0801MB1336; 3:ap3gbJy77bBLbG0UooSt0CLD86CokHh/HLsAmD9CJftw0MFWK1wCiDauw3nVXuf3HcqjNmMFwI1c5GCLy5AV0czxqu/LdjBJzwkLbf/sKmaU1Ewf7bn9utUy6h3a7oEekHlh/O7TSmoDmla3LL42JSpJ9u6/3Zns11z26Xuj8ztz2fFBpMJeEa82X1R0gZ0D/T7NJDPKt8BnUlmP0q6oLKzK+403RvKXRTxzKJo52ZgYi1RQYMQBfD05PynLg8O4; 25:H1g1Xrh6IprKIcdrCDN52B6cgKQFAzhtNmuq0KQ6BSNktLTXSoQEiSMlCnValoHpMqI3IkgdBamriafRAN4UongualwSk78kn3RR5mXQEKEOYug4tOKpSbrmQMV352+7VF9HErh0jVP9jN4roGyVqbfYJmEc+GsL5C9dDkjCGfzjCbSImbwkJYPHPrIOAWfGjClOhnsZB9GHNKmhxVMCaJ8j2T13gP6qCl8wLjn8egZimZptYqE/kCQ2nZjMBvBqyJkp3kXK36v0n/Li9SUKuJ2aVMF4yUlINNZTWMGyBtiqkvNJMmRzC7QgUdxe7sCHhgZL4yLeLu9Gqpu5btJbOQ==; 31:tx8Nctj9aiVQBjdUAYyJ02wqejNbjBURzzdsHkHTzm10mYcuPRyVBjH/XFMgvfVortPOqDOjVsDVY8RHO6HgF9rkZFfA25lCAZJpqvSlSIjv4Wyknk/xqGwMSS2CL8JThnc0NWvF0IyMPJPG+I6icj2kY67+qmNn3mi0NwViSIkg8Q//bXtbq4AMWwy+S099K6nFl9ESeJjz6Ox4ckWyXRXx7aFkwQiwjgSragc7/ks= X-MS-TrafficTypeDiagnostic: DB6PR0801MB1336: X-Microsoft-Exchange-Diagnostics: 1; DB6PR0801MB1336; 20:b8FvyWVeTOI4Xmw4JpoRSVPbF0/MiUV8d9C6r1joNUd1iP7ENIBg3D//3QawzXQGnv+6Z4E+OGeZvsjTPrIkQQ3S0w0AX9jpU8JyW4snqdzQtJG7WJs2kkL/IBgu0aa4jDvhTuIvdEis2nu2FpPkSyfLI/TPirvuAAuuPmxc3Vud/DFB5luAB5Q1TRgi4fqp4pNIynOhveqlJqKRfX9ocePLSlu4rZ8ZJPlTvGLxy9VmgEfkfn8OQm7A5MjtsSPhrdT6G9P34IqRQpuGgR3ErIM8gaq/buxl/LoOWw7wcjQpel2uk9jTjh/WiH8zPWe2Z7BoTiJRiXkgc8AfQn4i1d/S6EEzJ7CD6FhbY4lTSW3k18Uk7KDAyG6PAJ3PaoKzV2Pn74mov4Y1AIn51w32Di/4mKxuuSPIM8YDlM5SOak=; 4:a7CvHlnstAeFIYLwMOc5r1RZiJbJCLGEe2f5jCG05OYN/Fh+mHGAD98NMtfBY9ysjHhxKY+I8J6RtCfXylxaygwGwWPrRLxqQ5WK2FfC4p9LxsgHxMEklQMCh/Hg3PpTSYqEd7h4mV+5CyGysRJBKv4yFvR06h29WKjgW11K3hfYPDVHGXKO6NOmzlHCqUhjVUuOr3cEXR90223A9IhJcv1nSMyM+BVp3N2Ke6ei457VzCNVXl0RfjcscydNK3306qTS8LbW4T19+phrPe1Oyw== X-Microsoft-Antispam-PRVS: <DB6PR0801MB13368B19555AC6372704777DCD0D0@DB6PR0801MB1336.eurprd08.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040470)(2401047)(5005006)(8121501046)(3002001)(10201501046)(93006095)(93001095)(3231023)(6041268)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(20161123558120)(20161123560045)(6072148)(201708071742011); SRVR:DB6PR0801MB1336; BCL:0; PCL:0; RULEID:(100000803101)(100110400095); SRVR:DB6PR0801MB1336; X-Forefront-PRVS: 0528942FD8 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(6069001)(346002)(39840400004)(376002)(39380400002)(396003)(366004)(199004)(189003)(106356001)(105586002)(53936002)(4326008)(39060400002)(50466002)(103116003)(2906002)(478600001)(6116002)(68736007)(230700001)(316002)(305945005)(58126008)(47776003)(3846002)(7736002)(16526018)(66066001)(386003)(97736004)(2486003)(23676004)(6506007)(59450400001)(7696005)(52116002)(81166006)(81156014)(33896004)(83506002)(8676002)(76176011)(8936002)(5660300001)(86362001)(2950100002)(575784001)(25786009)(55016002)(9686003)(61506002); DIR:OUT; SFP:1102; SCL:1; SRVR:DB6PR0801MB1336; H:localhost.localdomain; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: virtuozzo.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?utf-8?q?1=3BDB6PR0801MB1336=3B23=3A8+?= =?utf-8?q?PCXKDSMjwkpgOfDPyPLAPwSoxZX/7WI0O1LZEKzcKnJA6/fR2/LfnVK0?= =?utf-8?q?Xfx97eSh2ZwOJFIPW3GpVqRJjqvcS8dvUhsytHJ76h7OLjpIMWllcWSD?= =?utf-8?q?WOl1ZcRS4mZBnuwm2uhm6msg1/bQl6FNd1wQWPhCCYZvo8dU1KvDioxU?= =?utf-8?q?DTmNuQ0Yr0SawilJpbAqqJtoAJXIRU8PdwO/Mx7jjKFR+1N6jaFth1Qj?= =?utf-8?q?LVbHXWF/F84/nUj0b9nwuhP1UL+Yjkm9U5LbfxmalfmGT+/qU/SRwbhJ?= =?utf-8?q?EL4ZA0yWGJzYV7pz/+xVuLFaVlFo17sDroKPj77iQfJLGsQKMwhiMICD?= =?utf-8?q?9l/6GICWHQ5cV7FkXYSyssIUJUWnI6rGOYrs2KFaqG4y1wKZVB7dUQjs?= =?utf-8?q?Lv3XpVNIVyyra6trJGgpIUBplmPkr5BgD2SB4Y6KGk7L/TK8WcDupWuo?= =?utf-8?q?yiIphPi+aODjGnO3ZV8BqDff8xZ5s5Jl1ko5KtPC/npaXthGgoS2OBPc?= =?utf-8?q?m66RpdETRs/3OKvb2UwQcndQmZE1ixeCpp2Y3wFGEEUIuyzn0HAYcEdJ?= =?utf-8?q?u4OyDDpy8Zbc0VMDBk9KTCxesMBKcvrS9AAB4AJAIJOjhwJapMX+tF80?= =?utf-8?q?ocahRuNh+pWdnfQ0VKxCnI2CJ4kxnUpLNMPDvZDOEOYKNdVxp/JDdQ2W?= =?utf-8?q?ASGejtA0MHNBkeCZGd/ZdNqhncROV8Zq/Q/sFBKTTSSLYbvCsP/Y8To0?= =?utf-8?q?ZJNfczipgPYvb9po6ledPq6ijsrDEsSnOOIeedhRxYeSTSD4vzvAzbdd?= =?utf-8?q?rR3FM8UgNovYr2FGEmvlS3cEW9VAb/Xb6A/WYyvfBDN1ZRS+pcSSk2C4?= =?utf-8?q?DktpmAsdvu9WhKCBYKpCqHw+CtuxWMx306BQbaSY1Jnzs65yOIesE/Hn?= =?utf-8?q?bJl+ljeJ5MDYViHJ5qtSGdChZ7JaLOvR9GgSg+XTuhWqtO+JqZCTYbcv?= =?utf-8?q?WkM4gPpOMsX91BD6KuVycpJQz6DsrXRspWGgo3wDI9J+4w7bRKRoSOBz?= =?utf-8?q?FCxANo6Ufw/ROaHIRUVpzd6632J+9ShMLWmVgwF5aQs4jTFt9OJRW9Xz?= =?utf-8?q?P39wsGAPkfebUcIcgLbXPovgs+Gb4Tr3axYTUSR7zfwDUu4LNq0Qiyyl?= =?utf-8?q?b1/Fy7Pvc3/Sw8t3jN2N8UHbhwmXj0O/9xArU9EmYOrMGd30iHiEDjhM?= =?utf-8?q?4YlhP5g/0zF1nVq65QDgQcieOm61RY8ExMixUR?= X-Microsoft-Exchange-Diagnostics: 1; DB6PR0801MB1336; 6:lnWVluqSc9stnGscNlQnZgQJJWmE4TFUp2UtN27KDKJsJOK44VUk8oRRnc6nkYuzcq0NbtgMCj/y03SroXNr70OvHaB9jJ/5/CywjUZZ2PzEZdk2yuH/ya/cO3UuWnUreJaKs1oOUm9p46JvPvMz/YMPlTFdt1rfbCtX8DiSscBOPUm8fUd0U4Lpi18/6cv+JV2H/50hFQa/KLD1rCwZFL154P4tKTPc5tjYVG17az2e7wU6qZaLXanWNhfrkvxvOm3fNK9Y3HKwyaMpHyL9l5bsOGwIPhnuzHmLwjO0Yj3Q3Gdt8VltYBi6x3X6oj/ZHeJtNi1pXVyJGj5T89vrvs6vcCNSkvyBy/Ky3y1QLB0=; 5:3z9ZRzg6QFTY8GqoSErwiW0SnFooeEq1ccjLZC6d7yaKCgAxOoK5bWsc84+/1E1VJ14Aico3dDmuIfyvCWzGwUTIbTudIEX33p6hwkBenkoO4eb3aaRqHot+FT/5avxdirVezpUXWy7vrsTSuFQrrIA6cecc9EyDKN6oQN0bmYE=; 24:dI44LtakIYPHbFLA+awYzL/70KiY2WRqKqXzIFceyKfy/SO6dqcOtldjI1v1t1ISPtVmOVusotEgCEyMQ6IHT01p8iiJjOtAWzH1ReNQjkU=; 7:/BhHDT1DHxDuCZcUoLhUx+CRv8dbzrPu+I7/s88EZkX5WHhRJJLkxhYkexSwqqV7M7Y98nP2BqNNDIo4HBoKnhK2+BFrx9rRXn6XALsLRY8prXxcmoojJJqEcnE5Yaur2OQiJ0Vym07Au5ze3vH5rJ9hEgr5jLfwejkIm2Am4SmEgsDgSt5UAC4CY9/Od0zvs+/UtCPZfW3Jlo1oT3H205mITSmOVmAD6AD2XcV8stHLsbCa8QvWDX0IyeJiblgd SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DB6PR0801MB1336; 20:IN9oOFqbYzXwcum0zodtrZgp7lyvLh4c27Xsmrql7oq+jn7/vQpXLvXG+A+HMbE5ymWE/jEn1S/pEB+qRzdCQnFDXeDP8gjZ67qCQnRFAcVp2vpSW9GwdtTb8NQyM6TcK0f1DpR8TWCl1p0+AgxQ8h+2qknp/1hJbemWNc0hLVk= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Dec 2017 13:14:13.5769 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 5aaf94a0-9166-4b29-19a1-08d54874bb73 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 0bc7f26d-0264-416e-a6fc-8352af79c58f X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0801MB1336 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: <netdev.vger.kernel.org> X-Mailing-List: netdev@vger.kernel.org |
Series |
[1/3] net: Fix possible race in peernet2id_alloc()
|
expand
|
diff --git a/include/net/net_namespace.h b/include/net/net_namespace.h index 10f99dafd5ac..ff0e47471d5b 100644 --- a/include/net/net_namespace.h +++ b/include/net/net_namespace.h @@ -195,7 +195,7 @@ void __put_net(struct net *net); static inline struct net *get_net(struct net *net) { - atomic_inc(&net->count); + BUG_ON(atomic_inc_return(&net->count) <= 1); return net; }
Since people may mistakenly obtain destroying net from net_namespace_list and from net::netns_ids without checking for its net::counter, let's protect against such situations and insert BUG_ON() to stop move on after this. Panic is better, than memory corruption and undefined behavior. Signed-off-by: Kirill Tkhai <ktkhai@virtuozzo.com> --- include/net/net_namespace.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)