Michal maybe forgot it merely, we should add code to check the return value of kzalloc to make the code more robust.
Signed-off-by: Wang YanQing <[email protected]> --- drivers/video/uvesafb.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/video/uvesafb.c b/drivers/video/uvesafb.c index b0e2a42..2f8f82d 100644 --- a/drivers/video/uvesafb.c +++ b/drivers/video/uvesafb.c @@ -659,6 +659,8 @@ static int __devinit uvesafb_vbe_getedid(struct uvesafb_ktask *task, task->t.flags = TF_BUF_RET | TF_BUF_ESDI; task->t.buf_len = EDID_LENGTH; task->buf = kzalloc(EDID_LENGTH, GFP_KERNEL); + if (!task->buf) + return -ENOMEM; err = uvesafb_exec(task); -- 1.7.11.1.116.g8228a23 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
