[Tarantool-patches] [PATCH] serilaizer: check for recursive serialization

Cyrill Gorcunov gorcunov at gmail.com
Tue Jul 14 13:40:27 MSK 2020


On Tue, Jul 14, 2020 at 12:45:33PM +0300, Igor Munkin wrote:
> > 
> > Wait. The @idx stands for index in a table as far as I remember,
> > this just happen to hit when you're calling youself recursively
> > but @idx may be > SERIALIZER_CRITICAL_RECURSION_DEPTH for nonrecursive
> > calls as well.
> 
> I guess you've just misread this part: @idx is just a guest stack slot
> where Lua object being serialized is stored. If I get your concerns
> right, you're talking about the following case:

Yes, you're right. Thanks, Igor! I don't have strong preference here
on how to handle the case. Thus

Reviewed-by: Cyrill Gorcunov <gorcunov at gmail.com>


More information about the Tarantool-patches mailing list