Re: [PATCH 44/53] Input: atmel_mxt_ts - Verify Information Blockchecksum on probe

From: Nick Dyer
Date: Mon Jun 10 2013 - 05:35:39 EST


Yufeng Shen wrote:
>> + dev_info(&client->dev,
>> + "Family: %u Variant: %u Firmware V%u.%u.%02X Objects: %u\n",
>> + info->family_id, info->variant_id, info->version >> 4,
>> + info->version & 0xf, info->build, info->object_num);
>> +
>
> Use data->info->XXX here, info was pointing to buf which could have
> changed after krealloc.

Thank you, that's a really good spot. Caused when I merged the dev_info
line into that function. It actually explains a report I had about getting
junk in this output but the info_crc check succeeding, hadn't had time to
investigate yet.

I will fix.
--
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/