newsyslog: fix compression arglist construction
ClosedPublic
Actions

Authored by dnelson_1901_yahoo.com on Jul 30 2018, 3:13 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 3, 7:12 AM

	Unknown Object (File)
	Thu, Oct 24, 11:05 PM

	Unknown Object (File)
	Thu, Oct 24, 11:05 PM

	Unknown Object (File)
	Thu, Oct 24, 11:05 PM

	Unknown Object (File)
	Thu, Oct 24, 10:43 PM

	Unknown Object (File)
	Wed, Oct 23, 11:12 PM

	Unknown Object (File)
	Oct 4 2024, 8:58 PM

	Unknown Object (File)
	Oct 2 2024, 9:16 AM

View All 43 Files

Subscribers

imp

Details

Reviewers

bapt

Commits

rS336913: newsyslog: fix compression arglist construction

Summary

Reuse of the index variable in two nested loops resulted in only the first
argument in the list being used (fine for gzip, not fine for zstd). Also
add tests for xz and zstd, and fix the COMPRESS_SUFFIX_MAXLEN macro.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dnelson_1901_yahoo.com created this revision.Jul 30 2018, 3:13 PM

Harbormaster completed remote builds in B18434: Diff 46024.Jul 30 2018, 3:13 PM

dnelson_1901_yahoo.com set the repository for this revision to rS FreeBSD src repository - subversion.Jul 30 2018, 3:25 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 30 2018, 3:25 PM

bapt accepted this revision.Jul 30 2018, 3:45 PM

This revision is now accepted and ready to land.Jul 30 2018, 3:45 PM

Closed by commit rS336913: newsyslog: fix compression arglist construction (authored by bapt). · Explain WhyJul 30 2018, 3:46 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

newsyslog/

newsyslog.c

10 lines

tests/

legacy_test.sh

24 lines

Diff 46027

View Options

head/usr.sbin/newsyslog/newsyslog.c