Question

我编写了一个代码来创建和设置VLAN接口的属性。我可以设置IP地址，但是我在设置网关地址和读取设备标志时遇到问题。请查看代码并提出可能出错的建议。

我在函数generic_ioctrlcall中遇到错误。

/**
* Create socket function
*/
int create_socket() 
{

  int sockfd = 0;

  sockfd = socket(AF_INET, SOCK_DGRAM, 0);
  if(sockfd == -1){
    fprintf(stderr, "Could not get socket.\n");
    return -1;
  }

  return sockfd;

}

/**
* Generic ioctrlcall to reduce code size
*/
int generic_ioctrlcall(int sockfd, u_long *flags, struct ifreq *ifr) {

  if (ioctl(sockfd, (long unsigned int)flags, &ifr) < 0) {
    fprintf(stderr, "ioctl: %s\n", (char *)flags);
    return -1;
  }
  return 1;
}

/**
* Set route with metric 100
*/
int set_route(int sockfd, char *gateway_addr,  struct sockaddr_in *addr) {

  struct rtentry route;
  int err = 0;
  memset(&route, 0, sizeof(route));
  addr = (struct sockaddr_in*) &route.rt_gateway;
  addr->sin_family = AF_INET;
  addr->sin_addr.s_addr = inet_addr(gateway_addr);
  addr = (struct sockaddr_in*) &route.rt_dst;
  addr->sin_family = AF_INET;
  addr->sin_addr.s_addr = inet_addr("0.0.0.0");
  addr = (struct sockaddr_in*) &route.rt_genmask;
  addr->sin_family = AF_INET;
  addr->sin_addr.s_addr = inet_addr("0.0.0.0");
  route.rt_flags = RTF_UP | RTF_GATEWAY;
  route.rt_metric = 100;

  if ((err = ioctl(sockfd, SIOCADDRT, &route)) < 0) {
    fprintf(stderr, "ioctl: %s\n", (char *)flags);
    return -1;
  }
  return 1;

}

/**
* Set ip function
*/
int set_ip(char *iface_name, char *ip_addr, char *gateway_addr)
{
  if(!iface_name)
    return -1;

  struct ifreq ifr;
  struct sockaddr_in sin;
  int sockfd = create_socket();

  sin.sin_family = AF_INET;

  // Convert IP from numbers and dots to binary notation
  inet_aton(ip_addr,&sin.sin_addr.s_addr);

  /* get interface name */
  strncpy(ifr.ifr_name, iface_name, IFNAMSIZ);

  /* Read interface flags */
  generic_ioctrlcall(sockfd, (u_long *)"SIOCGIFFLAGS", &ifr);

  /*
  * Expected in <net/if.h> according to
  * "UNIX Network Programming".
  */
  #ifdef ifr_flags
  # define IRFFLAGS       ifr_flags
  #else   /* Present on kFreeBSD */
  # define IRFFLAGS       ifr_flagshigh
  #endif

  // If interface is down, bring it up
  if (ifr.IRFFLAGS | ~(IFF_UP)) {
    ifr.IRFFLAGS |= IFF_UP;
    generic_ioctrlcall(sockfd, (u_long *)"SIOCSIFFLAGS", &ifr);
  }

  // Set route
  set_route(sockfd, gateway_addr,  &sin);

  memcpy(&ifr.ifr_addr, &sin, sizeof(struct sockaddr)); 

  // Set interface address
  if (ioctl(sockfd, SIOCSIFADDR, &ifr) < 0) {
    fprintf(stderr, "Cannot set IP address. ");
    perror(ifr.ifr_name);
    return -1;
  }             
  #undef IRFFLAGS

  return 0;
}

Answer 1

这里有很多问题。困难的最直接原因是ioctl的第二个参数是一个长整数，而不是指向字符串的指针。

generic_ioctrlcall(sockfd, (u_long *)"SIOCGIFFLAGS", &ifr);

不会创建指向SIOCGIFFLAGS的整数值的指针，这可能是您的意图。其他问题：

（1）由于ioctl采用长整数作为第二个参数，为什么要将指针传递给整数呢？只需将整数作为参数传递。

（2）如果做希望传递指向整数的指针，则需要执行以下操作：

u_long ioctl_arg = SIOCGIFFLAGS;
generic_ioctrlcall(sockfd, &ioctl_arg, &ifr);

（3）只需通过强制转换就不能将指向整数转换为整数。您需要取消引用指针。那就是：

Change:
    if (ioctl(sockfd, (long unsigned int)flags, &ifr) < 0) {
to:
    if (ioctl(sockfd, *flags, &ifr) < 0) {

（4）在ioctl失败时，您应该在失败后打印errno（或者，更好的是，它的文本翻译），以便弄清楚出了什么问题：

if (ioctl(sockfd, *flags, &ifr) < 0) {
    fprintf(stderr, "ioctl: flags %ld errno %d/%s\n", *flags, errno, strerror(errno));

顺便说一句，您的errno来电中的generic_ioctl似乎不太可能ENODEV（没有这样的设备） - 正如您的标题所暗示的那样。似乎errno更可能是EINVAL，因为指针到字符串形成有效ioctl整数参数的可能性非常小。

IOCTL返回No such device

1 个答案: