FiberOVERLAPPED::OverlappedCompletionRoutine后出现uaf问题（Win系统）

[xincun]

echo_client_stackfull.cpp 测试这个client发现，echo_client 方法退出了GQCS还在跑然后出现这个问题。

原因是WSAxxx函数同步成功，然后GQCS还是会收到通知（https://learn.microsoft.com/en-us/windows/win32/api/winbase/nf-winbase-setfilecompletionnotificationmodes），但这个时候echo_client已经退栈了。

[microcai]

??? 什么同步成功？

[xincun]

是的

[xincun]

只要同步成功出现就uaf

[microcai]

就是说 WSASend 后，没有调用 get_overlapped_result , 而是直接判定成功。导致 事件回调的时候 OV 已经撤销了对吧？
那 WSASend 返回值是啥？ getlasterror 的返回值是啥？

[microcai]

@xincun 试试 https://github.com/microcai/iocp/tree/fix_issue_5 这个分支看看有没有 uaf

[xincun]

就是说 WSASend 后，没有调用 get_overlapped_result , 而是直接判定成功。导致 事件回调的时候 OV 已经撤销了对吧？✔
那 WSASend 返回值是啥？ 0
getlasterror 的返回值是啥？0

[xincun]

@xincun 试试 https://github.com/microcai/iocp/tree/fix_issue_5 这个分支看看有没有 uaf

这个分支不复现了，跑路多次同步返回有数据的情况没crash，这个是console的输出：

这个代码需要在connect之前bind，不bind的话lasterror是10022（WSAEINVAL）：https://learn.microsoft.com/en-us/windows/win32/api/mswsock/nc-mswsock-lpfn_connectex

SOCKADDR_IN local{};
local.sin_family = AF_INET;
local.sin_addr.s_addr = htonl(INADDR_ANY);
local.sin_port = 0;
if (bind(sock, (SOCKADDR*)&local, sizeof(local)) == SOCKET_ERROR) {
  printf("bind failed: %d\n", WSAGetLastError());
  return;
}

[microcai]

在 master 分支已经修正了。 @xincun 可以再试试看

[xincun]

1、fix fcontextbug 这个commit会导致client一起来就crash，revert 后正常

2、WSAConnectEx 前不bind会导致10022错误，参考前一个反馈。 SOCKADDR_IN local{}; local.sin_family = AF_INET; local.sin_addr.s_addr = htonl(INADDR_ANY); local.sin_port = 0; if (bind(sock, (SOCKADDR*)&local, sizeof(local)) == SOCKET_ERROR) { printf("bind failed: %d\n", WSAGetLastError()); return; }

[xincun]

这两个操作后，master分支下的echo_client_stackfull在跑就没复现问题了。

[microcai]

@xincun 发现了一个 使用 fcontext 的 Fiber 代码的 一个bug. 解决了。