Re: [PATCH] image: Fix potentially uninitialized data variable

2023-03-31 Thread Tom Rini
On Mon, Feb 27, 2023 at 08:56:31PM +0100, Marek Vasut wrote: > In case fitImage support is disabled, and image_locate_script() is > passed a fitImage, then the 'data' variable is used uninitialized. > Drop into the default: branch of the switch-case statement and do > not return the uninitialized

Re: [PATCH] image: Fix potentially uninitialized data variable

2023-03-01 Thread Marek Vasut
On 3/1/23 17:02, Tom Rini wrote: On Mon, Feb 27, 2023 at 08:56:31PM +0100, Marek Vasut wrote: In case fitImage support is disabled, and image_locate_script() is passed a fitImage, then the 'data' variable is used uninitialized. Drop into the default: branch of the switch-case statement and do

Re: [PATCH] image: Fix potentially uninitialized data variable

2023-03-01 Thread Simon Glass
On Mon, 27 Feb 2023 at 12:56, Marek Vasut wrote: > > In case fitImage support is disabled, and image_locate_script() is > passed a fitImage, then the 'data' variable is used uninitialized. > Drop into the default: branch of the switch-case statement and do > not return the uninitialized data, and

Re: [PATCH] image: Fix potentially uninitialized data variable

2023-03-01 Thread Tom Rini
On Mon, Feb 27, 2023 at 08:56:31PM +0100, Marek Vasut wrote: > In case fitImage support is disabled, and image_locate_script() is > passed a fitImage, then the 'data' variable is used uninitialized. > Drop into the default: branch of the switch-case statement and do > not return the uninitialized

[PATCH] image: Fix potentially uninitialized data variable

2023-02-27 Thread Marek Vasut
In case fitImage support is disabled, and image_locate_script() is passed a fitImage, then the 'data' variable is used uninitialized. Drop into the default: branch of the switch-case statement and do not return the uninitialized data, and do not modify the return pointer either, just print an