X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fmisc%2Flogger.c;h=a275498c803564f6b4af19457332360d3d9ef2e0;hb=371b0ddd9c66218094043699b44e4fdd59433a42;hp=88d49624b89f64015f220b89adee23cfa1458083;hpb=d96b3d9b4ad33e18070e582354bdbd436022ba0e;p=vlc diff --git a/modules/misc/logger.c b/modules/misc/logger.c index 88d49624b8..a275498c80 100644 --- a/modules/misc/logger.c +++ b/modules/misc/logger.c @@ -16,9 +16,9 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA. + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA. *****************************************************************************/ /***************************************************************************** @@ -32,15 +32,11 @@ #include #include #include -#include +#include #include #include -#ifdef UNDER_CE -# define _IONBF 0x0004 -#endif - #define MODE_TEXT 0 #define MODE_HTML 1 #define MODE_SYSLOG 2 @@ -72,7 +68,7 @@ " \n" \ "\n" -#if HAVE_SYSLOG_H +#ifdef HAVE_SYSLOG_H #include #endif @@ -120,15 +116,48 @@ static const char *const mode_list_text[] = { N_("Text"), "HTML" }; #define LOGMODE_TEXT N_("Log format") -#ifdef HAVE_SYSLOG_H +#ifndef HAVE_SYSLOG_H #define LOGMODE_LONGTEXT N_("Specify the log format. Available choices are " \ - "\"text\" (default), \"html\", and \"syslog\" (special mode to send to " \ - "syslog instead of file.") + "\"text\" (default) and \"html\".") #else + #define LOGMODE_LONGTEXT N_("Specify the log format. Available choices are " \ - "\"text\" (default) and \"html\".") + "\"text\" (default), \"html\", and \"syslog\" (special mode to send to " \ + "syslog instead of file.") + +#define SYSLOG_FACILITY_TEXT N_("Syslog facility") +#define SYSLOG_FACILITY_LONGTEXT N_("Select the syslog facility where logs " \ + "will be forwarded. Available choices are \"user\" (default), \"daemon\", " \ + "and \"local0\" through \"local7\".") + +/* First in list is the default facility used. */ +#define DEFINE_SYSLOG_FACILITY \ + DEF( "user", LOG_USER ), \ + DEF( "daemon", LOG_DAEMON ), \ + DEF( "local0", LOG_LOCAL0 ), \ + DEF( "local1", LOG_LOCAL1 ), \ + DEF( "local2", LOG_LOCAL2 ), \ + DEF( "local3", LOG_LOCAL3 ), \ + DEF( "local4", LOG_LOCAL4 ), \ + DEF( "local5", LOG_LOCAL5 ), \ + DEF( "local6", LOG_LOCAL6 ), \ + DEF( "local7", LOG_LOCAL7 ) + +#define DEF( a, b ) a +static const char *const fac_name[] = { DEFINE_SYSLOG_FACILITY }; +#undef DEF +#define DEF( a, b ) b +static const int fac_number[] = { DEFINE_SYSLOG_FACILITY }; +#undef DEF +enum { fac_entries = sizeof(fac_name)/sizeof(fac_name[0]) }; +#undef DEFINE_SYSLOG_FACILITY + #endif +#define LOGVERBOSE_TEXT N_("Verbosity") +#define LOGVERBOSE_LONGTEXT N_("Select the verbosity to use for log or -1 to " \ +"use the same verbosity given by --verbose.") + vlc_module_begin () set_shortname( N_( "Logging" ) ) set_description( N_("File logging") ) @@ -136,12 +165,19 @@ vlc_module_begin () set_category( CAT_ADVANCED ) set_subcategory( SUBCAT_ADVANCED_MISC ) - add_file( "logfile", NULL, NULL, + add_savefile( "logfile", NULL, N_("Log filename"), N_("Specify the log filename."), false ) - add_string( "logmode", "text", NULL, LOGMODE_TEXT, LOGMODE_LONGTEXT, + add_string( "logmode", "text", LOGMODE_TEXT, LOGMODE_LONGTEXT, false ) change_string_list( mode_list, mode_list_text, 0 ) - +#ifdef HAVE_SYSLOG_H + add_string( "syslog-facility", fac_name[0], SYSLOG_FACILITY_TEXT, + SYSLOG_FACILITY_LONGTEXT, true ) + change_string_list( fac_name, fac_name, 0 ) +#endif + add_integer( "log-verbose", -1, LOGVERBOSE_TEXT, LOGVERBOSE_LONGTEXT, + false ) + add_obsolete_string( "rrd-file" ) set_capability( "interface", 0 ) @@ -158,7 +194,7 @@ static int Open( vlc_object_t *p_this ) char *psz_mode; CONSOLE_INTRO_MSG; - msg_Info( p_intf, "using logger..." ); + msg_Info( p_intf, "using logger." ); /* Allocate instance and initialize some members */ p_sys = p_intf->p_sys = (intf_sys_t *)malloc( sizeof( intf_sys_t ) ); @@ -167,7 +203,7 @@ static int Open( vlc_object_t *p_this ) p_sys->msg.p_intf = p_intf; p_sys->msg.i_mode = MODE_TEXT; - psz_mode = var_CreateGetString( p_intf, "logmode" ); + psz_mode = var_InheritString( p_intf, "logmode" ); if( psz_mode ) { if( !strcmp( psz_mode, "text" ) ) @@ -196,14 +232,17 @@ static int Open( vlc_object_t *p_this ) if( p_sys->msg.i_mode != MODE_SYSLOG ) { - char *psz_file = config_GetPsz( p_intf, "logfile" ); + char *psz_file = var_InheritString( p_intf, "logfile" ); if( !psz_file ) { #ifdef __APPLE__ - if( asprintf( &psz_file, "%s/"LOG_DIR"/%s", config_GetHomeDir(), + char *home = config_GetUserDir(VLC_DOCUMENTS_DIR); + if( home == NULL + || asprintf( &psz_file, "%s/"LOG_DIR"/%s", home, (p_sys->msg.i_mode == MODE_HTML) ? LOG_FILE_HTML : LOG_FILE_TEXT ) == -1 ) psz_file = NULL; + free(home); #else switch( p_sys->msg.i_mode ) { @@ -222,7 +261,7 @@ static int Open( vlc_object_t *p_this ) /* Open the log file and remove any buffering for the stream */ msg_Dbg( p_intf, "opening logfile `%s'", psz_file ); - p_sys->msg.p_file = utf8_fopen( psz_file, "at" ); + p_sys->msg.p_file = vlc_fopen( psz_file, "at" ); if( p_sys->msg.p_file == NULL ) { msg_Err( p_intf, "error opening logfile `%s'", psz_file ); @@ -250,7 +289,36 @@ static int Open( vlc_object_t *p_this ) { p_sys->msg.p_file = NULL; #ifdef HAVE_SYSLOG_H - openlog( "vlc", LOG_PID|LOG_NDELAY, LOG_DAEMON ); + int i_facility; + char *psz_facility = var_InheritString( p_intf, "syslog-facility" ); + if( psz_facility ) + { + bool b_valid = 0; + for( size_t i = 0; i < fac_entries; ++i ) + { + if( !strcmp( psz_facility, fac_name[i] ) ) + { + i_facility = fac_number[i]; + b_valid = 1; + break; + } + } + if( !b_valid ) + { + msg_Warn( p_intf, "invalid syslog facility `%s', using `%s'", + psz_facility, fac_name[0] ); + i_facility = fac_number[0]; + } + free( psz_facility ); + } + else + { + msg_Warn( p_intf, "no syslog facility specified, using `%s'", + fac_name[0] ); + i_facility = fac_number[0]; + } + + openlog( "vlc", LOG_PID|LOG_NDELAY, i_facility ); #endif } @@ -300,16 +368,27 @@ static void Close( vlc_object_t *p_this ) */ static void Overflow (msg_cb_data_t *p_sys, msg_item_t *p_item, unsigned overruns) { - int verbosity = var_CreateGetInteger( p_sys->p_intf, "verbose" ); - int priority = 0; + VLC_UNUSED(overruns); + int verbosity = var_InheritInteger( p_sys->p_intf, "log-verbose" ); + if (verbosity == -1) + verbosity = var_InheritInteger( p_sys->p_intf, "verbose" ); switch( p_item->i_type ) { - case VLC_MSG_WARN: priority = 1; break; - case VLC_MSG_DBG: priority = 2; break; + case VLC_MSG_INFO: + case VLC_MSG_ERR: + if( verbosity < 0 ) return; + break; + case VLC_MSG_WARN: + if( verbosity < 1 ) return; + break; + case VLC_MSG_DBG: + if( verbosity < 2 ) return; + break; } - if (verbosity < priority) - return; + + + int canc = vlc_savecancel(); switch( p_sys->i_mode ) { @@ -326,6 +405,8 @@ static void Overflow (msg_cb_data_t *p_sys, msg_item_t *p_item, unsigned overrun TextPrint( p_item, p_sys->p_file ); break; } + + vlc_restorecancel( canc ); } static const char ppsz_type[4][11] = { @@ -337,8 +418,8 @@ static const char ppsz_type[4][11] = { static void TextPrint( const msg_item_t *p_msg, FILE *p_file ) { - fprintf( p_file, "%s%s%s\n", p_msg->psz_module, ppsz_type[p_msg->i_type], - p_msg->psz_msg ); + utf8_fprintf( p_file, "%s%s%s\n", p_msg->psz_module, + ppsz_type[p_msg->i_type], p_msg->psz_msg ); } #ifdef HAVE_SYSLOG_H