aboutsummaryrefslogtreecommitdiffstats
path: root/prog/mutation.go
diff options
context:
space:
mode:
authorDmitry Vyukov <dvyukov@google.com>2022-12-17 12:12:05 +0100
committerDmitry Vyukov <dvyukov@google.com>2022-12-22 10:11:08 +0100
commita0df376348d2ad1d3e557ea221e75c78a5d9fd96 (patch)
treeb15ead346eb5f9f01e71dca1f4f4d1966da3e0cf /prog/mutation.go
parent09ff16760eac2d6f03e93bd7d50892a6d536ed1b (diff)
pkg/image: make Decompress easier to use
Change DecompressWriter to DecompressCheck: checking validity of the image is the only useful use of DecompressWriter. Change Decompress to MustDecompress which does not return an error. We check validity during program deserialization, so all other uses already panic on errors. Also add dtor return value in preparation for subsequent changes.
Diffstat (limited to 'prog/mutation.go')
-rw-r--r--prog/mutation.go6
1 files changed, 2 insertions, 4 deletions
diff --git a/prog/mutation.go b/prog/mutation.go
index 100c47a91..5a38cfa88 100644
--- a/prog/mutation.go
+++ b/prog/mutation.go
@@ -396,10 +396,8 @@ func (r *randGen) mutateImage(compressed []byte) (data []byte, retry bool) {
// Reconsider when/if we move mutation to the host process.
imageMu.Lock()
defer imageMu.Unlock()
- data, err := image.Decompress(compressed)
- if err != nil {
- panic(fmt.Sprintf("could not decompress data: %v", err))
- }
+ data, dtor := image.MustDecompress(compressed)
+ defer dtor()
if len(data) == 0 {
return compressed, true // Do not mutate empty data.
}