On 05/08/2018 08:20 PM, Andrey Konovalov wrote:
> diff --git a/mm/slub.c b/mm/slub.c
> index 44aa7847324a..4fcd1442a761 100644
> --- a/mm/slub.c
> +++ b/mm/slub.c
> @@ -1351,10 +1351,10 @@ static inline void dec_slabs_node(struct kmem_cache *s, int node,
> * Hooks for other subsystems that check memory allocations. In a typical
> * production configuration these hooks all should produce no code at all.
> */
> -static inline void kmalloc_large_node_hook(void *ptr, size_t size, gfp_t flags)
> +static inline void kmalloc_large_node_hook(void **ptr, size_t size, gfp_t flags)
> {
> - kmemleak_alloc(ptr, size, 1, flags);
> - kasan_kmalloc_large(ptr, size, flags);
> + kmemleak_alloc(*ptr, size, 1, flags);
> + *ptr = kasan_kmalloc_large(*ptr, size, flags);
Why not 'return ptr' like everywhere else?
