From owner-svn-src-head@FreeBSD.ORG Sun Sep 16 16:06:12 2012 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 787D11065673 for ; Sun, 16 Sep 2012 16:06:12 +0000 (UTC) (envelope-from lists@eitanadler.com) Received: from mail-pz0-f54.google.com (mail-pz0-f54.google.com [209.85.210.54]) by mx1.freebsd.org (Postfix) with ESMTP id 414998FC12 for ; Sun, 16 Sep 2012 16:06:12 +0000 (UTC) Received: by dadr6 with SMTP id r6so3864157dad.13 for ; Sun, 16 Sep 2012 09:06:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=eitanadler.com; s=0xdeadbeef; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type; bh=QAI+qv46z2LVY2EEXRQ8ZYLM4whKsP1pnMHkHh0T+GM=; b=C7hBqH7d3ivETlQYmVV4Vi1RViu5n5aKXHULgEnmqAASKlaKCQjvPZ+AZJFkdVUc6t pFHc4UXOvr3E0liae4MohHxURqn+o1M4egpapfPBeESaOk9v4pnmSl5XDpQ0AHjf4B5D oBjK+O6yHHUSVBmHxAadtqbKxfWTu/oKzJbuk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :x-gm-message-state; bh=QAI+qv46z2LVY2EEXRQ8ZYLM4whKsP1pnMHkHh0T+GM=; b=Q1GWodGXiWiohSKHWboWQdpBsOzAB+a11HLH8sg5+RPmni3hTba4karBMbPnaDwRxD CDozDSk0rSEkZhKN/NNnvxgexl/lCQwgziZor9iXncn3Vt3voblMO5cdDozrqOTCCbNR 9kUjz2xVy6QKP4lF2Lx3hkvFZ3n2kZVUaakzwr2yAUYsFicEHVl9OWPVwLP/F8ZvvfA/ gXY9gVW61onglKLuIiT21YU3DC8SmviAxZLwfNVGymOOXpQPYNl3GLOXZgBXJHLiQpLo e/ZE/Cby2CA/SntLIsaclr7r7Q4F6dQlTzMRkA7gxYdFgIWnf90m9brpQ/IEHy8+9cAT LFNQ== Received: by 10.68.210.207 with SMTP id mw15mr123048pbc.81.1347811571801; Sun, 16 Sep 2012 09:06:11 -0700 (PDT) MIME-Version: 1.0 Sender: lists@eitanadler.com Received: by 10.66.87.41 with HTTP; Sun, 16 Sep 2012 09:05:41 -0700 (PDT) In-Reply-To: <5055E121.2030208@coosemans.org> References: <201209142347.q8ENlN7N034951@svn.freebsd.org> <5054EBCB.6070105@coosemans.org> <5054F116.8090503@cran.org.uk> <5055E121.2030208@coosemans.org> From: Eitan Adler Date: Sun, 16 Sep 2012 12:05:41 -0400 X-Google-Sender-Auth: 1uxuEuBFkE9yU7Rwb09DhC4HenE Message-ID: To: Tijl Coosemans Content-Type: text/plain; charset=UTF-8 X-Gm-Message-State: ALoCoQnhEIY5wzbXdgXR/PvB7UlRgIkfPiwZSe9A48p0/3YhPRI4f5MvucAeiXgK1XYdeaAxW+BX Cc: Bruce Cran , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r240527 - head/bin/df X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 16 Sep 2012 16:06:12 -0000 On 16 September 2012 10:24, Tijl Coosemans wrote: > On 16-09-2012 01:27, Eitan Adler wrote: >> On 15 September 2012 17:20, Bruce Cran wrote: >>> On 15/09/2012 21:57, Tijl Coosemans wrote: >>>> >>>> Freeing memory right before exiting is a waste of time. The tool shouldn't >>>> complain about it. >> >> Perhaps, but tools do. This has already been brought up on cfe-dev. >> >>> "/Stop wasting time and just exit already/." - >>> http://blogs.msdn.com/b/oldnewthing/archive/2012/01/05/10253268.aspx >> >> I read this blog post when it came out. :) > > In this case the free is actually wrong, because the pointer can point > to memory allocated by getmntinfo(3) and that manpage says an application > cannot free it. Ah, I missed that. I wish this point was brought up earlier. I'll revert the commit -- Eitan Adler Source & Ports committer X11, Bugbusting teams