Microsoft MVP성태의 닷넷 이야기
메모리 해제 예외 처리 관련.. [링크 복사], [링크+제목 복사],
조회: 15022
글쓴 사람
김길
홈페이지
첨부 파일
 

타인의 소스를 유지보수 해야 하는데 메모리 할당과 해제 부분의 예외 처리가 쉽지 않아 문의 드립니다.

try
{
    IntPtr hglobal = Marshal.AllocHGlobal(1);
    .............
    unsafe
    {
    fixed (~~~~~~)
    {
              .............
              hglobal = Marshal.ReAllocHGlobal(hglobal, (IntPtr)(RealSize + ReadBufSize));
              .............
    }//fixed
    }//unsafe
    .............
    Marshal.FreeHGlobal(hglobal);
    .............
}catch(~)
{
}

크게 핵심 로직을 보면 기존 개발자도 위처럼 FreeHGlobal을 호출할때
해당 포인터에 메모리가 할당되어있는지 판단하기가 어려워서 그랬는지 이게 정석인지 모르겠지만

일단 1바이트를 강제로 할당한 후에 필요한 부분에서 ReAllocHGlobal로 메모리를 재 할당하는 형태로 구현되어 있으며,
마지막에 Marshal.FreeHGlobal(hglobal);을 호출하는 형태입니다.
..... 으로 되어 있는 부분에서 상황에 따라서는 FreeHGlobal을 이용해서 메모리를 해제하거나 할당하는 로직이 있습니다.

현재 심각한 문제는 관련 메소드가 2000라인이 넘어 가는데다 메모리 누수도 있고
메모리 처리와 관련된 로직 처리가 깔끔하지 않습니다.


혹시나 해서 FreeHGlobal이 연속으로 호출되는 경우 어떤 식으로 동작하나 테스트 해보니
Marshal.FreeHGlobal(hglobal); 을 연속으로 2번 호출하게되면 어떤 경우에는 프로그램이 죽어 버리더군요.
그렇다보니 중간 중간에 예외가 발생할 경우 이미 해제된 상태에서 또 다시 해제를 시도하다 프로그램이 죽는 경우도 있습니다.

고객의 요청에 의해서 메모리 처리에 좀 더 안전을 기하기 위해
try~catch~finally 형태로 구현해서 finally부분에서 반드시 할당된 메모리를 해제하도록 하려고 하는데
로직상 예외가 아니더라도 중간 중간에 FreeHGlobal을 호출하는 경우도 있어서
finally 부분에서 hglobal에 메모리가 할당된 상태인지 아닌지를 파악하기가 어렵습니다.^^;;;



메모리가 할당된 상태인지 확인할 수 있는 방법을 몰라서
현재는 무식하지만 try문 바깥에 bool isAlloc = true; 변수 하나를 선언해서
메모리가 할당되면 true로 설정하고, 메모리를 해제했으면 false로 그때 그때 매번 값을 변경 한 뒤
finally문에서는 if (isAlloc)인 경우 Marshal.FreeHGlobal(hglobal);을 실행하고 있습니다.

이런 식으로 처리하는것 말고 좀 더 효율적인 방법이 없을런지요? ^^;;








[최초 등록일: ]
[최종 수정일: 6/6/2011]


비밀번호

댓글 작성자
 



2011-06-06 07시55분
엄밀히 따지면, 아무리 2000 라인이라도 isAlloc 처리가 제대로 되었다면 문제 될 상황이 아닌데 메모리 leak이 있다는 것은 다시 한번 꼼꼼하게 보셔야 할 것 같습니다.

어쨌든, 다른 방법이라면 isAlloc 플래그를 별도로 두지 마시고, Free 할 때마다 아예 hglobal = null;을 해두고 마지막에 if (hglobal == null)이라고 비교하는 것이 더 안전할 것 같습니다.

아니면, 이런 경우에 명시적으로 SafeHandle을 사용하시는 것이 좋을 것 같습니다. 다음의 문서에 보면 AllocHGlobal을 감싸는 SafeHandle을 사용하는 코드가 함께 있습니다.

SafeHandleZeroOrMinusOneIsInvalid Class
; https://learn.microsoft.com/en-us/dotnet/api/microsoft.win32.safehandles.safehandlezeroorminusoneisinvalid
정성태
2011-06-07 12시43분
[김길] 굳이 변수를 선언하지 않고 그렇게 하는 방법이 있었군요.^^

아직 unsafe나 Unmanaged Code에 대해서 제대로 이해를 못 하고 있는데
백그라운드 워커와 맞 물려있다보니 기능 하나 적용하기가 살짝 두렵네요.
알려주신 SafeHandle에 대해서 열심히 검토해봐야겠습니다.
빠른 답변 감사합니다.
[guest]

... 61  62  63  64  65  66  67  68  [69]  70  71  72  73  74  75  ...
NoWriterDateCnt.TitleFile(s)
816박진오7/29/200910739다국어 사이트의 컨텐츠 저장 방식에 대해.. [2]
814서광원7/16/200919735IWebBrowser2를 이용한 프로그램에서 javascript의 alert 창 무시하는 법? [1]
813윤상균7/16/200910987비관리코드와의 상호운용에서 마샬링 질문 [1]
812김현우7/13/200911418usercontrol은 mdi container가 될수 없는데 이를 구현할 방법은 무엇일런지요? [2]
811조민수7/3/200911010MSDN Magazine 한글화 않되나요? [1]
810세경6/29/200916316SmartClient Vista 64bit IE7 [4]
809윤석준6/24/200916245IE -nomerge 옵션으로 새창을 열려고 합니다. [1]
808한승훈6/4/200914389dll import하기 위해 struct 구성시에 struct가 struct를 가지고 있고 포함된 struct가 ByValArray형태일때 해결 [1]
806곰티5/26/200912874defcon pro 설치 원천 봉쇄 방법 문의 [3]
802채승수5/8/200912181신뢰사이트 등록/적용에 관해 질문드립니다. [1]
801채승수4/15/200912891IE8 새세션을 코드로 구현할수 없을까요 [1]
800신동열4/7/200913354IE8에서 스마트 클라이언트 로딩 문제 [2]
7993/27/200917040이벤트 로그 오류 [1]
798천해3/26/200913338IE8.0 에 관해 질문 드립니다. [2]
797궁금..3/23/200913232IE 8 관련 질문.. [2]
796정성태3/20/200912194스마트클라이언트와 ActiveX에 관한 질문 [1]
795김기용3/19/200911793[질문] DHTML 다이얼로그 관련 [2]
794박평옥3/18/200912231Vista에서 URL Shortcut 실행 시 SetSite가 두 번 호출되는 증상에 관해 조언 부탁드립니다. [2]
792김기용3/12/200911375어제 세미나 잘 들었습니다. 질문사항이 있습니다.(ie8 마이그레이션 관련) [4]
791vb표성백2/17/200916605ATL 로 만든 COM 에 문자열 전달하기! C#에서 어떻게 하나요? [1]
790고민중2/16/20099703vista에 vs2005를 사용중입니다. [1]
789지언2/14/200911845MFC & C#(COM) 호환 관련하여 답변좀 부탁드립니다 [2]
788하루야채2/3/200911060스마트클라이언트 Windowless 설정에 대해서 문의드립니다. [2]
787궁금이2/2/200911316TFS 관련하여 질문드리고자 합니다. [2]
786맨날맑음1/30/200912386WPF를 SmartClient로 배포할순 없을까요? [2]
785정성우12/16/200811651Vista 환경에서 VB6로 개발한 어플리케이션이 IE 통해서는 런칭이 안됩니다.. [4]
... 61  62  63  64  65  66  67  68  [69]  70  71  72  73  74  75  ...