0

我已经学习了大约 2 周的 C 语言，并决定尝试使用一个简单的端口扫描仪。我可以毫无错误地编译代码，但是每当我尝试执行它时，我都会遇到分段错误。

#include<stdio.h>
#include<sys/socket.h>
#include<errno.h>
#include<netdb.h>
#include<string.h>
#include<stdlib.h>

int scanPort(char hostaddr[], int port) {
    struct hostent *host;
    int err, i, sock;
    struct sockaddr_in sa;

    strncpy((char*)&sa , "" , sizeof sa);
    sa.sin_family = AF_INET;

    if (isdigit(hostaddr[0])) {
        sa.sin_addr.s_addr = inet_addr(hostaddr);
    } else if ((host = gethostbyname(hostaddr)) != 0) {
        strncpy((char*)&sa.sin_addr , (char*)host->h_addr , sizeof sa.sin_addr);
    } else {
        printf("\n[!] Failed to resolve host!\n");
        exit(1);
    }

    sa.sin_port = htons(port);
    sock = socket(AF_INET, SOCK_STREAM, 0);

    if (sock < 0) {
        printf("\n[!] Failed to create socket!\n");
        exit(1);
    }

    err = connect(sock,  (struct sockaddr*)&sa , sizeof sa);

    close(sock);

    if (err < 0) {
        return 0;
    } else {
        return 1;
    }
}

int main(int argc, char *argv[]) {
    if (argc != 4) {
        printf("usage: ./portscan [TARGET] [START PORT] [STOP PORT]\n");
        exit(0);
    }

    char host[20];
    strcpy(host, argv[1]);
    int beginport;
    int endport;

    if (isdigit(argv[2]) && isdigit(argv[3])) {
        beginport = atoi(argv[2]);
        endport = atoi(argv[3]);
    } else {
        printf("[!] Invalid port range given\n");
    }

    printf("[*] Beginning Scan...\n\n");

    int i = beginport;

    for (i; i<=endport; i++) {
        if (scanPort(host, i)) {
            printf("Port %d: Open\n", i);
        }
    }

    printf("\n[*] Scan complete!");
    return 0;
}

我知道代码中有一些不正确/不安全的函数使用，但我只是想让这个程序正常运行。这只是一个测试，不是一个实际使用的程序。

这是编译和执行的屏幕截图，但我认为它不会有太大帮助： 在此处输入图像描述

更新：我将参数传递给它，仍然得到分段错误： 在此处输入图像描述

更新 2：我添加了几行来评估 argc

感谢您的时间。

-默认

4

1 回答 1

2

崩溃发生在这里：

if (isdigit(argv[2]) && isdigit(argv[3])) {

该isdigit函数需要一个int（实际上是 achar转换为 an int），但您传入 a char *。这会调用未定义的行为，在这种情况下（对你来说很幸运）表现为崩溃。

您可能想要检查每个字符串的第一个字符，因此传入argv[2][0]和传递argv[3][0]给此函数。

此外，这是不正确的：

strncpy((char*)&sa.sin_addr , (char*)host->h_addr , sizeof sa.sin_addr);

该strncpy函数用于复制字符串，而不是任意字节。如果正在复制的字节中有一个空字节，则不会复制该字节之后的其他字节。改为使用memcpy，它复制确切数量的字节：

memcpy((char*)&sa.sin_addr , (char*)host->h_addr , sizeof sa.sin_addr);
于 2017-03-14T23:44:55.730 回答