diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-06-23 02:05:59 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-23 07:43:08 -0700 |
| commit | a5d6839b75ca4a29b5e0a5595914a8715e8f746d (patch) | |
| tree | 8bb38547046d71b42b687e7c07350fe02d9b3d63 /drivers/md | |
| parent | 226a6b84aaaf1fac7a5d41cf4e7387fd9ba895d5 (diff) | |
| download | kernel_samsung_crespo-a5d6839b75ca4a29b5e0a5595914a8715e8f746d.zip kernel_samsung_crespo-a5d6839b75ca4a29b5e0a5595914a8715e8f746d.tar.gz kernel_samsung_crespo-a5d6839b75ca4a29b5e0a5595914a8715e8f746d.tar.bz2 | |
[PATCH] drivers/md/raid6algos.c: fix a NULL dereference
This patch fixes a NULL dereference spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Neil Brown <neilb@cse.unsw.edu.au>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/raid6algos.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/md/raid6algos.c b/drivers/md/raid6algos.c index 51c63c0..9265761 100644 --- a/drivers/md/raid6algos.c +++ b/drivers/md/raid6algos.c @@ -139,15 +139,14 @@ int __init raid6_select_algo(void) } } - if ( best ) + if (best) { printk("raid6: using algorithm %s (%ld MB/s)\n", best->name, (bestperf*HZ) >> (20-16+RAID6_TIME_JIFFIES_LG2)); - else + raid6_call = *best; + } else printk("raid6: Yikes! No algorithm found!\n"); - raid6_call = *best; - free_pages((unsigned long)syndromes, 1); return best ? 0 : -EINVAL; |