YQ-3786 heap use after free / to stable (#10796)

kardymonds · Alek5andr-Kotov · web-flow · commit 40620ffbba9d · 2024-10-24T10:01:39.000+03:00
Co-authored-by: Alek5andr-Kotov &lt;akotov@ydb.tech&gt;
diff --git a/ydb/tests/fq/pq_async_io/ut/dq_pq_rd_read_actor_ut.cpp b/ydb/tests/fq/pq_async_io/ut/dq_pq_rd_read_actor_ut.cpp
@@ -68,7 +68,7 @@ struct TFixture : public TPqIoTestFixture {
     auto ExpectCoordinatorRequest(NActors::TActorId coordinatorId) {
         auto eventHolder = CaSetup->Runtime->GrabEdgeEvent<NFq::TEvRowDispatcher::TEvCoordinatorRequest>(coordinatorId, TDuration::Seconds(5));
         UNIT_ASSERT(eventHolder.Get() != nullptr);
-        return eventHolder.Get();
+        return eventHolder;
     }
 
     void ExpectStartSession(ui64 expectedOffset, NActors::TActorId rowDispatcherId) {

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ struct TFixture : public TPqIoTestFixture {`
`68`	`68`	`auto ExpectCoordinatorRequest(NActors::TActorId coordinatorId) {`
`69`	`69`	`auto eventHolder = CaSetup->Runtime->GrabEdgeEvent<NFq::TEvRowDispatcher::TEvCoordinatorRequest>(coordinatorId, TDuration::Seconds(5));`
`70`	`70`	`UNIT_ASSERT(eventHolder.Get() != nullptr);`
`71`		`- return eventHolder.Get();`
	`71`	`+ return eventHolder;`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`void ExpectStartSession(ui64 expectedOffset, NActors::TActorId rowDispatcherId) {`