d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix the check for a "duplicate filename to compress", so that we're checking

Browse Source 
the *filename* and not the pid_file(!).   Stupid brain-fault on my part.
This could cause a segfault under -neworder if newsyslog had to rotate
multiple files, and later ones had specifed the 'N' flag.

Bug first reported by:	le
MFC after:	3 days
This commit is contained in:
Garance A Drosehn 2004-06-19 03:28:42 +00:00
parent 2a6bf1fadb
commit 5ff798ca5f
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=130707
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
usr.sbin/newsyslog/newsyslog.c
View File

@ -1890,7 +1890,7 @@ save_zipwork(const struct conf_entry *ent, const struct sigwork_entry *swork,
zprev = NULL;
ndiff = 1;
SLIST_FOREACH(ztmp, &zwhead, zw_nextp) {
ndiff = strcmp(ent->pid_file, ztmp->zw_fname);
ndiff = strcmp(zipfname, ztmp->zw_fname);
if (ndiff == 0)
break;
if (zsize > ztmp->zw_fsize)