#577 memset in v4l2 driver

[Jayen Ashar]

Not strictly a patch, but I noticed line 553 of playerstage > code > player > trunk > server > drivers > camera > v4l2 > camerav4l2.cc has:
memset(data, 0, sizeof data);
and i think it should be:
memset(data, 0, sizeof *data);

or, alternatively, use calloc on line 547.

--Jayen

http://playerstage.svn.sourceforge.net/viewvc/playerstage/code/player/trunk/server/drivers/camera/v4l2/camerav4l2.cc?revision=8408&view=markup

[Paul Osmialowski]

Oh, lol, it is, only four bytes are filled with 0. Should be: memset(data, 0, sizeof(player_camera_data_t));
Fortunately, in following lines all relevant fields of player_camera_data_t structure are filled by correct values.

[Paul Osmialowski]

Fixed in patch 2962489, thanks for pointing this!

[Toby Collett]

The changes have been made in SVN trunk.
Thank you for your contribution.