Re: question about drivers/gpu/drm/drm_ioc32.c

From: devendra.aaru
Date: Mon Jan 30 2012 - 06:33:27 EST


On Mon, Jan 30, 2012 at 5:28 AM, Julia Lawall <julia.lawall@xxxxxxx> wrote:
> On Mon, 30 Jan 2012, Dave Airlie wrote:
>
>> On Sat, Jan 28, 2012 at 7:28 PM, Julia Lawall <julia.lawall@xxxxxxx>
>> wrote:
>>>
>>> In the function compat_drm_getclient, I have the impression that the
>>> structure c32 is copied to user level with the field idx uninitialized?
>>
>>
>> Looks like it alright,
>>
>> care to send a patch?
>
>
> What should it be initialized to?  The idx value that was got and put
> earlier?
>
> thanks,
> julia
>
There was again a variable in drm_client32_t struct, idx which was
uninitialised.
c32.idx = idx is ok?

Thanks,
Dev.
> --
> 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/
--
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/