diff options
| author | Cyrill Gorcunov <gorcunov@gmail.com> | 2008-04-19 00:27:29 +0400 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2008-04-29 18:10:03 +0400 |
| commit | 90fe17f4df2f830601ffd422b11d1f7f9a9d0355 (patch) | |
| tree | f5e04bd8ea19125ed5217e41a6fb2dc3b992b5b2 /drivers/misc/thinkpad_acpi.c | |
| parent | a815ab8b5891f3d2515316655729272f68269e3b (diff) | |
| download | linux-90fe17f4df2f830601ffd422b11d1f7f9a9d0355.tar.xz | |
thinkpad_acpi: fix possible NULL pointer dereference if kstrdup failed
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/misc/thinkpad_acpi.c')
| -rw-r--r-- | drivers/misc/thinkpad_acpi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/misc/thinkpad_acpi.c b/drivers/misc/thinkpad_acpi.c index 6cb781262f94..31115c9cfb34 100644 --- a/drivers/misc/thinkpad_acpi.c +++ b/drivers/misc/thinkpad_acpi.c @@ -5826,7 +5826,7 @@ static void __init get_thinkpad_model_data(struct thinkpad_id_data *tp) tp->model_str = kstrdup(dmi_get_system_info(DMI_PRODUCT_VERSION), GFP_KERNEL); - if (strnicmp(tp->model_str, "ThinkPad", 8) != 0) { + if (tp->model_str && strnicmp(tp->model_str, "ThinkPad", 8) != 0) { kfree(tp->model_str); tp->model_str = NULL; } |