Delivered-To: arisawa@ken.aichi-u.ac.jp
X-Authentication-Warning: claven.cse.psu.edu: majordom set sender to owner-9fans using -f
From: arisawa@ar.aichi-u.ac.jp
To: 9fans@cse.psu.edu
Date: Sat, 3 Jun 2000 17:23:42 +0900
Subject: [9fans] memory leak in cron.c
Sender: owner-9fans@cse.psu.edu
Reply-To: 9fans@cse.psu.edu

Hello 9fans,

auth/cron.c has memory leak.
the following function:

void
freejobs(Job *j)
{
	Job *fj;

	for(fj = j; fj; fj = j){
		j = j->next;
		free(fj);
	}
}

should be replaced by:

void
freejobs(Job *j)
{	Job *i;
	while(j){
		i = j;
		j = j->next;
		free(i->cmd);
		free(i);
	}
}

# sorry, I prefer beginner's programming style.

Kenji Arisawa
E-mail: arisawa@aichi-u.ac.jp