diff for duplicates of <4AADB5EE.9090902@redhat.com> diff --git a/a/1.txt b/N1/1.txt index d537c69..c114083 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -52,7 +52,7 @@ On 09/14/2009 02:33 AM, Eric Paris wrote: > [snip] > [ 59.923261] Object 0xffff8800087fb028: 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b kkkkkkkkkkkkkkkk > [ 59.923261] Object 0xffff8800087fb038: 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b kkkkkkkkkkkkkkkk -> [ 59.923261] Redzone 0xffff8800087fc048: bb bb bb bb bb bb bb bb »»»»»»»» +> [ 59.923261] Redzone 0xffff8800087fc048: bb bb bb bb bb bb bb bb A>>A>>A>>A>>A>>A>>A>>A>> > [ 59.923261] Padding 0xffff8800087fc088: 5a 5a 5a 5a 5a 5a 5a 5a ZZZZZZZZ > [ 59.923261] Pid: 1781, comm: insmod Not tainted 2.6.31-rc2 #33 > [ 59.923261] Call Trace: diff --git a/a/2.txt b/N1/2.txt index a0726c5..8b13789 100644 --- a/a/2.txt +++ b/N1/2.txt @@ -1,30 +1 @@ ->From f5a04f3db3adaa605e951ae9f46708da63979bda Mon Sep 17 00:00:00 2001 -From: Xiaotian Feng <dfeng@redhat.com> -Date: Mon, 14 Sep 2009 11:12:27 +0800 -Subject: [PATCH] slub: fix kmem_cache wrongly freed in kmem_cache_create -commit a0e1d1b move sysfs operations outside of slub_lock, but wrongly -freed kmem_cache in any cases (should only free kmem_cache when open failed). - -Signed-off-by: Xiaotian Feng <dfeng@redhat.com> ---- - mm/slub.c | 4 ++-- - 1 files changed, 2 insertions(+), 2 deletions(-) - -diff --git a/mm/slub.c b/mm/slub.c -index b627675..40e12d5 100644 ---- a/mm/slub.c -+++ b/mm/slub.c -@@ -3337,8 +3337,8 @@ struct kmem_cache *kmem_cache_create(const char *name, size_t size, - goto err; - } - return s; -- } -- kfree(s); -+ } else -+ kfree(s); - } - up_write(&slub_lock); - --- -1.6.2.5 diff --git a/a/content_digest b/N1/content_digest index 9a05f75..7a0ac1a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -64,7 +64,7 @@ "> [snip]\n" "> [ 59.923261] Object 0xffff8800087fb028: 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b kkkkkkkkkkkkkkkk\n" "> [ 59.923261] Object 0xffff8800087fb038: 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b 6b kkkkkkkkkkkkkkkk\n" - "> [ 59.923261] Redzone 0xffff8800087fc048: bb bb bb bb bb bb bb bb \302\273\302\273\302\273\302\273\302\273\302\273\302\273\302\273\n" + "> [ 59.923261] Redzone 0xffff8800087fc048: bb bb bb bb bb bb bb bb A>>A>>A>>A>>A>>A>>A>>A>>\n" "> [ 59.923261] Padding 0xffff8800087fc088: 5a 5a 5a 5a 5a 5a 5a 5a ZZZZZZZZ\n" "> [ 59.923261] Pid: 1781, comm: insmod Not tainted 2.6.31-rc2 #33\n" "> [ 59.923261] Call Trace:\n" @@ -91,35 +91,5 @@ "\01:2\0" "fn\00001-slub-fix-kmem_cache-wrongly-freed-in-kmem_cache_cre.patch\0" "b\0" - ">From f5a04f3db3adaa605e951ae9f46708da63979bda Mon Sep 17 00:00:00 2001\n" - "From: Xiaotian Feng <dfeng@redhat.com>\n" - "Date: Mon, 14 Sep 2009 11:12:27 +0800\n" - "Subject: [PATCH] slub: fix kmem_cache wrongly freed in kmem_cache_create\n" - "\n" - "commit a0e1d1b move sysfs operations outside of slub_lock, but wrongly\n" - "freed kmem_cache in any cases (should only free kmem_cache when open failed).\n" - "\n" - "Signed-off-by: Xiaotian Feng <dfeng@redhat.com>\n" - "---\n" - " mm/slub.c | 4 ++--\n" - " 1 files changed, 2 insertions(+), 2 deletions(-)\n" - "\n" - "diff --git a/mm/slub.c b/mm/slub.c\n" - "index b627675..40e12d5 100644\n" - "--- a/mm/slub.c\n" - "+++ b/mm/slub.c\n" - "@@ -3337,8 +3337,8 @@ struct kmem_cache *kmem_cache_create(const char *name, size_t size,\n" - " \t\t\t\tgoto err;\n" - " \t\t\t}\n" - " \t\t\treturn s;\n" - "-\t\t}\n" - "-\t\tkfree(s);\n" - "+\t\t} else\n" - "+\t\t\tkfree(s);\n" - " \t}\n" - " \tup_write(&slub_lock);\n" - " \n" - "-- \n" - 1.6.2.5 -cf729a171e48b0e8e18857180004ef3ab14a27642be0b580b68fdd996f5a3560 +f8212af7c1182479583730cfbde31cbe0ed4a6aa4074847b8461fa685915b0f7
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.