Re: [PATCH v2 1/2] usb: gadget: configfs: NUL-terminate extended property data

[sashiko-bot@kernel.org]

> Unicode extended properties include a UTF-16 terminator in their wire
> length.  The configfs store path keeps the raw UTF-8 bytes, but the
> configfs readback and descriptor conversion consume the source byte
> implied by that terminator-inclusive length.
> 
> The allocation currently contains only the input bytes, so both paths
> can read one byte past it.  Allocate the trailing NUL with kmemdup_nul()
> and centralize the raw-to-wire length conversion used by the store path.
> Avoid dereferencing page[-1] on an empty configfs write.
> 
> Fixes: 7419485f197c ("usb: gadget: configfs: OS Extended Properties descriptors support")
> Reported-by: Bingquan Chen <patzilla007@gmail.com>
> Link: https://lore.kernel.org/r/20260421141010.5607-1-patzilla007@gmail.com
> Suggested-by: Neill Kapron <nkapron@google.com>
> Link: https://lore.kernel.org/r/179338899796.2590875.15337207674245295434.b4-review@b4
> Cc: stable@vger.kernel.org
> Signed-off-by: Yingjie Wang <x-professor@outlook.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/cover.1793470427.git.x-professor@outlook.com?part=1