diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-06-23 11:05:59 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-23 16:43:08 +0200 |
| commit | a5d6839b75ca4a29b5e0a5595914a8715e8f746d (patch) | |
| tree | 8bb38547046d71b42b687e7c07350fe02d9b3d63 /drivers/md/raid6algos.c | |
| parent | [PATCH] CodingStyle: add typedefs chapter (diff) | |
| download | linux-a5d6839b75ca4a29b5e0a5595914a8715e8f746d.tar.xz linux-a5d6839b75ca4a29b5e0a5595914a8715e8f746d.zip | |
[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/raid6algos.c')
| -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 51c63c0cf1c9..926576156578 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; |