[dm-devel] [PATCH] dm-mpath: Track invalid map_context
Alasdair G Kergon
agk at redhat.com
Mon Mar 19 15:20:07 UTC 2012
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
More information about the dm-devel
mailing list