Dlaczego pisze tylko do ostniego pliku?

Mam demon samodzielny bez nohup,
wołam "program 12"

#include <stdio.h>
#include <iostream>
#include <fstream>
#include <thread>
#include <unistd.h>
#include <fcntl.h>

using namespace std;

int main(int argc, char * argv[])
{
    char cwd[1024];
    string sCWD;
    if (getcwd(cwd, sizeof(cwd)) != NULL)
       sCWD = cwd; else sCWD = "/";
    if (sCWD.back()!='/') sCWD+="/";
    const int autorestart = 5;
    if (argc < 2)
        exit(1);

    int numsec = atoi(argv[1]);
    if (numsec==0) exit(1);


    ofstream file;
    file.open(sCWD+"stdout.out"+to_string(numsec),ios::out);
    file.close();
    file.open(sCWD+"stderr.out"+to_string(numsec),ios::out);
    file.close();

    int pid = fork();
    if (pid < 0)
        exit(1); // fork failed
    if (pid > 0)
        exit(0); // still in parent process, so quit


    // in child process, obtain new session and process group at the same time
    setsid();



    {

    int devnull = open("/dev/null", O_RDWR);
    int stdoutfile = open((sCWD+"stdout.out"+to_string(numsec)).c_str(), O_WRONLY );
    int stderrfile = open((sCWD+"stderr.out"+to_string(numsec)).c_str(), O_WRONLY );

    // close stdin stdout and stderr which are connected to current terminal
    close(0);
    close(1);
    close(2);

    // attach stdin stdout and stderr to /dev/null
    // by dup-ing the devnull descriptor
    dup2(devnull, 0);
    dup2(stdoutfile, 1);
    dup2(stderrfile, 2);

    // get rid of devnull
    close(devnull);
    close(stdoutfile);
    close(stderrfile);
    }

    printf("numsec=%d\n",numsec);
    for (int i=0; i<min(autorestart,numsec); i++)
    {
        this_thread::sleep_for(chrono::seconds(1));
        printf("%d\n",i);
    }


    if (numsec<=autorestart) exit(0);
    execl(argv[0],argv[0],to_string(numsec - autorestart).c_str(),NULL);
    return 0;
}

Działa jak jest napisany. :) A może napisz, czego się spodziewasz? Poza tym, dlaczego robisz dziwne maniany z plikami -- najpierw otwierasz, potem zamykasz, a potem znowu otwierasz, ale już innym sposobem? Takie mieszanie różnych interfejsów programistycznych dla jednego pliku fizycznego nigdy w niczym nie pomaga, a zwykle prowokuje jakąś katastrofę...

Rzeczywiście teraz działa, jest execl(argv[0],argv[0], przedtem było tylko raz argv[0]

Jak używasz dup2 to nie musisz zamykać, to raz.
Dwa, sprawdzaj zwrotkę z setsid.
Trzy, zdecyduj się czy printf i fopoen czy write i open. Teoretycznie nie ma problemu z mieszaniem C, C++ i syscalli Linuxa...a praktyka bywa trudniejsza.
Cztery, dobrą praktyką (acz niezawsze konieczną) jest dodatkowy fork po setsid.
Dodatkowo jak używasz plików (zamiast calla syslog) poczytaj o flock dla uzyskania wyłącznego dostępu.

alagner napisał(a):

Trzy, zdecyduj się czy printf i fopoen czy write i open. Teoretycznie nie ma problemu z mieszaniem C, C++ i syscalli Linuxa...a praktyka bywa trudniejsza.

Wydaje mi się, że teoretycznie właśnie taki problem może być. Skąd czerpiesz swoją teoretyczną wiedzę, jakieś źródła? :)

A problem może być z powodu różnych buforów, a także dodatkowych poziomów interfejsu...