Re: [PATCH v2] platform/x86: dell-wmi-sysman: bound enumeration string aggregation

[Ilpo Järvinen]

On Wed, 08 Apr 2026 08:38:21 +0800, Pengpeng Hou wrote:

> populate_enum_data() aggregates firmware-provided value-modifier
> and possible-value strings into fixed 512-byte struct members.
> The current code bounds each individual source string but then
> appends every string and separator with raw strcat() and no
> remaining-space check.
> 
> Switch the aggregation loops to a bounded append helper and
> reject enumeration packages whose combined strings do not fit
> in the destination buffers.
> 
> [...]


Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/1] platform/x86: dell-wmi-sysman: bound enumeration string aggregation
      commit: 3c34471c26abc52a37f5ad90949e2e4b8027eb14

--
 i.