V4L/DVB (7756): ivtv: use strlcpy instead of strcpy

Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

4 files changed, 15 insertions, 13 deletions

diff --git a/drivers/media/video/ivtv/ivtv-cards.c b/drivers/media/video/ivtv/ivtv-cards.c
index bf9b32a6c1ad..33a2f58f66d7 100644
--- a/drivers/media/video/ivtv/ivtv-cards.c
+++ b/drivers/media/video/ivtv/ivtv-cards.c
@@ -1122,7 +1122,8 @@ int ivtv_get_input(struct ivtv *itv, u16 index, struct v4l2_input *input)
         if (index >= itv->nof_inputs)
                 return -EINVAL;
         input->index = index;
-        strcpy(input->name, input_strs[card_input->video_type - 1]);
+        strlcpy(input->name, input_strs[card_input->video_type - 1],
+                        sizeof(input->name));
         input->type = (card_input->video_type == IVTV_CARD_INPUT_VID_TUNER ?
                         V4L2_INPUT_TYPE_TUNER : V4L2_INPUT_TYPE_CAMERA);
         input->audioset = (1 << itv->nof_audio_inputs) - 1;
@@ -1139,7 +1140,7 @@ int ivtv_get_output(struct ivtv *itv, u16 index, struct v4l2_output *output)
         if (index >= itv->card->nof_outputs)
                 return -EINVAL;
         output->index = index;
-        strcpy(output->name, card_output->name);
+        strlcpy(output->name, card_output->name, sizeof(output->name));
         output->type = V4L2_OUTPUT_TYPE_ANALOG;
         output->audioset = 1;
         output->std = V4L2_STD_ALL;
@@ -1158,7 +1159,8 @@ int ivtv_get_audio_input(struct ivtv *itv, u16 index, struct v4l2_audio *audio)
         memset(audio, 0, sizeof(*audio));
         if (index >= itv->nof_audio_inputs)
                 return -EINVAL;
-        strcpy(audio->name, input_strs[aud_input->audio_type - 1]);
+        strlcpy(audio->name, input_strs[aud_input->audio_type - 1],
+                        sizeof(audio->name));
         audio->index = index;
         audio->capability = V4L2_AUDCAP_STEREO;
         return 0;
@@ -1169,6 +1171,6 @@ int ivtv_get_audio_output(struct ivtv *itv, u16 index, struct v4l2_audioout *aud
         memset(aud_output, 0, sizeof(*aud_output));
         if (itv->card->video_outputs == NULL || index != 0)
                 return -EINVAL;
-        strcpy(aud_output->name, "A/V Audio Out");
+        strlcpy(aud_output->name, "A/V Audio Out", sizeof(aud_output->name));
         return 0;
 }
diff --git a/drivers/media/video/ivtv/ivtv-i2c.c b/drivers/media/video/ivtv/ivtv-i2c.c
index 9824eafee021..771adf47e944 100644
--- a/drivers/media/video/ivtv/ivtv-i2c.c
+++ b/drivers/media/video/ivtv/ivtv-i2c.c
@@ -167,7 +167,8 @@ int ivtv_i2c_register(struct ivtv *itv, unsigned idx)
                 return -1;
         id = hw_driverids[idx];
         memset(&info, 0, sizeof(info));
-        strcpy(info.driver_name, hw_drivernames[idx]);
+        strlcpy(info.driver_name, hw_drivernames[idx],
+                        sizeof(info.driver_name));
         info.addr = hw_addrs[idx];
         for (i = 0; itv->i2c_clients[i] && i < I2C_CLIENTS_MAX; i++) {}
 
diff --git a/drivers/media/video/ivtv/ivtv-ioctl.c b/drivers/media/video/ivtv/ivtv-ioctl.c
index 15cac1812122..6282387ca05c 100644
--- a/drivers/media/video/ivtv/ivtv-ioctl.c
+++ b/drivers/media/video/ivtv/ivtv-ioctl.c
@@ -741,10 +741,9 @@ int ivtv_v4l2_ioctls(struct ivtv *itv, struct file *filp, unsigned int cmd, void
                 struct v4l2_capability *vcap = arg;
 
                 memset(vcap, 0, sizeof(*vcap));
-                strcpy(vcap->driver, IVTV_DRIVER_NAME);     /* driver name */
-                strncpy(vcap->card, itv->card_name,
-                                sizeof(vcap->card)-1);      /* card type */
-                strcpy(vcap->bus_info, pci_name(itv->dev)); /* bus info... */
+                strlcpy(vcap->driver, IVTV_DRIVER_NAME, sizeof(vcap->driver));
+                strlcpy(vcap->card, itv->card_name, sizeof(vcap->card));
+                strlcpy(vcap->bus_info, pci_name(itv->dev), sizeof(vcap->bus_info));
                 vcap->version = IVTV_DRIVER_VERSION;        /* version */
                 vcap->capabilities = itv->v4l2_cap;         /* capabilities */
 
@@ -1018,7 +1017,7 @@ int ivtv_v4l2_ioctls(struct ivtv *itv, struct file *filp, unsigned int cmd, void
                                 ivtv_std_60hz : ivtv_std_50hz;
                 vs->index = idx;
                 vs->id = enum_stds[idx].std;
-                strcpy(vs->name, enum_stds[idx].name);
+                strlcpy(vs->name, enum_stds[idx].name, sizeof(vs->name));
                 break;
         }
 
@@ -1102,10 +1101,10 @@ int ivtv_v4l2_ioctls(struct ivtv *itv, struct file *filp, unsigned int cmd, void
                 ivtv_call_i2c_clients(itv, VIDIOC_G_TUNER, vt);
 
                 if (test_bit(IVTV_F_I_RADIO_USER, &itv->i_flags)) {
-                        strcpy(vt->name, "ivtv Radio Tuner");
+                        strlcpy(vt->name, "ivtv Radio Tuner", sizeof(vt->name));
                         vt->type = V4L2_TUNER_RADIO;
                 } else {
-                        strcpy(vt->name, "ivtv TV Tuner");
+                        strlcpy(vt->name, "ivtv TV Tuner", sizeof(vt->name));
                         vt->type = V4L2_TUNER_ANALOG_TV;
                 }
                 break;
diff --git a/drivers/media/video/ivtv/ivtvfb.c b/drivers/media/video/ivtv/ivtvfb.c
index 3b23fc05f7c4..df789f683e63 100644
--- a/drivers/media/video/ivtv/ivtvfb.c
+++ b/drivers/media/video/ivtv/ivtvfb.c
@@ -532,7 +532,7 @@ static int ivtvfb_get_fix(struct ivtv *itv, struct fb_fix_screeninfo *fix)
 
         IVTVFB_DEBUG_INFO("ivtvfb_get_fix\n");
         memset(fix, 0, sizeof(struct fb_fix_screeninfo));
-        strcpy(fix->id, "cx23415 TV out");
+        strlcpy(fix->id, "cx23415 TV out", sizeof(fix->id));
         fix->smem_start = oi->video_pbase;
         fix->smem_len = oi->video_buffer_size;
         fix->type = FB_TYPE_PACKED_PIXELS;