On 1/16/2020 4:31 PM, Scott Branden wrote:
> Hi Shiping,
>
> Here is another small change to cleanup.
>> + } else {
>> + memset(info, 0, sizeof(struct ecc_error_info));
> This should be sizeof(*info), not sizeof(struct ecc_error_info)
> for better programming to allow info to change type in the future
> without the code changing.
Yes, two occurrences will be replaced in the next patch, thanks!
--
Best regards,
Shiping Ji
