소스 검색

Xref change holding reference could crash

The gh-352 change, unintentionally introduced a reference to the endpoint
when should have been a copy, consequently those endpoints could become
invalid and sequently cause a core on access. This would tend to happen
when a client already had a few previous connections to a dafilesrv,
but where the original connection (CRemoteFile) had been destroyed along
with it's endpoint.

Signed-off-by: Jake Smith <jake.smith@lexisnexis.com>
Jake Smith 13 년 전
부모
커밋
c828095d94
1개의 변경된 파일1개의 추가작업 그리고 1개의 파일을 삭제
  1. 1 1
      common/remote/sockfile.cpp

+ 1 - 1
common/remote/sockfile.cpp

@@ -1479,7 +1479,7 @@ class CCritTable;
 class CEndpointCS : public CriticalSection, public CInterface
 {
     CCritTable &table;
-    const SocketEndpoint &ep;
+    const SocketEndpoint ep;
 public:
     CEndpointCS(CCritTable &_table, const SocketEndpoint &_ep) : table(_table), ep(_ep) { }
     const void *queryFindParam() const { return &ep; }