diff options
| author | Nadia Derbey <Nadia.Derbey@bull.net> | 2008-11-20 00:36:08 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-11-20 03:49:57 +0100 |
| commit | e00b4ff7ebf098b11b11be403921c1cf41d9e321 (patch) | |
| tree | ca0b085b803128efea92ab98723adcefb35556fc /lib/dump_stack.c | |
| parent | drivers/video: bad error test before a dereference (diff) | |
| download | linux-e00b4ff7ebf098b11b11be403921c1cf41d9e321.tar.xz linux-e00b4ff7ebf098b11b11be403921c1cf41d9e321.zip | |
sysvipc: fix the ipc structures initialization
A problem was found while reviewing the code after Bugzilla bug
http://bugzilla.kernel.org/show_bug.cgi?id=11796.
In ipc_addid(), the newly allocated ipc structure is inserted into the
ipcs tree (i.e made visible to readers) without locking it. This is not
correct since its initialization continues after it has been inserted in
the tree.
This patch moves the ipc structure lock initialization + locking before
the actual insertion.
Signed-off-by: Nadia Derbey <Nadia.Derbey@bull.net>
Reported-by: Clement Calmels <cboulte@gmail.com>
Cc: Manfred Spraul <manfred@colorfullife.com>
Cc: <stable@kernel.org> [2.6.27.x]
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/dump_stack.c')
0 files changed, 0 insertions, 0 deletions