On Mon, May 22, 2017 at 03:22:30AM -0400, Keith Busch wrote: > > +out_free_bufs: > > + while (--i >= 0) { > > + struct nvme_host_mem_buf_desc *desc = &dev->host_mem_descs[i]; > > I think you want to use the local desc's variable since > dev->host_mem_descs isn't set if you goto this out_free_bufs label. Indeed..