Address complaints by code scanner

[ric-plt/lib/rmr.git] / src / rmr / common / src / logging.c

diff --git a/src/rmr/common/src/logging.c b/src/rmr/common/src/logging.c
index e38d7b0..0407eed 100644 (file)
--- a/src/rmr/common/src/logging.c
+++ b/src/rmr/common/src/logging.c
@@ -147,12 +147,17 @@ extern void rmr_vlog( int write_level, char* fmt, ... ) {
 
        memset( msg, 0, sizeof( msg ) );                                                                // logging is slow; this ensures 0 term if msg is too large
        hlen = snprintf( msg, sizeof( msg ), "%ld %d/RMR [%s] ", (long) time( NULL ), log_pid, log_situations[write_level] );
+       if( hlen > sizeof( msg ) - 1024 ) {                                                             // should never happen, but be parinoid
+               return;
+       }
        body = msg + hlen;
 
        va_start( argp, fmt );          // suss out parm past fmt
 
        vsnprintf( body, sizeof( msg ) - (hlen+2), fmt, argp );                 // add in user message formatting it along the way
        fprintf( stderr, "%s", msg );                                                                   // we grew from printfs so all existing msg have \n; assume there
+
+       va_end( argp );
 }
 
 /*
@@ -187,6 +192,8 @@ extern void rmr_vlog_force( int write_level, char* fmt, ... ) {
 
        vsnprintf( body, sizeof( msg ) - (hlen+2), fmt, argp );                 // add in user message formatting it along the way
        fprintf( stderr, "%s", msg );                                                                   // we grew from printfs so all existing msg have \n; assume there
+
+       va_end( argp );
 }
 
 // -------------------- public functions that are needed -----------------