diff options
| author | Chen Gang <gang.chen@asianux.com> | 2013-01-07 05:42:46 +0100 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2013-01-07 20:34:45 +0100 |
| commit | 407ee23725bba0f273963bb744fea6cb3cf78bf4 (patch) | |
| tree | 6a3edef9552166a9ebfb1ac125ab2582678d7784 /drivers/net | |
| parent | drivers/net/wireless/mwl8k.c: avoid use-after-free (diff) | |
| download | linux-407ee23725bba0f273963bb744fea6cb3cf78bf4.tar.xz linux-407ee23725bba0f273963bb744fea6cb3cf78bf4.zip | |
drivers/net/wireless/iwlegacy: use strlcpy instead of strncpy
The fields must be null-terminated, or simple_strtoul will cause issue.
Signed-off-by: Chen Gang <gang.chen@asianux.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/wireless/iwlegacy/3945-mac.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/iwlegacy/3945-mac.c b/drivers/net/wireless/iwlegacy/3945-mac.c index d604b4036a76..3726cd6fcd75 100644 --- a/drivers/net/wireless/iwlegacy/3945-mac.c +++ b/drivers/net/wireless/iwlegacy/3945-mac.c @@ -3273,7 +3273,7 @@ il3945_store_measurement(struct device *d, struct device_attribute *attr, if (count) { char *p = buffer; - strncpy(buffer, buf, min(sizeof(buffer), count)); + strlcpy(buffer, buf, sizeof(buffer)); channel = simple_strtoul(p, NULL, 0); if (channel) params.channel = channel; |