Hi,
shouldn't there be a check when allocating jpg_hdr? or i am missing something?


 sd->jpeg_hdr = kmalloc(JPEG_HDR_SZ, GFP_KERNEL);
 jpeg_define(sd->jpeg_hdr, height, width,  0x21);
 jpeg_set_qual(sd->jpeg_hdr, sd->quality);

Best Regards.
Nice to see microdia project here :)


On Sat, Jul 18, 2009 at 11:19 AM, Brian Johnson<brij...@gmail.com> wrote:
>> - in the 1st patch, in gspca.c, why is the get_chip_ident check needed
>>   in vidioc_g_register / vidioc_s_register)?
>>
> According the the current v4l2 spec regarding the set and get
> register ioctls drivers that support them must also support the get
> chip ident ioctl. As such in the vidioc_s/g_register ioctls I do the
> check to ensure that the driver also defines the get_chip_ident ioctl.
>
>
> Brian Johnson
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to