[MediaContent] Wrong Implement of Dispose pattern

[myroot]

TizenFX/src/Tizen.Content.MediaContent/Tizen.Content.MediaContent/ObjectKeeper.cs

Lines 52 to 63 in 3566375

	protected virtual void Dispose(bool disposing)
	{
	if (!disposedValue)
	{
	if (disposing)
	{
	_handle.Free();
	}
	disposedValue = true;
	}
	}

_handle should be always freed.

https://msdn.microsoft.com/ko-kr/library/system.runtime.interopservices.gchandle(v=vs.110).aspx
When the handle goes out of scope you must explicitly release it by calling the M:System.Runtime.InteropServices.GCHandle.Free method; otherwise, memory leaks may occur. When you free a pinned handle, the associated object will be unpinned and will become eligible for garbage collection, if there are no other references to it.

[myroot]

Of course, this object always uses Dispose, so there is no real problem.

[coderhyme]

You mean the problem is when the disposing value is false. Right?

[myroot]

Yes, the disposing value was false when disposed by finalizer and in this case _handle was not released

[coderhyme]

Has been fixed by #81.