From mboxrd@z Thu Jan  1 00:00:00 1970
From: erik quanstrom <quanstro@quanstro.net>
Date: Sun,  2 Jun 2013 23:44:11 -0400
To: 9fans@9fans.net
Message-ID: <f9e98d63a812ffcfacc8863f84ccecb0@brasstown.quanstro.net>
In-Reply-To: <a0a1de4b257a1108b4897608b09c4b92@gmx.de>
References: <a0a1de4b257a1108b4897608b09c4b92@gmx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit
Subject: Re: [9fans] test(1) -older bug?
Topicbox-Message-UUID: 609a4156-ead8-11e9-9d60-3106f5b1d025

On Sun Jun  2 17:31:05 EDT 2013, cinap_lenrek@gmx.de wrote:
> its /n/sources/patch/testolder, also leaks dir in the case:
>
> 	if(rel)
> 		n = time(0) - n;
> 	if(n < 0)
> 		return 0; <----- HERE
> 	r = dir->mtime < n;
>
> 	free(dir);
> 	return r;

thanks.  i appreciate the catch.

[sources] applied patch: /n/atom/patch/applied/testolder1

email
	quanstro@quanstro.net
readme
	fix memory leak pointed out by cinap
removed

files
	/sys/src/cmd/test.c	test.c

/sys/src/cmd/test.c	test.c
test.c.orig:373,380 - test.c:373,381
  	if(rel)
  		n = time(0) - n;
  	if(n < 0)
- 		return 0;
- 	r = dir->mtime < n;
+ 		r =  0;
+ 	else
+ 		r = dir->mtime < n;

  	free(dir);
  	return r;
------
merge...backup...copy...
cpfile test.c /n/dist/sys/src/cmd/test.c
done