Re: [External] Re: [PATCH v2 1/2] negotiator/default: avoid stack overflow

[Han Xin <hanxin.hx@xxxxxxxxxxxxx>]

On Wed, Apr 26, 2023 at 7:13 PM Derrick Stolee <derrickstolee@xxxxxxxxxx> wrote:
>
> On 4/26/2023 12:05 AM, Han Xin wrote:
> > mark_common() in negotiator/default.c may overflow the stack due to
> > recursive function calls. Avoid this by instead recursing using a
> > heap-allocated data structure.
> >
> > This is the same case as [1].
> >
> > 1. 4654134976f (negotiator/skipping: avoid stack overflow, 2022-10-25)
>
> We would typically write this inline, such as:
>
>   This is the same case as 4654134976f (negotiator/skipping: avoid
>   stack overflow, 2022-10-25)
>

Make sense.

Thanks
-Han Xin