-2

私はCでWebサーバーを作成しようとしていますが、現在コードがセグフォールトしているのですが、その理由がわかりません。それは私のstrcatsと関係があるようですが、それは私が得ることができた限りです。コードとgdb出力を投稿しました。どんな助けでも大歓迎です。

***コード** *

    /*  web-server.c    */

    #include <arpa/inet.h>
    #include <netdb.h>
    #include <sys/types.h>
    #include <sys/socket.h>
    #include <stdio.h>
    #include <stdlib.h>
    #include <string.h>
    #include <unistd.h>

    #define BUFLEN 1500
    #define BACKLOG 10

    static int server_socket(int port) {
int fd;
struct sockaddr_in servaddr;

if ((fd = socket(AF_INET, SOCK_STREAM, 0)) == -1){
    perror("Unable to create socket");
    return -1;
}
printf("created socket\n");
servaddr.sin_family = AF_INET;
servaddr.sin_addr.s_addr = INADDR_ANY;
servaddr.sin_port = htons(port);

if (bind(fd, (struct sockaddr *)&servaddr, sizeof(servaddr)) == -1) {
    perror("Unable to bind to port");
    return -1;
}
printf("binding completed\n");
if (listen(fd, BACKLOG) == -1){
    perror("Unable to listen for connections");
    return -1;
}
printf("socket setup\n");
return fd;

    }

    static char *html_request(char *request) {
char request_code[3], url[BUFLEN], http_code[BUFLEN];
printf(request);
printf("\n");
sscanf(request, "%3s %s %s", request_code, url, http_code);
return url;
    }


    int main() {
int connfd, servfd, rlen, i, eof;
int content_length = 0;
char buf[BUFLEN];
struct sockaddr_in saddr;
char *response;
char *read_buffer = malloc(1);
    char *response200 = "HTTP/1.1 200 OK\nContent-Type: text/html\nConnection: close\nConnection-Length: ";
char *response404 = "HTTP/1.1 404 Not Found\nContent-Type: text/html\nConnection: close\nConnection-Length: ";
FILE *in;
char buffer[1500], content_buffer[5];
char *headers = malloc(BUFLEN);
char *output = malloc(BUFLEN);
socklen_t saddr_len = sizeof(saddr);
servfd = server_socket(8080);
printf("before connection\n");
if ((connfd = accept(servfd, (struct sockaddr *)&saddr, &saddr_len)) == -1) {
    perror("Unable to accept connection");
    return -1;
}
else {
printf("Accept connection\n");

}
if ((rlen = read(connfd, buf, BUFLEN)) > 0) {
    realloc(read_buffer,rlen);
    for (i = 0; i < rlen; i++){
        printf("%c", buf[i]);
        read_buffer[i] = buf[i];
    }
    printf("\n");

}
printf("IMMA BEAST!\n");
response = html_request(read_buffer);
printf(response);

if (strcmp(response, "/index.html") == 0){
    printf("\nMatches!");
    in = fopen("index.txt", "r");
    if (in == NULL){
        printf("Read fail!");
    }
    while ((eof = fread(buffer, 1, 296, in )) > 0){
        content_length = content_length + 1;
    }
    snprintf(content_buffer, 5, "%d" ,content_length);
    headers = strcat(response200, content_buffer);
}
else{
    printf("\nFails!");
    in = fopen("index-error.txt", "r");
    if (in == NULL){
        printf("Read fail!");
    }
    while ((eof = fread(buffer, 1, 300, in )) > 0){
        content_length = content_length + 1;
    }
    snprintf(content_buffer, 5, "%d" ,content_length);
    headers = strcat(response404, content_buffer);
}
fclose(in);
output = strcat(headers, buffer);
write(connfd, output, BUFLEN);
close(connfd);
free(read_buffer);
free(headers);
free(output);

return 0;

}

*GDB出力*

Program received signal SIGSEGV, Segmentation fault.
0x0000003d4647eff0 in strcat () from /lib64/libc.so.6
(gdb) where
#0  0x0000003d4647eff0 in strcat () from /lib64/libc.so.6
#1  0x0000000000400ecd in main ()
(gdb) n
Single stepping until exit from function strcat,
which has no line number information.

Program terminated with signal SIGSEGV, Segmentation fault.
The program no longer exists.
4

2 に答える 2

8

これはまったくOKではありません:

strcat("HTTP/1.1 404 Not Found\nContent-Type: text/html\nConnection: close\nConnection-Length: ", content_buffer);

strcat()2番目の引数を最初の引数に追加します。最初の引数にはリテラル文字列があり、変更してはなりません。char[]Cがこれをコンパイルできるようにする唯一の理由は、古いコードが。ではなく型のリテラル文字列に依存しているという不幸な事実ですconst char[]

于 2013-02-26T13:03:21.320 に答える
0

http://www.manpagez.com/man/3/strcat/

最初の文字列を適切なサイズ(最初の文字列の長さ+2番目の文字列の長さ+1)でmallocする必要があります。

http://www.manpagez.com/man/3/malloc/

何かのようなもの:

char *str;
int len = (strlen("HTTP/1.1 200 OK\nContent-Type: text/html\nConnection: close\nConnection-Length: ") + strlen(content_buffer));

str = malloc((len + 1) * sizeof(char));
bzero(str, len);
strcpy(str, "HTTP/1.1 200 OK\nContent-Type: text/html\nConnection: close\nConnection-Length: ");
strcat(str, content_buffer);
str[len] = 0;
于 2013-02-26T13:07:44.910 に答える