diff options
| author | Badari Pulavarty <pbadari@gmail.com> | 2007-04-13 08:13:42 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-04-16 18:09:18 -0700 |
| commit | 6f29e35e2d4cdbc3e8785982314e54ec5df4ad37 (patch) | |
| tree | 60b1e8cba4f69a5a1e24fa32cfde38f944f986b3 /arch | |
| parent | 1d00e832f0d65039abdebf83054b323a9aa05687 (diff) | |
| download | lwn-6f29e35e2d4cdbc3e8785982314e54ec5df4ad37.tar.gz lwn-6f29e35e2d4cdbc3e8785982314e54ec5df4ad37.zip | |
cache_k8_northbridges() overflows beyond allocation
cache_k8_northbridges() is storing config values to incorrect locations
(in flush_words) and also its overflowing beyond the allocation, causing
slab verification failures.
Signed-off-by: Badari Pulavarty <pbadari@us.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/x86_64/kernel/k8.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86_64/kernel/k8.c b/arch/x86_64/kernel/k8.c index 6416682d33d0..bc11b32e8b4d 100644 --- a/arch/x86_64/kernel/k8.c +++ b/arch/x86_64/kernel/k8.c @@ -61,8 +61,8 @@ int cache_k8_northbridges(void) dev = NULL; i = 0; while ((dev = next_k8_northbridge(dev)) != NULL) { - k8_northbridges[i++] = dev; - pci_read_config_dword(dev, 0x9c, &flush_words[i]); + k8_northbridges[i] = dev; + pci_read_config_dword(dev, 0x9c, &flush_words[i++]); } k8_northbridges[i] = NULL; return 0; |