From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Ahern Subject: Re: [PATCH iproute2] libnetlink: fix leak and using unused memory on error Date: Sun, 16 Sep 2018 20:54:49 -0700 Message-ID: <4689db12-8858-e219-2eb7-3c3f6a3803d7@gmail.com> References: <20180913193338.20233-1-stephen@networkplumber.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: Mahesh Bandewar , linux-netdev To: =?UTF-8?B?TWFoZXNoIEJhbmRld2FyICjgpK7gpLngpYfgpLYg4KSs4KSC4KSh4KWH4KS1?= =?UTF-8?B?4KS+4KSwKQ==?= , Stephen Hemminger Return-path: Received: from mail-pl1-f194.google.com ([209.85.214.194]:37307 "EHLO mail-pl1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726169AbeIQJUW (ORCPT ); Mon, 17 Sep 2018 05:20:22 -0400 Received: by mail-pl1-f194.google.com with SMTP id f1-v6so6753794plt.4 for ; Sun, 16 Sep 2018 20:54:53 -0700 (PDT) In-Reply-To: Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 9/14/18 10:48 AM, Mahesh Bandewar (महेश बंडेवार) wrote: > On Thu, Sep 13, 2018 at 12:33 PM, Stephen Hemminger > wrote: >> If an error happens in multi-segment message (tc only) >> then report the error and stop processing further responses. >> This also fixes refering to the buffer after free. >> >> The sequence check is not necessary here because the >> response message has already been validated to be in >> the window of the sequence number of the iov. >> >> Reported-by: Mahesh Bandewar >> Fixes: 7b2ee50c0cd5 ("hv_netvsc: common detach logic") $ git show 7b2ee50c0cd5 fatal: ambiguous argument '7b2ee50c0cd5': unknown revision or path not in the working tree. but we knew that since iproute2 does not have hv_netsvc code. Copy and paste error?