staging: ft1000: fix kernel info leak to userland for statistics
get_stat_data is not fully initialized before copy_to_user() call, e.g. LedStat is not always set. Signed-off-by: Vasiliy Kulikov <segooon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
b7378b9ed6
commit
86ee1e49cf
|
@ -616,7 +616,7 @@ static long ft1000_ChIoctl (struct file *File, unsigned int Command,
|
||||||
break;
|
break;
|
||||||
case IOCTL_GET_DSP_STAT_CMD:
|
case IOCTL_GET_DSP_STAT_CMD:
|
||||||
//DEBUG("FT1000:ft1000_ChIoctl: IOCTL_FT1000_GET_DSP_STAT called\n");
|
//DEBUG("FT1000:ft1000_ChIoctl: IOCTL_FT1000_GET_DSP_STAT called\n");
|
||||||
|
memset(&get_stat_data, 0, sizeof(get_stat_data));
|
||||||
memcpy(get_stat_data.DspVer, info->DspVer, DSPVERSZ);
|
memcpy(get_stat_data.DspVer, info->DspVer, DSPVERSZ);
|
||||||
memcpy(get_stat_data.HwSerNum, info->HwSerNum, HWSERNUMSZ);
|
memcpy(get_stat_data.HwSerNum, info->HwSerNum, HWSERNUMSZ);
|
||||||
memcpy(get_stat_data.Sku, info->Sku, SKUSZ);
|
memcpy(get_stat_data.Sku, info->Sku, SKUSZ);
|
||||||
|
|
Loading…
Reference in New Issue