We should add the Dispose pattern to the style guide so we don't have to go hunting for it whenever we need it. We should also audit the existing code to make sure it follows the correct pattern where it needs to.

private bool disposed;
public virtual void Dispose(bool disposing)
{
    if (disposed) return;
    disposed = true;
    if (disposing) {
        // dispose managed stuff
    }
    // dispose native stuff that we hold an exclusive reference to (nobody else is going to kill them off via GC or finalizer). Do not access any data via managed references here.
}

public void Dispose()
{
  Dispose(true);
}