Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 24 Sep 2012 10:06:03 +0800
From:      Kevin Lo <kevlo@FreeBSD.org>
To:        Garrett Cooper <yanegomi@gmail.com>
Cc:        Konstantin Belousov <kostikbel@gmail.com>, svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: Re: svn commit: r240850 - head/lib/libstand
Message-ID:  <505FC00B.5020204@FreeBSD.org>
In-Reply-To: <CAGH67wQf5whjgQmXD3Ln_-r=WsLMYrBQUBj_G-HoGf1a5F_4cg@mail.gmail.com>
References:  <201209230838.q8N8c6Tu056083@svn.freebsd.org> <20120923105220.GL37286@deviant.kiev.zoral.com.ua> <CAGH67wQf5whjgQmXD3Ln_-r=WsLMYrBQUBj_G-HoGf1a5F_4cg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2012/09/24 03:27, Garrett Cooper wrote:
> On Sun, Sep 23, 2012 at 3:52 AM, Konstantin Belousov
> <kostikbel@gmail.com> wrote:
>> On Sun, Sep 23, 2012 at 08:38:06AM +0000, Kevin Lo wrote:
>>> Author: kevlo
>>> Date: Sun Sep 23 08:38:06 2012
>>> New Revision: 240850
>>> URL: http://svn.freebsd.org/changeset/base/240850
>>>
>>> Log:
>>>    Avoid NULL dereference
>>>
>>> Modified:
>>>    head/lib/libstand/nfs.c
> ...
>
>> I do not see how this change is useful. libstand' Free() function handles
>> NULL parameter fine, as well as all other free(3) implementations I am
>> aware of.
>      +1. free(3) should silently ignore NULL parameters passed into it.

Well, The patch is harmless. I suppose I could argue it is a safety belt
against free(NULL) should be silently ignored.
I have no problem changing it back, but there are two other places where
the same comment could apply.

Index: lib/libstand/nfs.c
===================================================================
--- lib/libstand/nfs.c  (revision 240879)
+++ lib/libstand/nfs.c  (working copy)
@@ -606,10 +606,8 @@ nfs_open(const char *upath, struct open_file *f)
         error = 0;

  out:
-       if (newfd)
-               free(newfd);
-       if (path)
-               free(path);
+       free(newfd);
+       free(path);
  #else
          currfd->iodesc = desc;

@@ -1256,10 +1254,8 @@ nfs_open(const char *upath, struct open_file *f)
         error = 0;

  out:
-       if (newfd)
-               free(newfd);
-       if (path)
-               free(path);
+       free(newfd);
+       free(path);
  #else
         currfd->iodesc = desc;


> Thanks,
> Garrett
>

     Kevin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?505FC00B.5020204>