Re: [PATCH V2] slub: avoid potential NULL dereference orcorruption

[Christoph Lameter]

On Wed, 23 Nov 2011, Pekka Enberg wrote:

> On Tue, 22 Nov 2011, Eric Dumazet wrote:
> > show_slab_objects() can trigger NULL dereferences or memory corruption.
> >
> > Another cpu can change its c->page to NULL or c->node to NUMA_NO_NODE
> > while we use them.
> >
> > Use ACCESS_ONCE(c->page) and ACCESS_ONCE(c->node) to make sure this
> > cannot happen.
> >
> > Signed-off-by: Eric Dumazet <eric.dumazet@xxxxxxxxx>
>
> Did someone test this patch? Does it fix any of the reported issues?

It does not fix any current issues but it is safe against potential
compiler refetching of variables that we have already checked against
bad values.

I thought I acked it already?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/