Describe the bug
Hi, I found a possible memory leak in streamGetEdgeID with static analysis.
si is initialized by streamIteratorStart(), the code doesn't streamIteratorStop() on it when things done. it's a memory leak.
https://github.com/redis/redis/blob/60250f50c2f308d301fb0bfcb6d07faa2560ce60/src/t_stream.c#L393-L405
According to the doc of streamIteratorStart(), we need to call streamIteratorStop() when things done.
https://github.com/redis/redis/blob/60250f50c2f308d301fb0bfcb6d07faa2560ce60/src/t_stream.c#L1037-L1058
Thanks for your further confirmation.
To reproduce
Steps to reproduce the behavior and/or a minimal code sample.
Expected behavior
A description of what you expected to happen.
Additional information
Any additional information that is relevant to the problem.
Comment From: sundb
@Yuuoniy Thx, you are right. Can you make a PR to fix it?
Comment From: Yuuoniy
@sundb Sure, I will send a PR later.