cleanup++ style++ and all that mumbo jambo

I also shot m_buffer dead, which was an amazingly useless member as we
alloc the buffer in lock() and free it in unlock(), both are callbacks
and in both we get a ptr to the buffer.