diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-09-29 02:10:48 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2011-09-29 02:12:33 -0400 |
commit | bb690c9e2702e49af3be2fb6f03d9b60e0afcab7 (patch) | |
tree | a63ec12c6a3fcfa166a8d93998022a8849413d7f /sound/oss | |
parent | 17d900c4a1b50bc191b3ca58cbd78acc04a1c5b3 (diff) |
sound: oss: use strlcpy() in sound_timer_init()
sound_timer.info.name is a 32 character buffer. This function only
has one caller (in sound/oss/ad1848.c) and it passes as 128 character
buffer as "name". I don't know if this is a problem in real life,
and I doubt we're going to add more OSS drivers so it's unlikely to
become an issue. But we may as well take care of it.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/oss')
-rw-r--r-- | sound/oss/sound_timer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/oss/sound_timer.c b/sound/oss/sound_timer.c index 48cda6c4c257..8021c85f076d 100644 --- a/sound/oss/sound_timer.c +++ b/sound/oss/sound_timer.c | |||
@@ -320,7 +320,7 @@ void sound_timer_init(struct sound_lowlev_timer *t, char *name) | |||
320 | n = sound_alloc_timerdev(); | 320 | n = sound_alloc_timerdev(); |
321 | if (n == -1) | 321 | if (n == -1) |
322 | n = 0; /* Overwrite the system timer */ | 322 | n = 0; /* Overwrite the system timer */ |
323 | strcpy(sound_timer.info.name, name); | 323 | strlcpy(sound_timer.info.name, name, sizeof(sound_timer.info.name)); |
324 | sound_timer_devs[n] = &sound_timer; | 324 | sound_timer_devs[n] = &sound_timer; |
325 | } | 325 | } |
326 | EXPORT_SYMBOL(sound_timer_init); | 326 | EXPORT_SYMBOL(sound_timer_init); |