[PATCH v2 0/7] (mainly) xz imports from Linux

Jan Beulich posted 7 patches 4 years, 2 months ago
Test gitlab-ci passed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/adba1753-4d6b-0ee4-a7c0-42c5ef520b67@suse.com
There is a newer version of this series
[PATCH v2 0/7] (mainly) xz imports from Linux
Posted by Jan Beulich 4 years, 2 months ago
While going through their 5.15.3 log I did notice two changes, which made
me go check what else we might be missing. The series here is the result.
Linux has also updated zstd, but that includes a pretty large change which
I'm not ready to deal with right now. Them moving closer to the upstream
zstd sources is certainly a good thing, so I suppose sooner or later we
will want to follow them in doing so.

The only change in v2 is the re-adding of some previously dropped S-o-b
representing the original, documentable submission flow.

1: xz: add fall-through comments to a switch statement
2: xz: fix XZ_DYNALLOC to avoid useless memory reallocations
3: decompressors: fix spelling mistakes
4: xz: avoid overlapping memcpy() with invalid input with in-place decompression
5: xz: fix spelling in comments
6: xz: move s->lzma.len = 0 initialization to lzma_reset()
7: xz: validate the value before assigning it to an enum variable

Jan


Re: [PATCH v2 0/7] (mainly) xz imports from Linux
Posted by Jan Beulich 4 years, 2 months ago
On 06.12.2021 14:29, Jan Beulich wrote:
> While going through their 5.15.3 log I did notice two changes, which made
> me go check what else we might be missing. The series here is the result.
> Linux has also updated zstd, but that includes a pretty large change which
> I'm not ready to deal with right now. Them moving closer to the upstream
> zstd sources is certainly a good thing, so I suppose sooner or later we
> will want to follow them in doing so.
> 
> The only change in v2 is the re-adding of some previously dropped S-o-b
> representing the original, documentable submission flow.

Oh, actually there was another (benign) change:

> 1: xz: add fall-through comments to a switch statement
> 2: xz: fix XZ_DYNALLOC to avoid useless memory reallocations
> 3: decompressors: fix spelling mistakes
> 4: xz: avoid overlapping memcpy() with invalid input with in-place decompression
> 5: xz: fix spelling in comments
> 6: xz: move s->lzma.len = 0 initialization to lzma_reset()
> 7: xz: validate the value before assigning it to an enum variable

The last three have been switched around to match the sequence they went
into upstream Linux (IOW the order spelled out above does not match the
actual series; I'm sorry).

Jan