Subject: CVS commit: src/usr.sbin/cron
To: None <source-changes@NetBSD.org>
From: Christos Zoulas <christos@netbsd.org>
List: source-changes
Date: 12/22/2004 00:49:14
Module Name:	src
Committed By:	christos
Date:		Wed Dec 22 00:49:14 UTC 2004

Modified Files:
	src/usr.sbin/cron: crontab.c

Log Message:
PR/28744: Ed Ravin: crontab with fewer than 3 comment lines dumps core
in crontab -l.
The problem is that the skip_header code has a side effect of trying to
write to the new temporary cron tab if we run out of comments. Since we
don't have an output file in -l, we core-dump trying to fputc to NULL.
Simplify the logic so that we don't write in the skip_header function,
and we always return the first character after the comment lines to the
caller, skipping the '\n'.


To generate a diff of this commit:
cvs rdiff -r1.23 -r1.24 src/usr.sbin/cron/crontab.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.