Re: [PATCH] staging: vt6655: fix potential memory conruption bug

From: Nam Cao
Date: Fri Sep 09 2022 - 02:01:45 EST


Hi Greg,

Just out of curiosity, how can we be sure that sizeof(unsigned int)
is never 8 bytes? The C standard doesn't say anything about this, as
far as I know.

Patch v2 will come later today or tomorrow.

Best regards,
Nam

On Fri, Sep 9, 2022 at 7:47 AM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
>
> sizeof(unsigned int) is not going to be 8 bytes, so there's no memory
> issue here. But your fix is good, so can you please just rewrite this
> to be "make it more obvious what is happening" type of patch?
>
> thanks,
>
> greg k-h