Question

我写了一些代码，但是没有用。我需要将传入的数据包发送到另一台主机（2.2.2.1）。此代码接收数据包但无法发送。我做错了什么？我正在接收数据包，而不是将数据包复制到另一个数组，而不是尝试使用sendto（）函数发送它。

#include <unistd.h> 
#include <arpa/inet.h>
#include <stdio.h> 
#include <string.h> 
#include <sys/socket.h> 
#include <netinet/ip.h> 
#include <netinet/tcp.h>   

#define PCKT_LEN 8192

int main(){
    int s, s1;
    char buffer[PCKT_LEN];
    struct sockaddr saddr;
    struct sockaddr_in daddr;
    memset(buffer, 0, PCKT_LEN);

    s = socket(AF_INET, SOCK_RAW, IPPROTO_TCP);
    if(s < 0){
        printf("socket() error");
        return -1;
    }
    int one = 1;
    const int *val = &one;
    if(setsockopt (s, IPPROTO_IP, IP_HDRINCL, val, sizeof(one))<0){
        printf("setsock");
        return -1;
    }
    s1 = socket(AF_INET, SOCK_RAW, IPPROTO_TCP);
    if(s1 < 0){
        printf("sock2 error");
        return -1;
    }
    if(setsockopt (s1, IPPROTO_IP, IP_HDRINCL, val, sizeof(one)) < 0){
        printf("setsock");
        return -1;
    }
    int saddr_size = sizeof(saddr);
    int daddr_size = sizeof(daddr); 
    int header_size = sizeof(struct iphdr) + sizeof(struct tcphdr);
    unsigned int count;

    daddr.sin_family = AF_INET;
    daddr.sin_port = htons(1234);
    daddr.sin_addr.s_addr = inet_addr ("2.2.2.1");

    char *datagram, *data;
    datagram = (char*)calloc(4096, sizeof(char));
    struct iphdr *iph;
    struct tcphdr *tcph;
    int counter = 0;
    while(counter != 3){
        counter++;
        if(recvfrom(s, buffer, PCKT_LEN , 0, &saddr, &saddr_size) < 0){
            printf("recvfrom() error");
            return -1;
        }
        else{
            int i = header_size;
            int packet_len = ntohs(iph->tot_len);           
            memset(datagram, 0 ,4096);
            iph = (struct iphdr*) buffer;
            tcph = (struct tcphdr*) (buffer + sizeof(struct ip));
            data = buffer + sizeof(struct iphdr) + sizeof(struct tcphdr);

            i = 0;
            for(; i < packet_len-sizeof(struct iphdr)-sizeof(struct tcphdr); i++)
                printf("%c", data[i]);
                 printf("\n");

            memcpy(datagram, iph, sizeof(struct iphdr));
            memcpy(datagram+sizeof(struct iphdr), tcph , sizeof(struct tcphdr));
            memcpy(datagram+sizeof(struct iphdr)+sizeof(struct tcphdr), data,packet_len-sizeof(struct iphdr)-sizeof(struct tcphdr));

            iph->daddr = daddr.sin_addr.s_addr;

            if (sendto (s, datagram, packet_len, 0, &daddr, &daddr_size) < 0){
                printf("sendto() error");
                return -1;
            }
        }
    }
    close(s);
    close(s1);
    free(datagram);
    return 0;
}

Answer 1

您没有包含stdlib.h，因此calloc()被隐式声明，因此编译器假定它返回int，这将导致许多问题。

此外，请不要使用calloc()，除非您确实要将数据初始化为0，因为如果您遇到数据初始化问题，那么不会发生任何不良事件，你永远不会知道。使用malloc()除非您稍后转到memset(pointer, 0, size);，这也不错，在我的情况下，我总是使用malloc()。
您没有包含unistd.h，因此您也错过了close()的声明。这个问题并没有造成问题，但仅仅是因为它会返回int。

您的sendto()来电也是错误的，最后一个参数应该是socklen_t类型而您传递的是int *，所以请按照这样的方式修复

sendto (s, datagram, packet_len, 0, (struct sockaddr *)&daddr, daddr_size) < 0
/*                                                            ^ no & here */

为避免在您要求编译器发出警告时出现的某些警告，您需要将这些警告声明为socklen_t
```
socklen_t saddr_size = sizeof(saddr);
socklen_t daddr_size = sizeof(daddr);
```
并且您还需要在(struct sockaddr *)&daddr上面看到的演员阵容。
最重要的问题是您尝试在不初始化的情况下访问iph
```
size_t packet_len = ntohs(iph->tot_len);
```
这应该在iph = (struct iphdr *)buffer;之后。

原因恰恰在于您的代码非常混乱，很难注意到这一点。

您可以通过使用编译器警告来避免所有这些，如下所示

gcc -Wall -Wextra -Werror sourcefile.c -o executable

注意：更好地整理您的代码，很难遵循，如果其他人不得不像我刚才那样审核它，那也不好对你而言并不好，因为当你从几个月开始看它时，你会对写这个烂摊子的人非常生气。

如何使用C中的原始套接字发送收到的数据包？

1 个答案: