Re: [PATCH v2 6/6] net-PPP: Delete another unnecessary assignment in mppe_alloc()

From: SF Markus Elfring
Date: Fri Dec 05 2014 - 07:50:36 EST


>> The data structure element "sha1" was assigned a null pointer by the
>> mppe_alloc() after a function call "crypto_alloc_hash" failed.
>
> This patch is also buggy.

Do you really want to keep a variable reset after a detected failure?

Regards,
Markus

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/