diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2006-11-27 23:44:57 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2006-11-27 23:44:57 +0000 |
| commit | 229b3d207216b23a107e4e1ee5584412f6b9318d (patch) | |
| tree | d21a19b020f0ff35f7d0fa24f9812d68057ff06a /networking/udhcp/socket.c | |
| parent | e2d3ded3549edd58fe3b39f2254c65f0808bcac2 (diff) | |
| download | busybox-w32-229b3d207216b23a107e4e1ee5584412f6b9318d.tar.gz busybox-w32-229b3d207216b23a107e4e1ee5584412f6b9318d.tar.bz2 busybox-w32-229b3d207216b23a107e4e1ee5584412f6b9318d.zip | |
do not overflow ifr_name. maybe it was safe in some places,
but not everywhere. err to the safe side.
Diffstat (limited to 'networking/udhcp/socket.c')
| -rw-r--r-- | networking/udhcp/socket.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/networking/udhcp/socket.c b/networking/udhcp/socket.c index b27dca300..92bf4cccf 100644 --- a/networking/udhcp/socket.c +++ b/networking/udhcp/socket.c | |||
| @@ -51,7 +51,7 @@ int read_interface(char *interface, int *ifindex, uint32_t *addr, uint8_t *arp) | |||
| 51 | } | 51 | } |
| 52 | 52 | ||
| 53 | ifr.ifr_addr.sa_family = AF_INET; | 53 | ifr.ifr_addr.sa_family = AF_INET; |
| 54 | strcpy(ifr.ifr_name, interface); | 54 | strncpy(ifr.ifr_name, interface, sizeof(ifr.ifr_name)); |
| 55 | if (addr) { | 55 | if (addr) { |
| 56 | if (ioctl(fd, SIOCGIFADDR, &ifr) == 0) { | 56 | if (ioctl(fd, SIOCGIFADDR, &ifr) == 0) { |
| 57 | bb_perror_msg("SIOCGIFADDR failed, is the interface up and configured?"); | 57 | bb_perror_msg("SIOCGIFADDR failed, is the interface up and configured?"); |