Changeset 10135

Timestamp:

Feb 7, 2010, 10:17:42 PM (13 years ago)

Author:

charles

Message:

(trunk daemon) #2877 "Don't let the daemon launch if settings.json is corrupt" -- implemented in trunk for 1.90

Location:

trunk

Files:

• daemon/daemon.c (modified) (6 diffs)
• libtransmission/session.c (modified) (6 diffs)
• libtransmission/utils.c (modified) (3 diffs)

trunk/daemon/daemon.c

@@ -229 +229 @@
 
 static void
+printMessage( tr_bool foreground, int level, const char * name, const char * message, const char * file, int line )
+{
+#ifdef HAVE_SYSLOG
+    if( foreground )
+    {
+        char timestr[64];
+        tr_getLogTimeStr( timestr, sizeof( timestr ) );
+        if( name )
+            fprintf( stderr, "[%s] %s %s (%s:%d)\n", timestr, name, message, file, line );
+        else
+            fprintf( stderr, "[%s] %s (%s:%d)\n", timestr, message, file, line );
+    }
+    else /* daemon... write to syslog */
+    {
+        int priority;
+
+        /* figure out the syslog priority */
+        switch( level ) {
+            case TR_MSG_ERR: priority = LOG_ERR; break;
+            case TR_MSG_DBG: priority = LOG_DEBUG; break;
+            default: priority = LOG_INFO; break;
+        }
+
+        if( name )
+            syslog( priority, "%s %s (%s:%d)", name, message, file, line );
+        else
+            syslog( priority, "%s (%s:%d)", message, file, line );
+    }
+#else
+    {
+        char timestr[64];
+        tr_getLogTimeStr( timestr, sizeof( timestr ) );
+        if( name )
+            fprintf( stderr, "[%s] %s %s (%s:%d)\n", timestr, name, message, file, line );
+        else
+            fprintf( stderr, "[%s] %s (%s:%d)\n", timestr, message, file, line );
+    }
+#endif
+}
+
+static void
 pumpLogMessages( tr_bool foreground )
 {
@@ -235 +276 @@
 
     for( l=list; l!=NULL; l=l->next )
-    {
-#ifdef HAVE_SYSLOG
-        if( foreground )
-        {
-            char timestr[64];
-            tr_getLogTimeStr( timestr, sizeof( timestr ) );
-            if( l->name )
-                fprintf( stderr, "[%s] %s %s (%s:%d)\n", timestr, l->name, l->message, l->file, l->line );
-            else
-                fprintf( stderr, "[%s] %s (%s:%d)\n", timestr, l->message, l->file, l->line );
-        }
-        else /* daemon... write to syslog */
-        {
-            int priority;
-
-            /* figure out the syslog priority */
-            switch( l->level ) {
-                case TR_MSG_ERR: priority = LOG_ERR; break;
-                case TR_MSG_DBG: priority = LOG_DEBUG; break;
-                default: priority = LOG_INFO; break;
-            }
-
-            if( l->name )
-                syslog( priority, "%s %s (%s:%d)", l->name, l->message, l->file, l->line );
-            else
-                syslog( priority, "%s (%s:%d)", l->message, l->file, l->line );
-        }
-#else
-        {
-            char timestr[64];
-            tr_getLogTimeStr( timestr, sizeof( timestr ) );
-            if( l->name )
-                fprintf( stderr, "[%s] %s %s (%s:%d)\n", timestr, l->name, l->message, l->file, l->line );
-            else
-                fprintf( stderr, "[%s] %s (%s:%d)\n", timestr, l->message, l->file, l->line );
-        }
-#endif
-    }
+        printMessage( foreground, l->level, l->name, l->message, l->file, l->line );
 
     tr_freeMessageList( list );
@@ -300 +304 @@
     configDir = getConfigDir( argc, (const char**)argv );
     loaded = tr_sessionLoadSettings( &settings, configDir, MY_NAME );
-    tr_bencDictAddBool( &settings, TR_PREFS_KEY_RPC_ENABLED, TRUE );
 
     /* overwrite settings from the comamndline */
@@ -391 +394 @@
     }
 
+    if( !loaded )
+    {
+        printMessage( foreground, TR_MSG_ERR, MY_NAME, "Error loading config file -- exiting.", __FILE__, __LINE__ );
+        return -1;
+    }
+
     if( dumpSettings )
     {
@@ -401 +410 @@
     if( !foreground && tr_daemon( TRUE, FALSE ) < 0 )
     {
-        fprintf( stderr, "failed to daemonize: %s\n", strerror( errno ) );
+        char buf[256];
+        tr_snprintf( buf, sizeof( buf ), "Failed to dameonize: %s", tr_strerror( errno ) );
+        printMessage( foreground, TR_MSG_ERR, MY_NAME, buf, __FILE__, __LINE__ );
         exit( 1 );
     }
@@ -407 +418 @@
     /* start the session */
     mySession = tr_sessionInit( "daemon", configDir, TRUE, &settings );
-
-    if( loaded )
-        tr_ninf( NULL, "Using settings from \"%s\"", configDir );
-    else
-        tr_nerr( NULL, "Couldn't find settings in \"%s\"; using defaults", configDir );
-
+    tr_ninf( NULL, "Using settings from \"%s\"", configDir );
     tr_sessionSaveSettings( mySession, configDir, &settings );
 

trunk/libtransmission/session.c

@@ -12 +12 @@
 
 #include <assert.h>
+#include <errno.h> /* ENOENT */
 #include <stdlib.h>
 #include <string.h> /* memcpy */
@@ -363 +364 @@
 tr_sessionLoadSettings( tr_benc * d, const char * configDir, const char * appName )
 {
+    int err = 0;
     char * filename;
     tr_benc fileSettings;
@@ -384 +386 @@
     /* file settings override the defaults */
     filename = tr_buildPath( configDir, "settings.json", NULL );
-    if( !tr_bencLoadFile( &fileSettings, TR_FMT_JSON, filename ) ) {
+    err = tr_bencLoadFile( &fileSettings, TR_FMT_JSON, filename );
+    if( !err ) {
         tr_bencMergeDicts( d, &fileSettings );
         tr_bencFree( &fileSettings );
-        success = TRUE;
     }
 
@@ -393 +395 @@
     tr_bencFree( &sessionDefaults );
     tr_free( filename );
+    success = (err==0) || (err==ENOENT);
     return success;
 }
@@ -411 +414 @@
     {
         tr_benc fileSettings;
-        if( !tr_bencLoadFile( &fileSettings, TR_FMT_JSON, filename ) )
+        const int err = tr_bencLoadFile( &fileSettings, TR_FMT_JSON, filename );
+        if( !err )
         {
             tr_bencMergeDicts( &settings, &fileSettings );
@@ -432 +436 @@
     /* save the result */
     tr_bencToFile( &settings, TR_FMT_JSON, filename );
-    tr_inf( "Saved \"%s\"", filename );
 
     /* cleanup */

trunk/libtransmission/utils.c

@@ -47 +47 @@
 
 
-int                   messageLevel = 0;
+int                   messageLevel = TR_MSG_INF;
 static tr_lock *      messageLock = NULL;
 static tr_bool        messageQueuing = FALSE;
@@ -285 +285 @@
     va_list ap;
 
-    tr_lockLock( messageLock );
+    if( messageLock != NULL )
+        tr_lockLock( messageLock );
 
     /* build the text message */
@@ -343 +344 @@
     }
 
-    tr_lockUnlock( messageLock );
+    if( messageLock != NULL )
+        tr_lockUnlock( messageLock );
+
     errno = err;
 }