lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 19 Feb 2016 10:41:32 +0900 From: Joonsoo Kim <js1304@...il.com> To: Alexander Potapenko <glider@...gle.com> Cc: Joonsoo Kim <iamjoonsoo.kim@....com>, kasan-dev@...glegroups.com, Christoph Lameter <cl@...ux.com>, LKML <linux-kernel@...r.kernel.org>, Dmitriy Vyukov <dvyukov@...gle.com>, Andrey Ryabinin <ryabinin.a.a@...il.com>, Linux Memory Management List <linux-mm@...ck.org>, Andrey Konovalov <adech.fo@...il.com>, Andrew Morton <akpm@...ux-foundation.org>, Steven Rostedt <rostedt@...dmis.org> Subject: Re: [PATCH v1 2/8] mm, kasan: SLAB support > On Mon, Feb 1, 2016 at 3:15 AM, Joonsoo Kim <iamjoonsoo.kim@....com> wrote: >> On Thu, Jan 28, 2016 at 02:29:42PM +0100, Alexander Potapenko wrote: >>> On Thu, Jan 28, 2016 at 1:37 PM, Alexander Potapenko <glider@...gle.com> wrote: >>> > >>> > On Jan 28, 2016 8:44 AM, "Joonsoo Kim" <iamjoonsoo.kim@....com> wrote: >>> >> >>> >> On Wed, Jan 27, 2016 at 07:25:07PM +0100, Alexander Potapenko wrote: >>> >> > This patch adds KASAN hooks to SLAB allocator. >>> >> > >>> >> > This patch is based on the "mm: kasan: unified support for SLUB and >>> >> > SLAB allocators" patch originally prepared by Dmitry Chernenkov. >>> >> > >>> >> > Signed-off-by: Alexander Potapenko <glider@...gle.com> >>> >> > --- >>> >> > Documentation/kasan.txt | 5 ++- >>> >> >>> >> ... >>> >> >>> >> > +#ifdef CONFIG_SLAB >>> >> > +struct kasan_alloc_meta *get_alloc_info(struct kmem_cache *cache, >>> >> > + const void *object) >>> >> > +{ >>> >> > + return (void *)object + cache->kasan_info.alloc_meta_offset; >>> >> > +} >>> >> > + >>> >> > +struct kasan_free_meta *get_free_info(struct kmem_cache *cache, >>> >> > + const void *object) >>> >> > +{ >>> >> > + return (void *)object + cache->kasan_info.free_meta_offset; >>> >> > +} >>> >> > +#endif >>> >> >>> >> I cannot find the place to store stack info for free. get_free_info() >>> >> isn't used except print_object(). Plese let me know where. >>> > >>> > This is covered by other patches in this patchset. >> >> This should be covered by this patch. Stroing and printing free_info >> is already done on SLUB and it is meaningful without quarantain. 2016-02-18 21:58 GMT+09:00 Alexander Potapenko <glider@...gle.com>: > However this info is meaningless without saved stack traces, which are > only introduced in the stackdepot patch (see "[PATCH v1 5/8] mm, > kasan: Stackdepot implementation. Enable stackdepot for SLAB") Not meaningless. You already did it for allocation caller without saved stack traces. What makes difference between alloc/free? Thanks.
Powered by blists - more mailing lists