diff options
author | Márton Németh <nm127@freemail.hu> | 2009-11-21 11:46:12 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-12-05 15:41:45 -0500 |
commit | b1858190fc0180df70e9e7bab24a679129643f43 (patch) | |
tree | 14f383168d571b79d195e37bedea997ccc510d11 | |
parent | cd0e280f1bbecebcd20ed0ddd4dd8fb03a506b3c (diff) |
V4L/DVB (13414): ttusb-dec: do not overwrite the first part of phys string
Use strlcat() to append a string to the previously created first part.
The semantic match that finds this kind of problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression dev;
expression phys;
expression str;
expression size;
@@
usb_make_path(dev, phys, size);
- strlcpy(phys, str, size);
+ strlcat(phys, str, size);
// </smpl>
Signed-off-by: Márton Németh <nm127@freemail.hu>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/dvb/ttusb-dec/ttusb_dec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb/ttusb-dec/ttusb_dec.c b/drivers/media/dvb/ttusb-dec/ttusb_dec.c index d91e0638448f..53baccbab17f 100644 --- a/drivers/media/dvb/ttusb-dec/ttusb_dec.c +++ b/drivers/media/dvb/ttusb-dec/ttusb_dec.c | |||
@@ -1198,7 +1198,7 @@ static int ttusb_init_rc( struct ttusb_dec *dec) | |||
1198 | int err; | 1198 | int err; |
1199 | 1199 | ||
1200 | usb_make_path(dec->udev, dec->rc_phys, sizeof(dec->rc_phys)); | 1200 | usb_make_path(dec->udev, dec->rc_phys, sizeof(dec->rc_phys)); |
1201 | strlcpy(dec->rc_phys, "/input0", sizeof(dec->rc_phys)); | 1201 | strlcat(dec->rc_phys, "/input0", sizeof(dec->rc_phys)); |
1202 | 1202 | ||
1203 | input_dev = input_allocate_device(); | 1203 | input_dev = input_allocate_device(); |
1204 | if (!input_dev) | 1204 | if (!input_dev) |