C读取mac地址 - 未定义的行为

时间:2016-03-10 09:24:18

标签: c linux sockets ethernet mac-address

我想读取目标的MAC地址。所以我使用以下代码

TS32 get_mac_address(TU8 ** aps8Mac)
{
    TS32 fd;
    struct ifreq ifr;
    CHAR *iface = "eth0";
    TU8 *mac;

    fd = socket(AF_INET, SOCK_DGRAM, 0);

    ifr.ifr_addr.sa_family = AF_INET;
    strncpy(ifr.ifr_name , iface , IFNAMSIZ-1);

    ioctl(fd, SIOCGIFHWADDR, &ifr);

    close(fd);

    mac = (unsigned char *)ifr.ifr_hwaddr.sa_data;
    *aps8Mac = mac;

    return 0;
}

int main(int argc, char **argv)
{
    TU8   *s8Mac = NULL;
    get_mac_address(&s8Mac);
    fprintf(stdout, "MAC address : %.2X:%.2X:%.2X:%.2X:%.2X:%.2X\n" , s8Mac[0], s8Mac[1], s8Mac[2], s8Mac[3], s8Mac[4], s8Mac[5]); 
    fprintf(stdout, "MAC address : %.2X:%.2X:%.2X:%.2X:%.2X:%.2X\n" , s8Mac[0], s8Mac[1], s8Mac[2], s8Mac[3], s8Mac[4], s8Mac[5]);
    return 0;
}

在目标上执行后,我得到了:

MAC地址:00:01:02:03:00:76
MAC地址:FE:76:9C:8C:AB:7E

为什么我不读

MAC地址:00:01:02:03:00:76
MAC地址:00:01:02:03:00:76

编辑1: 针对作为答案给出的建议修改了get_mac_address

TS32 get_mac_address(TU8 ** aps8Mac)
{
    TS32 fd;
    struct ifreq ifr;
    CHAR *iface = "eth0";

    TU8 *mac = malloc(sizeof(TU8) * 17);

    fd = socket(AF_INET, SOCK_DGRAM, 0);

    ifr.ifr_addr.sa_family = AF_INET;
    strncpy(ifr.ifr_name , iface , IFNAMSIZ-1);

  ioctl(fd, SIOCGIFHWADDR, &ifr);

    close(fd);

    mac = (TU8 *)ifr.ifr_hwaddr.sa_data;
    *aps8Mac = mac;
    return 0;
}

2 个答案:

答案 0 :(得分:3)

您正在返回指向本地堆栈变量的指针。这是非法的 - 一旦函数返回,就不再允许您访问其局部变量。正确的方法是在调用者中分配内存并让被调用者将结果复制到那里。

正确结果打印一次的原因是从C函数返回不会使其堆栈空间为零。这些值在堆栈上仍然完好无损,因此它们会传递给fprintf。然后fprintf调用使用堆栈完成其工作,将不同的数据放在指向的位置。

答案 1 :(得分:1)

编辑完成后,为mac地址分配了17个字节 (我不确定为什么你想要17个字节...典型的MAC地址只有6个字节长!)

但是,您没有正确填写这些数据。

你试图在线填写:

mac = (TU8 *)ifr.ifr_hwaddr.sa_data;

但所有这一切都是为mac指针提供与sa_data相同的指针,同时泄漏之前分配的内存。

我相信您希望 数据从一个结构复制到另一个结构:

// Copy 17 bytes from the sa_data to the allocated, reserved space for mac-address
// After this, ifr-->sa_data can be overwritten or destroyed; we don't care
// We have our own copy of the data in mac.
memcpy(mac, (TU8 *)ifr.ifr_hwaddr.sa_data, 17*sizeof(TU8) );

我会尝试像这样重写它:

TS32 get_mac_address(TU8 ** aps8Mac)
{
    TS32 fd;
    struct ifreq ifr;
    CHAR *iface = "eth0";
    const int MAC_SIZE = 6; // MAC addresses are 6-bytes.

    *aps8Mac = malloc(sizeof(TU8) * MAC_SIZE);

    fd = socket(AF_INET, SOCK_DGRAM, 0);

    ifr.ifr_addr.sa_family = AF_INET;
    strncpy(ifr.ifr_name , iface , IFNAMSIZ-1);

    ioctl(fd, SIOCGIFHWADDR, &ifr);
    close(fd);

    // Copy the 6-byte address to the output.
    memcpy(*aps8Mac, ifr.ifr_hwaddr.sa_data, MAC_SIZE);

    return 0;
}