[Someone]

I haven't bothered to check the docs, but isn't that a race condition in memory_init, registering the device before allocating the buffer?

Or are drivers loaded while holding a global lock? Even though driver (un)loading probably is fairly rare, that seems a bit heavy to me.

Worse, the 'goto fail' path, if it is ever hit, seems to leak a register_chrdev call.