Re: [PATCH 1/2] smb: client: fix off-by-8 bounds check in check_wsl_eas()
Greg Kroah-Hartman <[email protected]> Wed, 8 Apr 2026 08:15:53 +0200
| Newsgroups | gmane.linux.kernel,gmane.linux.kernel.cifs,gmane.network.samba.internals,gmane.linux.kernel.stable |
|---|---|
| Message-ID | <2026040833-espresso-triangle-425d@gregkh> |
On Wed, Apr 08, 2026 at 01:58:27PM +0800, ChenXiaoSong wrote:
> The for loop does not seem to catch cases where `outlen` is excessively
> large. In such cases, smb2_compound_op() would use this large `outlen` to
> `memcpy()`, which could lead to OOB.
>
> ```
> smb2_compound_op()
> {
> ...
> size[0] = outlen; // very large
> check_wsl_eas()
> memcpy(..., outlen) // out-of-bounds
> ...
> }
> ```
Ah, I missed the caller site. Yeah, probably a good thing to check as
well, want to make up a patch?
thanks,
greg k-h