On Tue, Mar 04, 2025 at 01:37:21PM +0100, Quentin Schulz wrote: > Hi Tom, > > On 3/4/25 5:41 AM, Tom Rini wrote: > > On Fri, 07 Feb 2025 00:47:59 +0300, Anton Moryakov wrote: > > > > > - Ensure `free_ctx` is called in both error and success paths. > > > - Fix memory leak in `ctx.signature` when `do_add` fails." > > > > > > Triggers found by static analyzer Svace. > > > > > > > > > > Applied to u-boot/next, thanks! > > > > I believe this commit is not doing anything at all, especially not what it > says it is doing. > > I believe this is a false positive from the static analyzer?
Yeah, you're right. What we had there before was written more compactly than required, but this ends up being a no-op to the compiler and just easier to read code. -- Tom
signature.asc
Description: PGP signature