Hello,

Inside the child process, I closed the file descriptor and then reopened
and redirected them using dup2() and later closed the opened file. I am not
sure if I understood and used the functionality of dup2() in the right
sense.

Regards,
Binathi

On Sat, Nov 1, 2014 at 10:18 PM, Lukasz Marek <lukasz.m.lu...@gmail.com>
wrote:

> On 01.11.2014 17:20, Binathi Bingi wrote:
>
>> +    if (config.ffserver_daemon) {
>> +        int ffserver_id = 0;
>>
>
> You may change int to pid_t too, which is actually returned by fork()
>
>  +        pid_t sid = 0;
>> +
>> +        ffserver_id = fork();
>>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
From 19c03969797800d43b7cf29fb0dfdbc2ac272a40 Mon Sep 17 00:00:00 2001
From: Binathi <binti...@gmail.com>
Date: Fri, 31 Oct 2014 23:27:20 +0530
Subject: [PATCH] Restore Daemon functionality in FFserver

Signed-off-by: Binathi Bingi <binti...@gmail.com>
---
 doc/ffserver.conf |  5 +++++
 doc/ffserver.texi |  7 ++++---
 ffserver.c        | 39 ++++++++++++++++++++++++++++++++++++---
 ffserver_config.c |  6 ++++--
 ffserver_config.h |  1 +
 5 files changed, 50 insertions(+), 8 deletions(-)

diff --git a/doc/ffserver.conf b/doc/ffserver.conf
index b756961..0b63555 100644
--- a/doc/ffserver.conf
+++ b/doc/ffserver.conf
@@ -25,6 +25,11 @@ MaxBandwidth 1000
 # '-' is the standard output.
 CustomLog -
 
+# Suppress NoDaemon and enable Daemon, if you want to launch ffserver in daemon mode.
+# If no option is specified, default option is NoDaemon.
+#NoDaemon
+#Daemon
+
 ##################################################################
 # Definition of the live feeds. Each live feed contains one video
 # and/or audio sequence coming from an ffmpeg encoder or another
diff --git a/doc/ffserver.texi b/doc/ffserver.texi
index 77273d2..d7a6cbe 100644
--- a/doc/ffserver.texi
+++ b/doc/ffserver.texi
@@ -405,9 +405,10 @@ In case the commandline option @option{-d} is specified this option is
 ignored, and the log is written to standard output.
 
 @item NoDaemon
-Set no-daemon mode. This option is currently ignored since now
-@command{ffserver} will always work in no-daemon mode, and is
-deprecated.
+Set no-daemon mode. This is the default.
+
+@item Daemon
+Set daemon mode. The default is NoDaemon 
 @end table
 
 @section Feed section
diff --git a/ffserver.c b/ffserver.c
index ea2a2ae..5143b84 100644
--- a/ffserver.c
+++ b/ffserver.c
@@ -3671,6 +3671,7 @@ static void handle_child_exit(int sig)
 static void opt_debug(void)
 {
     config.debug = 1;
+    config.ffserver_daemon = 0;
     snprintf(config.logfilename, sizeof(config.logfilename), "-");
 }
 
@@ -3694,7 +3695,7 @@ int main(int argc, char **argv)
 {
     struct sigaction sigact = { { 0 } };
     int ret = 0;
-
+    int fd;
     config.filename = av_strdup("/etc/ffserver.conf");
 
     parse_loglevel(argc, argv, options);
@@ -3736,10 +3737,42 @@ int main(int argc, char **argv)
     build_feed_streams();
 
     compute_bandwidth();
-
+    
+    if (config.ffserver_daemon) {	
+	pid_t ffserver_id = 0; 
+	pid_t sid = 0;
+	
+	ffserver_id = fork(); 
+
+	if (ffserver_id < 0) {
+            ret = AVERROR(errno);
+	    av_log(NULL, AV_LOG_ERROR, "Impossible to start in daemon mode: %s\n", av_err2str(ret)); 
+	    exit(1);
+	}
+
+	if (ffserver_id > 0) { 
+	    exit(0);
+	}
+	
+	sid = setsid(); 
+	if (sid < 0) {
+	    exit(1); 
+	}
+	
+	if (strcmp(config.logfilename, "-") != 0) {
+            close(0);
+	
+	fd = open("/dev/null", O_RDWR);
+	dup2(fd,0);
+	dup2(fd,1);
+	dup2(fd,2);
+	close(fd);
+        
+        }
+    }
     /* signal init */
     signal(SIGPIPE, SIG_IGN);
-
+    
     if (http_server() < 0) {
         http_log("Could not start server\n");
         exit(1);
diff --git a/ffserver_config.c b/ffserver_config.c
index e44cdf7..63cfd43 100644
--- a/ffserver_config.c
+++ b/ffserver_config.c
@@ -358,8 +358,10 @@ static int ffserver_parse_config_global(FFServerConfig *config, const char *cmd,
         ffserver_get_arg(arg, sizeof(arg), p);
         if (resolve_host(&config->http_addr.sin_addr, arg) != 0)
             ERROR("%s:%d: Invalid host/IP address: %s\n", arg);
-    } else if (!av_strcasecmp(cmd, "NoDaemon")) {
-        WARNING("NoDaemon option has no effect, you should remove it\n");
+    } else if (!av_strcasecmp(cmd, "Daemon")){
+        config->ffserver_daemon = 1;
+    } else if (!av_strcasecmp(cmd, "NoDaemon")){
+        config->ffserver_daemon = 0;
     } else if (!av_strcasecmp(cmd, "RTSPPort")) {
         ffserver_get_arg(arg, sizeof(arg), p);
         val = atoi(arg);
diff --git a/ffserver_config.h b/ffserver_config.h
index 36d61d0..e3957b1 100644
--- a/ffserver_config.h
+++ b/ffserver_config.h
@@ -100,6 +100,7 @@ typedef struct FFServerConfig {
     unsigned int nb_max_http_connections;
     unsigned int nb_max_connections;
     uint64_t max_bandwidth;
+    int ffserver_daemon;
     int debug;
     char logfilename[1024];
     struct sockaddr_in http_addr;
-- 
1.9.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to