mbox series

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

Message ID adba1753-4d6b-0ee4-a7c0-42c5ef520b67@suse.com (mailing list archive)
Headers show
Series (mainly) xz imports from Linux | expand

Message

Jan Beulich Dec. 6, 2021, 1:29 p.m. UTC
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

Comments

Jan Beulich Dec. 6, 2021, 1:35 p.m. UTC | #1
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