On Thu Oct 15, 2026 at 10:27 AM CEST, Devansh Patel -X (devanshp - E INFOCHIPS 
PRIVATE LIMITED at Cisco) wrote:
> Hi Yoann,

Hello,

> I compared the submitted backport with upstream. curl 8.7.1 lacks
> first.h and uses the older test.h harness. Its res_easy_setopt() and
> res_global_init() macros assign to res, requiring the rename in
> init1647() and test().

Ok, now I see it. To give you an idea, this is the kind of info I want
in the backport notes. Can you send a v2 of the series with that added?
You will have to rebase on top of stable/scarthgap-review, I took some
patches of the series.

> run1647() retains upstream’s result. Following
> your earlier feedback [1], init1647() also retains the CURLcode type.
> With those adjustments, the test passes; reversing the library fix
> reproduces the Digest header leak.

This info can go into your commit message. Knowing what kind of test was
done is always good.

> [1] 
> https://www.mail-archive.com/openembedded-core%40lists.openembedded.org/msg226412.html
> Regards,
> Devansh

Thanks!
-- 
Yoann Congal
Smile ECS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#247899): 
https://lists.openembedded.org/g/openembedded-core/message/247899
Mute This Topic: https://lists.openembedded.org/mt/121625923/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to