Segmentierungsfehler (Core Dumped) von Befehlszeilenargument

Question

Ich versuche, Code aus dem Terminal mithilfe von Argumenten auszuführen. Hier ist mein Code:

int PORT_NUM = 0; 
int BYTES_TO_SEND = 0; 
char* LocalAddresss = ""; 


char* concat(char *s1, char *s2) 
{ 
    char *result = malloc(strlen(s1)+strlen(s2)+1); 
    strcpy(result, s1); 
    strcat(result, s2); 
    return result; 
} 

void error(const char *msg) 
{ 
    perror(msg); 
    exit(0); 
} 

int main(int argc, char *argv[]) 
{ 
    int sockfd, portno, n; 
    struct sockaddr_in serv_addr; 
    struct hostent *server; 

    char buffer[256]; 
    char data[BYTES_TO_SEND]; 

    LocalAddresss = concat("192.168.1.",argv[1]); 
    PORT_NUM = atoi(argv[2]); 
    BYTES_TO_SEND = atoi(argv[3]); 

    fflush(stdout); 

    for(int i = 0; i < BYTES_TO_SEND; i++){ 
     data[i] = 'Z'; 
    } 

    char* message_to_send = data; 

    portno = PORT_NUM; 
    sockfd = socket(AF_INET, SOCK_STREAM, 0); 
    if (sockfd < 0) 
     error("ERROR opening socket"); 

    server = gethostbyname(LocalAddresss); 
    if (server == NULL) { 
     fprintf(stderr,"ERROR, no such host\n"); 
     exit(0); 
    } 
    bzero((char *) &serv_addr, sizeof(serv_addr)); 
    serv_addr.sin_family = AF_INET; 
    bcopy((char *)server->h_addr, 
     (char *)&serv_addr.sin_addr.s_addr, 
     server->h_length); 
    serv_addr.sin_port = htons(portno); 

    if (connect(sockfd,(struct sockaddr *) &serv_addr,sizeof(serv_addr)) < 0) 
     error("ERROR connecting"); 

    while(1){ 
     n = write(sockfd,message_to_send,strlen(message_to_send)); 
    } 

    close(sockfd); 
    return 0; 
} 

Mein Problem ist dies. Folgendes funktioniert: ./TCP_client 16 2000 100 in der Befehlszeile.

Dies gilt nicht: ./TCP_client 16 2000 3000

ich "Segmentation fault (core dumped)"

Gibt es Beschränkungen für die Anzahl der gesamten Byte in Befehlszeilenargumente oder so etwas? Wie kann ich dieses Problem beheben?

Nach der Ausführung mit GDB heißt es "Programm empfangen Signal SIGBUS, Bus error.0x0000000100000ccf in main()" Nicht wirklich sicher, was das ist.

Answer 1

Wenn ich Sie wäre, würde ich aus der string.h-Bibliothek die Funktion strcat() verwenden.

#include <string.h> 
... 
char LocalAddresss[15] = "192.168.1."; 
strcat(LocalAddress, argv[1]); 

Answer 2

dies fehlschlägt weil Sie Daten mit einer Größe von 0

int BYTES_TO_SEND = 0; 
... 
char data[BYTES_TO_SEND]; 
''' 
LocalAddresss = concat("192.168.1.",argv[1]); 
PORT_NUM = atoi(argv[2]); 
BYTES_TO_SEND = atoi(argv[3]); 

Sie die Linien um

int BYTES_TO_SEND = 0; 
... 
LocalAddresss = concat("192.168.1.",argv[1]); 
PORT_NUM = atoi(argv[2]); 
BYTES_TO_SEND = atoi(argv[3]); 
char data[BYTES_TO_SEND]; 
''' 

bewegen müssen deklarieren, aber dies ist ein schlechter Plan - ein großes Putting Puffer wie das auf dem Stapel, besser zu malloc es. Ihr Beispiel wird funktionieren, aber es ist kein gutes Design