Почему не работает демон?

Question

[egorggegor]

Всем привет.
Написал следующий демон, он не работает. Предполагаю это из-за закрытия потоков ввода/вывода, но до конца не понимаю, что нужно исправить, чтобы заработало.

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <string.h>

int main(int argc, char* argv[])
{
	int fd;
	pid_t process_id = 0;
	pid_t sid = 0;

	process_id = fork();

	if (process_id < 0)
	{
		printf("fork failed!\n");
		exit(1);
	}

	if (process_id > 0)
	{
		printf("process_id of child process %d \n", process_id);
		exit(0);
	}

	umask(0);
	sid = setsid();
	if(sid < 0)
	{
		exit(1);
	}

	chdir("/");

	close(STDIN_FILENO);
	close(STDOUT_FILENO);
	close(STDERR_FILENO);

	while(1) {
		fd = open("/dev/mydev", O_RDONLY);
		read(fd, &buf, 7);
		close(fd);
	}
	return EXIT_SUCCESS;
}

Спасибо.

Comments

[Евгений]

А что в stderr пишет?

[egorggegor]

Евгений, как он может что-то писать в stderr, если он закрывается?

[Saboteur]

egorggegor, А почему не может написать при закрытии?
из командной строки если запускается, то сразу закрывается?

[Евгений]

egorggegor, вывод в stderr определятся не только вашим кодом. возможно у вас валится какой-нибудь permission denied. какой exit code после закрытия вашего приложения? ./your-app; echo $?

Answer 1

[Владимир Дубровин]

1. В read наверняка надо передавать buf а не &buf (точно сказать нельзя т.к. описания нет)
2. В зависимости от того, как вы запускаете процесс, дочерний процесс может получить SIGHUP при закрытии родительского процесса или SIGPIPE на printf, стоит добавить обработчики сигналов до fork().

Answer 2

[Vitsliputsli]

Не хватает, как минимум:

#include <fcntl.h>

char buf[7];

Comments

[egorggegor]

Если бы не хватало, то, я думаю, не скомпилировалось бы

[Кот Абсолютный]

egorggegor, ага, щазз. Внимательнее копипастить надо :) Читать куда будешь, в какую область? Вот эти семь байт, которые ты собрался прочитать из /dev/mydev - что это за странное устройство такое? - куда читать будешь?

[res2001]

egorggegor,

я думаю, не скомпилировалось бы

А вы хотите сказать, что оно скомпилировалось?
https://ideone.com/loDx1q

buf нужно объявлять как то так:
char buf[7];
Вызов read в этом случае такой:
read(fd, buf, 7);