mbox series

[0/2] lmb: rework logic to validate load address for network commands

Message ID 20240916152025.75789-1-sughosh.ganu@linaro.org
Headers show
Series lmb: rework logic to validate load address for network commands | expand

Message

Sughosh Ganu Sept. 16, 2024, 3:20 p.m. UTC 
Rework the logic to verify the load address so that address re-use is
not an issue.


Note: To be applied on next, on top of
https://patchwork.ozlabs.org/project/uboot/patch/20240913073251.2286529-2-prasad.kummari@amd.com/

Sughosh Ganu (2):
  tftp: rework the logic to validate the load address
  wget: rework the logic to validate the load address

 net/tftp.c | 19 +------------------
 net/wget.c | 36 +-----------------------------------
 2 files changed, 2 insertions(+), 53 deletions(-)

Comments

Tom Rini Sept. 21, 2024, 2:39 a.m. UTC | #1 
On Mon, 16 Sep 2024 20:50:23 +0530, Sughosh Ganu wrote:

> Rework the logic to verify the load address so that address re-use is
> not an issue.
> 
> 
> Note: To be applied on next, on top of
> https://patchwork.ozlabs.org/project/uboot/patch/20240913073251.2286529-2-prasad.kummari@amd.com/
> 
> [...]

Applied to u-boot/next, thanks!
Vaishnav Achath Oct. 1, 2024, 6:07 a.m. UTC | #2 
Hi Tom,

On 21/09/24 08:09, Tom Rini wrote:
> On Mon, 16 Sep 2024 20:50:23 +0530, Sughosh Ganu wrote:
> 
>> Rework the logic to verify the load address so that address re-use is
>> not an issue.
>>
>>
>> Note: To be applied on next, on top of
>> https://patchwork.ozlabs.org/project/uboot/patch/20240913073251.2286529-2-prasad.kummari@amd.com/
>>
>> [...]
> 
> Applied to u-boot/next, thanks!

Can these fixes be applied to master also as the failures are seen now 
in master as well?

Thanks and Regards,
Vaishnav

>
Sughosh Ganu Oct. 1, 2024, 8:49 a.m. UTC | #3 
On Tue, 1 Oct 2024 at 11:37, Vaishnav Achath <vaishnav.a@ti.com> wrote:
>
> Hi Tom,
>
> On 21/09/24 08:09, Tom Rini wrote:
> > On Mon, 16 Sep 2024 20:50:23 +0530, Sughosh Ganu wrote:
> >
> >> Rework the logic to verify the load address so that address re-use is
> >> not an issue.
> >>
> >>
> >> Note: To be applied on next, on top of
> >> https://patchwork.ozlabs.org/project/uboot/patch/20240913073251.2286529-2-prasad.kummari@amd.com/
> >>
> >> [...]
> >
> > Applied to u-boot/next, thanks!
>
> Can these fixes be applied to master also as the failures are seen now
> in master as well?

The lmb patch series has not made it to master yet. How are you seeing
these issues on the master branch when the lmb scope is local ?

-sughosh

>
> Thanks and Regards,
> Vaishnav
>
> >
Vaishnav Achath Oct. 3, 2024, 5:47 a.m. UTC | #4 
Hi Sughosh, Tom,

On 01/10/24 14:19, Sughosh Ganu wrote:
> On Tue, 1 Oct 2024 at 11:37, Vaishnav Achath <vaishnav.a@ti.com> wrote:
>>
>> Hi Tom,
>>
>> On 21/09/24 08:09, Tom Rini wrote:
>>> On Mon, 16 Sep 2024 20:50:23 +0530, Sughosh Ganu wrote:
>>>
>>>> Rework the logic to verify the load address so that address re-use is
>>>> not an issue.
>>>>
>>>>
>>>> Note: To be applied on next, on top of
>>>> https://patchwork.ozlabs.org/project/uboot/patch/20240913073251.2286529-2-prasad.kummari@amd.com/
>>>>
>>>> [...]
>>>
>>> Applied to u-boot/next, thanks!
>>
>> Can these fixes be applied to master also as the failures are seen now
>> in master as well?
> 
> The lmb patch series has not made it to master yet. How are you seeing
> these issues on the master branch when the lmb scope is local ?
> 
> -sughosh
> 

Sorry this was a confusion from my end, please ignore my previous 
comment, master does not have any issue related to this.

Thanks and Regards,
Vaishnav

>>
>> Thanks and Regards,
>> Vaishnav
>>
>>>