From: Christopher Brannon Date: Tue, 22 Nov 2011 19:46:23 +0000 (-0600) Subject: Staging: speakup: Don't try to access an unallocated struct. X-Git-Tag: v3.3-rc1~152^2~101^2~252 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4afaee1561e207683dbb886b30a842ffcc22e366;p=profile%2Fivi%2Fkernel-x86-ivi.git Staging: speakup: Don't try to access an unallocated struct. In speakup_init, we have the following: if (quiet_boot) spk_shut_up |= 0x01; And in spk_types.h: This patch moves the statement in speakup_init so that speakup_console[vc->vc_num] is guaranteed to be allocated when it executes. Signed-off-by: Christopher Brannon Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c index 8be5604..c7b03f0 100644 --- a/drivers/staging/speakup/main.c +++ b/drivers/staging/speakup/main.c @@ -2268,8 +2268,6 @@ static int __init speakup_init(void) set_mask_bits(0, i, 2); set_key_info(key_defaults, key_buf); - if (quiet_boot) - spk_shut_up |= 0x01; /* From here on out, initializations can fail. */ err = speakup_add_virtual_keyboard(); @@ -2292,6 +2290,9 @@ static int __init speakup_init(void) goto error_kobjects; } + if (quiet_boot) + spk_shut_up |= 0x01; + err = speakup_kobj_init(); if (err) goto error_kobjects;