Re: cvs commit: src/bin/rm rm.c

["Jonas Trollvik" <jontro@xxxxxxxxx>]

What's the point of adding this?
I remember a discussion on the lists here a year ago or so where it's
stated that the OS will take care of that as soon as the process exits
and that it'll do so faster than the process itself.
It seems to me that the next operation after the fts_close() is exit(eval);

-Jonas



On 6/19/06, Simon Schubert <corecode@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> corecode    2006/06/19 05:08:25 PDT
>
> DragonFly src repository
>
> Modified files:
>    bin/rm               rm.c
> Log:
> Coverity CID 1670: Call fts_close() at the end of rm_tree().
>
> Taken from: NetBSD via Coverity Prevent
> Submitted by: Gary <gary@xxxxxxxxxxxxxxxxxxxx>
>
> Revision  Changes    Path
> 1.16      +1 -0      src/bin/rm/rm.c
>
>
> http://www.dragonflybsd.org/cvsweb/src/bin/rm/rm.c.diff?r1=1.15&r2=1.16&f=u