Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 25 Jun 2021 12:03:11 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Gerd Rausch <gerd.rausch@oracle.com>
Cc:     linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org,
        Doug Ledford <dledford@redhat.com>
Subject: Re: [PATCH 1/1] RDMA/cma: Fix rdma_resolve_route memory leak
Message-ID: <20210625150311.GA3006866@nvidia.com>
References: <f6662b7b-bdb7-2706-1e12-47c61d3474b6@oracle.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <f6662b7b-bdb7-2706-1e12-47c61d3474b6@oracle.com>
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from mlx.ziepe.ca (47.55.113.94) by BL0PR0102CA0065.prod.exchangelabs.com (2603:10b6:208:25::42) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4264.20 via Frontend Transport; Fri, 25 Jun 2021 15:03:12 +0000
Received: from jgg by mlx with local (Exim 4.94)        (envelope-from <jgg@nvidia.com>)        id 1lwnMN-00CcFM-LP; Fri, 25 Jun 2021 12:03:11 -0300
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 0a140246-023b-4cc9-cd2f-08d937ea5a39
X-MS-TrafficTypeDiagnostic: BL1PR12MB5304:
X-Microsoft-Antispam-PRVS: <BL1PR12MB5304423232F8B6932A7DD5E5C2069@BL1PR12MB5304.namprd12.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:6790;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: uZwKL6XTfqfAr4zYO4/UtY39OPKj8u00mkG4xnno479eJ7EvVkx/XfGX4GgmYw0jCJB7D7MuejG90krCAvO2dTfBTjXwpaasgxl5qHXBliEyJJRUc2WaV42atHa9CIoA2MCll6Bqt/xT34yjKPsXRQLyl2dcfe+f0Hor8JA7jSZrO8/bDd6PsJptYtQIBYdSO2SR7B/4JjELKyGvQpqzi8pEgzcTLNtsbHe8Lf4wQsRXySZtz6UxINo9r9NTmj4WAn1r78TZSXo4clZ17vLgiNMXIRqjdSt+sxtnto9cU6XlC2+siDagNsaUjz6he8/CrFvDp/jjvxHU4zjCunEkyTgEk5AZkl+2IBGWBc76EGZh1QBAGy7Q61579d2WAw8Benx+jvbs5zCG1KPYSuvzxapuM4JsPyanV8pBBRBu2Bav4dCPurfR8APHRfJ7iVaWEJWAqkGesI/GL4gPmS3HWuOj/ysPxvbZ3IBJhVdjgdvs8zKe8G1SAY06jR+DGgQALu+owwza8P2+heOX0dHHeLFjRz/GS0HMUubypvEcJH2+E5rIOP8dtzCi60tmQWt9JrSM/hrb+qLk1CqklQZ9KU7jytsxEQHNIpU6YnFeAF8=
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR12MB5506.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(396003)(39860400002)(376002)(346002)(136003)(4744005)(9746002)(6916009)(478600001)(38100700002)(1076003)(86362001)(66556008)(36756003)(8936002)(8676002)(186003)(5660300002)(66946007)(9786002)(66476007)(26005)(83380400001)(316002)(426003)(2906002)(2616005)(4326008)(33656002);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sxlIg6Rd9yI9c0RDOjPggIclbeBWy2mcL8epzoCSJwBPQT0jE46eZK85DC9l?=
 =?us-ascii?Q?FI5UrklWtQyAqcumJnunJzKH/Poj8F1afU389JJPZjSh4i/7s/PjXELnLAhm?=
 =?us-ascii?Q?rzmaEkz1+QcKpLs2zWPJL2oTX4U6EZHIiU5/GuB4GQAOwwI/xf2/+1QBKWK9?=
 =?us-ascii?Q?z6uB/ZzHMfP/GntB4Ef3ywU0I8DwKWPFbvgh+QHNrAw9GUhr/TKex0TrG/uC?=
 =?us-ascii?Q?CoKIvWGMw4EDZ6h0GIa0FSjhdzCyGDB0NL9wY+sEUI/ZKvbNTz7U1mG7P/zy?=
 =?us-ascii?Q?1YwbZSD+XdPJ2cS3SI1l+KoBREc9pS5mAgYx+P0iRPxjSvXHuiqNmZdqMQc6?=
 =?us-ascii?Q?8VtDythCG305WYKsf4a4EgFIiMV73io4yMz/fpte6CvjR9YmtP8fnb4a1Mu1?=
 =?us-ascii?Q?yIcav9aVyvI6r/rY+DiHHk/O2/Otx0CIfeZpcZlkaHU8bnvEwUtJ0mEmPQfd?=
 =?us-ascii?Q?zRk3QybwtRf5bbBrntRH+BBYbiRIsw1XREnhfNp+lcmZmGCM4KWLVmTxlLeR?=
 =?us-ascii?Q?4X8KcoFjn0wQb3nwLzXV4p59vxVL2V0FU8nCTG7KKNCqnFdOzBKmlJQoE9Ir?=
 =?us-ascii?Q?HFL/vYtfGgnW/nM9dKXdYFc/1sszx3hf1LsBpBKCobuyX+gIccda2LDOSPu4?=
 =?us-ascii?Q?ca8iS905T3M3E3UCz6BR6Dn3b3UAIn9T6rCYl2TjoqwRAF9lRvp7+bu0NeAW?=
 =?us-ascii?Q?7VayMNwNTRu267DFrEkjnhE6uSo+OuQUlJQ1ljDAR+yBg4XZyTyw95aVJtio?=
 =?us-ascii?Q?VjMykgch4NDGaFSvOPd04BiXm1mH2RQeOc+ufgx/rLlLYAwgWJxpxzzxzH57?=
 =?us-ascii?Q?Rx7XJnmMEnNemXycKh6eOxgtnI+kNWKuhlMBhJ83lvm5KZKfH6T2oBBArd4o?=
 =?us-ascii?Q?sOl7Wu+XxXScnZqDnk3JNCa+sfbyawgrggZp7cPd8jKlhhT+5UvnRPZcAe33?=
 =?us-ascii?Q?Rr1hYilkq1JzRQ0DXww03LfAYE95yRoW1PDVTnxT888pUVK+QEeZ04mEQhbS?=
 =?us-ascii?Q?rSdfYp1694GY9GQc2cnMWVqBvX9qmETzv54akKyC+9gGuanuEBCfM88lV6FV?=
 =?us-ascii?Q?ZCUwHsERAQEER8MjthZnbjGojMwJXxO7qzitPMA2ri0u3Rwko8ar5ADyBEwv?=
 =?us-ascii?Q?0ZwLrVwbh1HRMlqJXcHcwbyi3t81/2u4E2+a+b5HwPL9Q3sd/UMxd4tSMvyB?=
 =?us-ascii?Q?LcNPA6/+J8nRjOdN1GcSw2xleewxcaSjUSGXsrPIc+SCyV+ChPHRzpCuyyzS?=
 =?us-ascii?Q?soIp1hEVehMvP6NBDhOAwwV3rXJkA0IHY+qT/No/H4LNZ8//ptgRx1+No7VI?=
 =?us-ascii?Q?NBNYPBoqFDvypy3s+lRM0e6O?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0a140246-023b-4cc9-cd2f-08d937ea5a39
X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB5506.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Jun 2021 15:03:13.1869
 (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: MhG6m7erQXM+Na26SN6BmfvcK7TNFN+gSoYYYqm7TPcfiSzJfdhDKrYqs6KpYEYa
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5304
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Jun 24, 2021 at 11:55:31AM -0700, Gerd Rausch wrote:
> Fix a memory leak when "rmda_resolve_route" is called
> more than once on the same "rdma_cm_id".
> 
> Signed-off-by: Gerd Rausch <gerd.rausch@oracle.com>
> ---
>  drivers/infiniband/core/cma.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

I wondered for a while if it would be better to clear this in
cma_query_handler(), but it seems this is OK

Applied to for-next

Thanks,
Jason