C-Programm mit fork() und while(1)

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • C-Programm mit fork() und while(1)

    Hallo an Alle,

    ich würde mich sehr über einen Tip in folgender Sache freuen:

    Habe ein kleines C-Programm unter Linux, dass einmal Kindprozesse mit fork() erzeugen soll und dass sich nicht beenden soll und bereit sein für weitere Anfragen. Das ständige bereit sein habe ich mit while(1) gelöst. Scheinbar darf fork nicht in der while(1) Schleife stehen. Nur wie könnte man das sonst lösen?

    Vielen Dank für einen Tip.

    Viele Grüße

    Walter

    Code:


    /*****************************************************************/
    /* Dateiname : ServerDateiSenden.c */
    /* Beschreibung: Server-Programm */
    /* Analysiert einen HTTP-GET-Befehl und versucht */
    /* die angeforderte Datei zu senden */
    /*****************************************************************/

    #include <sys/types.h>
    #include <sys/socket.h>
    #include <stdio.h>
    #include <netinet/in.h>
    #include <arpa/inet.h>
    #include <string.h>

    int main()
    {
    int server_socket, client_socket;
    int anzahl, laenge, childpid;
    struct sockaddr_in serverinfo, clientinfo;
    char zeichen;
    char empfangen[1000];
    char *position1, *position2;
    char dateiname[100];
    char text_http_ok[] = "HTTP/1.0 200 OK\r\n\r\n";
    char text_html_anfang[] = "<HTML><BODY>";
    char text_html_ende[] = "</BODY></HTML>";
    FILE *datei;

    server_socket = socket(AF_INET, SOCK_STREAM, 0);

    serverinfo.sin_family = AF_INET;
    serverinfo.sin_addr.s_addr = htonl(INADDR_ANY);
    serverinfo.sin_port = htons(5000);
    laenge = sizeof(serverinfo);

    bind(server_socket, (struct sockaddr *)&serverinfo, laenge);
    listen(server_socket, 3);

    while (1)
    {
    printf("\n Der Server wartet...");
    fflush(stdout);

    client_socket = accept(server_socket, (struct sockaddr *)&clientinfo, &laenge);

    printf("\n Verbindung mit %s",inet_ntoa(clientinfo.sin_addr));
    childpid = fork();
    if (childpid ==0)
    {
    close(server_socket);
    }


    anzahl = read(client_socket,empfangen,sizeof(empfangen));
    empfangen[anzahl]=0;
    write(client_socket,text_http_ok,strlen(text_http_ok));

    if (position1 = strstr(empfangen,"GET"))
    {
    if (position2 = strstr(empfangen,"HTTP"))
    {
    laenge = position2 - position1 - 6;
    strncpy(dateiname,position1+5,laenge);
    dateiname[laenge]=0;
    printf("\n GET Befehl für Datei %s gefunden",dateiname);
    datei = fopen(dateiname,"rt");
    if (datei==NULL)
    {
    printf("\n Datei existiert nicht");
    write(client_socket,text_html_anfang,strlen(text_html_anfang));
    write(client_socket,"<br>Datei existiert nicht",25);
    write(client_socket,text_html_ende,strlen(text_html_ende));
    }
    else
    {
    printf("\n Sende Datei %s",dateiname);

    do
    {
    zeichen = fgetc(datei);
    if (!feof(datei))
    {
    write(client_socket,&zeichen,1);
    }
    } while (!feof(datei));

    fclose(datei);
    }
    }
    else
    {
    printf("\n Der HTTP-Parser hat keinen HTTP-Befehl gefunden");
    write(client_socket,text_html_anfang,strlen(text_html_anfang));
    write(client_socket,"kein HTTP-Befehl gefunden",25);
    write(client_socket,text_html_ende,strlen(text_html_ende));
    }
    }
    else
    {
    printf("\n Der HTTP-Parser hat keinen GET-Befehl gefunden");
    write(client_socket,text_html_anfang,strlen(text_html_anfang));
    write(client_socket,"kein GET-Befehl gefunden",24);
    write(client_socket,text_html_ende,strlen(text_html_ende));
    }
    printf("\n");
    close(client_socket);
    }
    }
    }

  • Kommt eine Fehlermeldung oder warum glaubst du, dass fork() nicht innerhalb der while-Schleife vorkommen darf?
    Ich konnte deinen Code bei mir problemlos kompilieren und ausführen (ohne etwas über den Port zu empfangen).
    Nur das letzte } ist zu viel und ich würde (als Tipp) noch unistd.h einbinden, sonst fehlen dem Compiler die Signaturen für read(), fork(), etc. (mit gcc -Wall einsehbar).
    Beim nächsten Mal bitte Code-Tags verwenden, damit die Einrückungen erhalten bleiben.
    Hier nochmals sauber formatiert:

    Quellcode

    1. /*****************************************************************/
    2. /* Dateiname : ServerDateiSenden.c */
    3. /* Beschreibung: Server-Programm */
    4. /* Analysiert einen HTTP-GET-Befehl und versucht */
    5. /* die angeforderte Datei zu senden */
    6. /*****************************************************************/
    8. #include <sys/types.h>
    9. #include <sys/socket.h>
    10. #include <unistd.h>
    11. #include <stdio.h>
    12. #include <netinet/in.h>
    13. #include <arpa/inet.h>
    14. #include <string.h>
    16. int main()
    17. {
    18. int server_socket, client_socket;
    19. int anzahl, laenge, childpid;
    20. struct sockaddr_in serverinfo, clientinfo;
    21. char zeichen;
    22. char empfangen[1000];
    23. char *position1, *position2;
    24. char dateiname[100];
    25. char text_http_ok[] = "HTTP/1.0 200 OK\r\n\r\n";
    26. char text_html_anfang[] = "<HTML><BODY>";
    27. char text_html_ende[] = "</BODY></HTML>";
    28. FILE *datei;
    30. server_socket = socket(AF_INET, SOCK_STREAM, 0);
    32. serverinfo.sin_family = AF_INET;
    33. serverinfo.sin_addr.s_addr = htonl(INADDR_ANY);
    34. serverinfo.sin_port = htons(5000);
    35. laenge = sizeof(serverinfo);
    37. bind(server_socket, (struct sockaddr *)&serverinfo, laenge);
    38. listen(server_socket, 3);
    40. while (1)
    41. {
    42. printf("\n Der Server wartet...");
    43. fflush(stdout);
    45. client_socket = accept(server_socket, (struct sockaddr *)&clientinfo, &laenge);
    47. printf("\n Verbindung mit %s",inet_ntoa(clientinfo.sin_addr));
    48. childpid = fork();
    49. if (childpid ==0)
    50. {
    51. close(server_socket);
    52. }
    55. anzahl = read(client_socket,empfangen,sizeof(empfangen));
    56. empfangen[anzahl]=0;
    57. write(client_socket,text_http_ok,strlen(text_http_ok));
    59. if (position1 = strstr(empfangen,"GET"))
    60. {
    61. if (position2 = strstr(empfangen,"HTTP"))
    62. {
    63. laenge = position2 - position1 - 6;
    64. strncpy(dateiname,position1+5,laenge);
    65. dateiname[laenge]=0;
    66. printf("\n GET Befehl für Datei %s gefunden",dateiname);
    67. datei = fopen(dateiname,"rt");
    68. if (datei==NULL)
    69. {
    70. printf("\n Datei existiert nicht");
    71. write(client_socket,text_html_anfang,strlen(text_html_anfang));
    72. write(client_socket,"<br>Datei existiert nicht",25);
    73. write(client_socket,text_html_ende,strlen(text_html_ende));
    74. }
    75. else
    76. {
    77. printf("\n Sende Datei %s",dateiname);
    79. do
    80. {
    81. zeichen = fgetc(datei);
    82. if (!feof(datei))
    83. {
    84. write(client_socket,&zeichen,1);
    85. }
    86. } while (!feof(datei));
    88. fclose(datei);
    89. }
    90. }
    91. else
    92. {
    93. printf("\n Der HTTP-Parser hat keinen HTTP-Befehl gefunden");
    94. write(client_socket,text_html_anfang,strlen(text_html_anfang));
    95. write(client_socket,"kein HTTP-Befehl gefunden",25);
    96. write(client_socket,text_html_ende,strlen(text_html_ende));
    97. }
    98. }
    99. else
    100. {
    101. printf("\n Der HTTP-Parser hat keinen GET-Befehl gefunden");
    102. write(client_socket,text_html_anfang,strlen(text_html_anfang));
    103. write(client_socket,"kein GET-Befehl gefunden",24);
    104. write(client_socket,text_html_ende,strlen(text_html_ende));
    105. }
    106. printf("\n");
    107. close(client_socket);
    108. }
    109. }
    Alles anzeigen
    Open Source --> Programmieren aus Leidenschaft :!:

    Ich stehe weder für privaten Support per PM noch über einen IM zur Verfügung. Danke.

  • while(1), fork()

    Hi Deadman44, da ich kein Programmierer bin, war mir nicht bewusst, dass ich den Code da so reinklatsche und Durch Dein formatieren ist mir bewusst geworden, wie schön der jetzt zu Lesen ist. Finde das sehr nett von Dir. Habe in Zeile 34 von Deinem Code als Port 80 eingetragen und das Programm übersetzt. Dabei passiert folgendes:


    Programm gestartet:

    root@j23764:/home/se# ./a.out

    Der Server wartet...

    Client fordert die Datei bla an:
    root@j23764:~# wget 127.0.0.1/bla

    Jetzt meldet der Client, dass er mit 127.0.0.1:80 connected ist und versucht die Datei zu speichern. Beim Server steht folgendes:

    Verbindung mit 127.0.0.1
    Get Befehl für Datei bla gefunden
    Sende Datei bla

    Es passiert aber nichts. Erst nachdem ich beim Client wget mit STRG+C abbreche, wird die Datei runtergeladen. Ohne fork() arbeitet das Script sauber. Vielleicht hättest Du ja noch einen Tip für mich.

    Viele Grüße

  • Ich habe nun den/die Fehler gefunden und den Code angepasst.

    Quellcode

    1. /*****************************************************************/
    2. /* Dateiname : ServerDateiSenden.c */
    3. /* Beschreibung: Server-Programm */
    4. /* Analysiert einen HTTP-GET-Befehl und versucht */
    5. /* die angeforderte Datei zu senden */
    6. /*****************************************************************/
    8. #include <sys/types.h>
    9. #include <sys/socket.h>
    10. #include <unistd.h>
    11. #include <stdio.h>
    12. #include <stdlib.h> // Für EXIT_SUCCESS eingebunden
    13. #include <netinet/in.h>
    14. #include <arpa/inet.h>
    15. #include <string.h>
    17. int main()
    18. {
    19. int server_socket, client_socket;
    20. int anzahl, laenge, childpid;
    21. struct sockaddr_in serverinfo, clientinfo;
    22. char zeichen;
    23. char empfangen[1000];
    24. char *position1, *position2;
    25. char dateiname[100];
    26. char text_http_ok[] = "HTTP/1.0 200 OK\r\n\r\n";
    27. char text_html_anfang[] = "<HTML><BODY>";
    28. char text_html_ende[] = "</BODY></HTML>";
    29. FILE *datei;
    31. server_socket = socket(AF_INET, SOCK_STREAM, 0);
    33. serverinfo.sin_family = AF_INET;
    34. serverinfo.sin_addr.s_addr = htonl(INADDR_ANY);
    35. serverinfo.sin_port = htons(80);
    36. laenge = sizeof(serverinfo);
    38. bind(server_socket, (struct sockaddr *)&serverinfo, laenge);
    39. listen(server_socket, 3);
    41. while (1)
    42. {
    43. printf("\n Der Server wartet...");
    44. fflush(stdout);
    46. client_socket = accept(server_socket, (struct sockaddr *)&clientinfo, &laenge);
    48. printf("\n Verbindung mit %s",inet_ntoa(clientinfo.sin_addr));
    49. childpid = fork();
    50. if (childpid != 0) // Falls Serverprozess, also Parent, wird die Schleife wiederholt und auf dem Port gelauscht
    51. {
    52. close(client_socket);
    53. }
    54. else // Code für Kindprozess, also für das Bearbeiten der Abfrage
    55. {
    56. close(server_socket);
    57. anzahl = read(client_socket,empfangen,sizeof(empfangen));
    58. empfangen[anzahl]=0;
    59. write(client_socket,text_http_ok,strlen(text_http_ok));
    61. if (position1 = strstr(empfangen,"GET"))
    62. {
    63. if (position2 = strstr(empfangen,"HTTP"))
    64. {
    65. laenge = position2 - position1 - 6;
    66. strncpy(dateiname,position1+5,laenge);
    67. dateiname[laenge]=0;
    68. printf("\n GET Befehl für Datei %s gefunden",dateiname);
    69. datei = fopen(dateiname,"rt");
    70. if (datei==NULL)
    71. {
    72. printf("\n Datei existiert nicht");
    73. write(client_socket,text_html_anfang,strlen(text_html_anfang));
    74. write(client_socket,"<br>Datei existiert nicht",25);
    75. write(client_socket,text_html_ende,strlen(text_html_ende));
    76. }
    77. else
    78. {
    79. printf("\n Sende Datei %s",dateiname);
    81. do
    82. {
    83. zeichen = fgetc(datei);
    84. if (!feof(datei))
    85. {
    86. write(client_socket,&zeichen,1);
    87. }
    88. } while (!feof(datei));
    90. fclose(datei);
    91. }
    92. }
    93. else
    94. {
    95. printf("\n Der HTTP-Parser hat keinen HTTP-Befehl gefunden");
    96. write(client_socket,text_html_anfang,strlen(text_html_anfang));
    97. write(client_socket,"kein HTTP-Befehl gefunden",25);
    98. write(client_socket,text_html_ende,strlen(text_html_ende));
    99. }
    100. }
    101. else
    102. {
    103. printf("\n Der HTTP-Parser hat keinen GET-Befehl gefunden");
    104. write(client_socket,text_html_anfang,strlen(text_html_anfang));
    105. write(client_socket,"kein GET-Befehl gefunden",24);
    106. write(client_socket,text_html_ende,strlen(text_html_ende));
    107. }
    108. printf("\n");
    109. close(client_socket);
    110. return EXIT_SUCCESS; // Kindprozess beenden, da Anfrage bearbeitet wurde
    111. }
    112. }
    113. }
    Alles anzeigen

    Hauptsächlich lag es daran, dass du beim Forken nicht ausreichend zwischen Parent und Child unterschieden hast. Child muss z. Bsp. am Ende terminieren und Parent sollte nicht noch durch den Child-Code laufen. Habe meine Änderungen kurz kommentiert, damit du sie nachvollziehen kannst.
    Open Source --> Programmieren aus Leidenschaft :!:

    Ich stehe weder für privaten Support per PM noch über einen IM zur Verfügung. Danke.

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von Deadman44 ()