Re: [PATCH v2] mptcp: pm: Return local variable instead of freed pointer

[Matthieu Baerts <matttbe@kernel.org>]

On 25/03/2025 19:45, Thorsten Blum wrote:
> On 25. Mar 2025, at 19:28, Matthieu Baerts wrote:
>>
>> It looks like the CI had some issues to apply your patch [1]. Strange,
>> it was fine for me to apply it with 'b4 shazam'. I manually sent it to
>> the CI, I will wait for it to validate it before applying it.
>>
>> https://patchew.org/MPTCP/20250325163251.135510-2-thorsten.blum@linux.dev/logs/git/
> 
> I get a 404 Not Found when opening the link.

Better with this one, then click on "apply log"?

https://patchew.org/MPTCP/20250325163251.135510-2-thorsten.blum@linux.dev/

Here is the log:

> Switched to a new branch '20250325163251.135510-2-thorsten.blum@linux.dev'
> Applying: mptcp: pm: Return local variable instead of freed pointer
> error: corrupt patch at line 27
> error: could not build fake ancestor
> hint: Use 'git am --show-current-patch=diff' to see the failed patch
> Patch failed at 0001 mptcp: pm: Return local variable instead of freed pointer
> When you have resolved this problem, run "git am --continue".
> If you prefer to skip this patch, run "git am --skip" instead.
> To restore the original branch and stop patching, run "git am --abort".

I didn't have that on my side.

> Weird, I made sure the patch applies to mptcp/export.

Yes, strange. There was an issue with the v1 as well apparently:

https://patchew.org/MPTCP/20250325110639.49399-2-thorsten.blum@linux.dev/

I will check later if it was not an issue with Patchew.

Cheers,
Matt
-- 
Sponsored by the NGI0 Core fund.