Possible fix to #17.

coderforlife · coderforlife · commit 8ba2fb09499c · 2015-09-15T23:31:44.000-07:00
Because the inflate methods return MSCOMP_STREAM_END or MSCOMP_POSSIBLE_STREAM_END instead of MSCOMP_OK when they are at a stopping point, MSCOMP_OK actually becomes an error message (that gets translated to MSCOMP_BUF_ERROR). A similar method was already in use for ALL_AT_ONCE_WRAPPER_COMPRESS.
diff --git a/include/mscomp/internal.h b/include/mscomp/internal.h
@@ -611,7 +611,7 @@ typedef const_byte* RESTRICT const_rest_bytes;
 		strm.out = out; \
 		strm.out_avail = *_out_len; \
 		status = name##_inflate(&strm); \
-		if (UNLIKELY(status < 0)) { name##_inflate_end(&strm); } \
+		if (UNLIKELY(status <= MSCOMP_OK)) { name##_inflate_end(&strm); if (status == MSCOMP_OK) { status = MSCOMP_BUF_ERROR; } } \
 		else if (LIKELY((status = name##_inflate_end(&strm)) == MSCOMP_OK)) { *_out_len = strm.out_total; } \
 		return status; \
 	}

Original file line number	Diff line number	Diff line change
`@@ -611,7 +611,7 @@ typedef const_byte* RESTRICT const_rest_bytes;`
`611`	`611`	`strm.out = out; \`
`612`	`612`	`strm.out_avail = *_out_len; \`
`613`	`613`	`status = name##_inflate(&strm); \`
`614`		`- if (UNLIKELY(status < 0)) { name##_inflate_end(&strm); } \`
	`614`	`+ if (UNLIKELY(status <= MSCOMP_OK)) { name##_inflate_end(&strm); if (status == MSCOMP_OK) { status = MSCOMP_BUF_ERROR; } } \`
`615`	`615`	`else if (LIKELY((status = name##_inflate_end(&strm)) == MSCOMP_OK)) { *_out_len = strm.out_total; } \`
`616`	`616`	`return status; \`
`617`	`617`	`}`