On Mon, Mar 19, 2012 at 04:15:28PM +0100, Hannes Reinecke wrote: > - if (r < 0 || r == DM_MAPIO_REQUEUE) > + if (r < 0 || r == DM_MAPIO_REQUEUE) { > mempool_free(mpio, m->mpio_pool); > + map_context->ptr = NULL; > + } What about the other places that do mempool_free() ? Should they clear it too? Is it better to swap the statement order - clear it *before* freeing i Alasdair -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel