【发布时间】：2019-09-18 06:20:16
【问题描述】：
我用 C 语言编写了一些代码来读取 CAN 总线数据。当我读取 11 位 CAN ID 时，一切正常。一旦我尝试读取 29 位 ID，它就会错误地显示 ID。
示例：
接收 29 位 ID 的消息：
0x01F0A020
然后打印出来
printf("%X\n", frame.can_id);
它打印 81F0A020。
11 位 ID 消息
0x7DF
然后打印出来
printf("%X\n", frame.can_id);
它可以正确打印7DF。
为什么会这样？
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
#include <net/if.h>
#include <fcntl.h>
#include <inttypes.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <sys/ioctl.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <linux/can.h>
#include <linux/can/raw.h>
#define MAX_DATA_LEN 8
#define MAX_FIELDS 23
#define MAX_FIELD_LEN 64
#include <limits.h>
char data_str[MAX_FIELDS][MAX_FIELD_LEN];
int i;
int
main(void)
{
int s;
int nbytes;
struct sockaddr_can addr;
struct can_frame frame;
unsigned short data[MAX_FIELDS];
int sockfd = 0;
int bcast = 1;
struct sockaddr_in src_addr;
struct sockaddr_in dst_addr;
int numbytes;
int fa;
struct ifreq ifr;
fa = socket(AF_INET, SOCK_DGRAM, 0);
ifr.ifr_addr.sa_family = AF_INET;
strncpy(ifr.ifr_name, "eth0", IFNAMSIZ-1);
ioctl(fa, SIOCGIFHWADDR, &ifr);
close(fa);
//
if((sockfd = socket(PF_INET, SOCK_DGRAM, 0)) == -1)
{
perror("Udp sockfd create failed");
exit(1);
}
//Enabled broadcast mode for udp
if((setsockopt(sockfd, SOL_SOCKET, SO_BROADCAST,
&bcast, sizeof bcast)) == -1)
{
perror("setsockopt failed for broadcast mode ");
exit(1);
}
src_addr.sin_family = AF_INET;
src_addr.sin_port = htons(8888);
src_addr.sin_addr.s_addr = INADDR_ANY;
memset(src_addr.sin_zero, '\0', sizeof src_addr.sin_zero);
if(bind(sockfd, (struct sockaddr*) &src_addr, sizeof src_addr) == -1)
{
perror("bind");
exit(1);
}
dst_addr.sin_family = AF_INET;
dst_addr.sin_port = htons(45454);
dst_addr.sin_addr.s_addr = inet_addr("127.0.0.1");
memset(dst_addr.sin_zero, '\0', sizeof dst_addr.sin_zero);
if((s = socket(PF_CAN, SOCK_RAW, CAN_RAW)) < 0) {
perror("Error while opening socket");
return -1;
}
addr.can_family = AF_CAN;
addr.can_ifindex = 0;
if(bind(s, (struct sockaddr *)&addr, sizeof(addr)) < 0) {
perror("Error in socket bind");
return -2;
}
//struct can_frame frame;
while(1)
{
nbytes = read(s, &frame, sizeof(struct can_frame));
if (nbytes < 0) {
perror("can raw socket read");
return 1;
}
/* Paranoid check ... */
if (nbytes < sizeof(struct can_frame)) {
fprintf(stderr, "read: incomplete CAN frame\n");
return 1;
}
// Print the received CAN ID
printf("%X\n", frame.can_id);
}
return 0;
}
【问题讨论】：
-
为什么会有可疑的
(struct sockaddr*)演员表？ -
@Bodo 不同结构类型之间的转换会调用未定义的行为，很可能是一个错误。它们是真正兼容的类型吗？如果是这样，为什么要使用两种不同的类型。要么是代码气味，要么是库气味。
-
浏览了 Github。 sys/socket.h 中的
struct sockaddr与struct sockaddr_can一点也不兼容，因此它似乎是一个库错误。它们是不兼容的类型，也是严格的别名违规。 -
@Bodo 唯一重要的是该结构是否在任何地方被取消引用为
struct sockaddr- 如果是这样，那就是一个错误。否则只是混乱的 API 设计。