Microsoft MVP성태의 닷넷 이야기
메모리 해제 예외 처리 관련.. [링크 복사], [링크+제목 복사]
조회: 9250
글쓴 사람
김길
홈페이지
첨부 파일
 
타인의 소스를 유지보수 해야 하는데 메모리 할당과 해제 부분의 예외 처리가 쉽지 않아 문의 드립니다.

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);을 실행하고 있습니다.

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




donaricano-btn



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


비밀번호

댓글 쓴 사람
 



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

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

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

SafeHandleZeroOrMinusOneIsInvalid Class
; http://msdn.microsoft.com/en-us/library/microsoft.win32.safehandles.safehandlezeroorminusoneisinvalid(v=vs.80).aspx

 
정성태
2011-06-07 12시43분
[김길] 굳이 변수를 선언하지 않고 그렇게 하는 방법이 있었군요.^^

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

[손님]

... 31  32  33  34  35  36  37  38  39  40  41  42  43  [44]  45  ...
NoWriterDateCnt.TitleFile(s)
1053폭풍코딩4/25/20125294스마트클라이언트 ctrl-c 복사가 작동하지 않습니다. [1]
1052에슈리온4/17/201218335ClickOnce 배포시 클라이언트의 설치경로를 지정할 수 없나요? [1]
1051최정수4/10/20128025WCF 클라이언트 비정상 종료 관련 [1]
1050에슈리온4/6/20128906관리자 권한과 ClickOnce, 그리고 Bootstrapper문제 [6]
1049김성혁4/4/20125298스마트클라이언트 stand-alone 방식의 배포 문제.. [1]
1048C#조으다3/27/20127179어셈블리 로드 / 언로드와 관련해서.. [6]
1047김우형3/23/20125716WCF 서비스를 이용한 데이터 전송 중 Exception 문의 [1]
1044임동찬2/29/20125190웹에서 COM Exception.. [2]
1043박성준2/24/20124639Lazy<T> 의 지연객체 생성 전에 실 객체의 Attribute분석 [2]파일 다운로드1
1042이성환2/20/20125130BlockingMethod에 빠진 스레드를 즉시 죽이고 싶습니다...;ㅅ; [2]파일 다운로드1
1039김재영2/15/20124562어셈블리에 사용자마다 다른 값을 적용하여 자동 빌드 방법이 있을까요? [2]
1037윈드로니2/12/20125864WPF 관련 질문 드립니다. [2]
1035임동찬2/9/20124408XML, XSD, XMLCodeGenerator 관련 [2]
1033곽성현2/7/20124216훔..윈도우 디바이스 드라이버 관련 [1]
1030궁빈2/2/20126466msbuild를 이용한 웹게시 및 자동 Dist 방법론에 대해 질문 드립니다!! [6]
1028신정환1/30/20124572기존 legacy Windows application이 WoA 환경에서 동작되지 않는 이유가 무얼까요? [2]
1026노현철1/19/20125754WPF가 XP에서 느리게 로딩되는 현상에 대해 질문드립니다. [4]
1025궁그미1/18/20126178네이버 소켓 로그인 이해가 안되는점이 있는데요 [2]
1022선무당1/11/20125016TFS 에서 공통 Assembly 공유 방법에 대해서 해결책이 없을까요? [2]
1021김재영1/8/20124355대리자를 메소드 파라메터로 넘겨도 됩니까? [2]파일 다운로드1
1019남산골12/28/20115518안녕하세요!! 성태님~~ [3]
1018강한구12/29/20118631WPF ClickOnce 배포시 콤포넌트 dll 다운로드 문제에 대하여 질문 있습니다. [1]
1017임동찬12/28/20116379smart client와 xbop, silverlight [6]
1015개똥이12/22/20115040RMCLOCK 설정 관련 질문이요ㅜ [2]
1011사길수12/16/20114378hyper-v 이더넷 카드 관련 문의 좀 드립니다. [1]
1006루로니12/13/20116691윈도우7과 XP의 DLL후킹 처리에 관해서 질문입니다. [3]
... 31  32  33  34  35  36  37  38  39  40  41  42  43  [44]  45  ...