ZSTDIO: Correctly initialize zstd context with provided 'level'
ClosedPublic
Actions

Authored by cem on Sep 13 2018, 4:17 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Sep 23, 1:36 AM

	Unknown Object (File)
	Tue, Sep 22, 9:48 PM

	Unknown Object (File)
	Sat, Sep 12, 5:19 AM

	Unknown Object (File)
	Tue, Sep 8, 5:12 PM

	Unknown Object (File)
	Sun, Sep 6, 1:56 AM

	Unknown Object (File)
	Sat, Sep 5, 2:09 AM

	Unknown Object (File)
	Fri, Sep 4, 8:06 PM

	Unknown Object (File)
	Fri, Sep 4, 4:33 AM

View All Files

Subscribers

imp

Details

Reviewers

markj
vangyzen

Group Reviewers

releng

Commits

rS339497: ZSTDIO: Correctly initialize zstd context with provided 'level'

Summary

Prior to this revision, we allocated sufficient context space for 'level'
but never actually set the compress level parameter, so we would always get
the default '3'.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

cem created this revision.Sep 13 2018, 4:17 AM

Harbormaster completed remote builds in B19558: Diff 47978.Sep 13 2018, 4:17 AM

vangyzen accepted this revision.Sep 13 2018, 1:04 PM

This revision is now accepted and ready to land.Sep 13 2018, 1:04 PM

markj accepted this revision.Sep 13 2018, 3:37 PM

Request for acceptance into head during freeze

Closed by commit rS339497: ZSTDIO: Correctly initialize zstd context with provided 'level' (authored by cem). · Explain WhyOct 20 2018, 9:50 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptOct 20 2018, 9:50 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

kern/

subr_compressor.c

23 lines

Diff 49364

View Options

head/sys/kern/subr_compressor.c

ZSTDIO: Correctly initialize zstd context with provided 'level'ClosedPublicActions