Additional command line option sanity checking

orbcode · Aug 22, 2024 · 2ca861d · 2ca861d
1 parent 8ca8a68
commit 2ca861d
Show file tree

Hide file tree

Showing 4 changed files with 71 additions and 7 deletions.
diff --git a/Inc/generics.h b/Inc/generics.h
@@ -96,7 +96,7 @@ char *genericsEscape( char *str );
 char *genericsUnescape( char *str );
 uint64_t genericsTimestampuS( void );
 uint32_t genericsTimestampmS( void );
-void genericsSetReportLevel( enum verbLevel lset );
+bool genericsSetReportLevel( enum verbLevel lset );
 void genericsPrintf( const char *fmt, ... );
 char *genericsGetBaseDirectory( void );
 const char *genericsBasename( const char *n );

diff --git a/Src/generics.c b/Src/generics.c
@@ -165,10 +165,18 @@ char *genericsUnescape( char *str )
 // ====================================================================================================
 static enum verbLevel lstore = V_WARN;
 
-void genericsSetReportLevel( enum verbLevel lset )
+bool genericsSetReportLevel( enum verbLevel lset )
 
 {
+    /* Set is respected to maintain historic behaviour, even if out of range */
     lstore = lset;
+
+    if ( ( lset < V_ERROR ) || ( lset > V_MAX_VERBLEVEL ) )
+    {
+        return false;
+    }
+
+    return true;
 }
 // ====================================================================================================
 enum verbLevel genericsGetReportLevel( void )

diff --git a/Src/orbcat.c b/Src/orbcat.c
@@ -669,6 +669,13 @@ bool _processOptions( int argc, char *argv[] )
             // ------------------------------------
             case 'C':
                 options.cps = atoi( optarg ) * 1000;
+
+                if ( options.cps <= 0 )
+                {
+                    genericsReport( V_ERROR, "cps out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
@@ -693,7 +700,6 @@ bool _processOptions( int argc, char *argv[] )
 
             // ------------------------------------
             case 'g':
-                printf( "%s" EOL, optarg );
                 options.tsTrigger = genericsUnescape( optarg )[0];
                 break;
 
@@ -758,6 +764,13 @@ bool _processOptions( int argc, char *argv[] )
             // ------------------------------------
             case 't':
                 options.tag = atoi( optarg );
+
+                if ( !options.tag || ( options.tag > 255 ) )
+                {
+                    genericsReport( V_ERROR, "tag out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
@@ -799,7 +812,12 @@ bool _processOptions( int argc, char *argv[] )
                     return false;
                 }
 
-                genericsSetReportLevel( atoi( optarg ) );
+                if ( !genericsSetReportLevel( atoi( optarg ) ) )
+                {
+                    genericsReport( V_ERROR, "Report level out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
@@ -809,7 +827,7 @@ bool _processOptions( int argc, char *argv[] )
 
                 chan = atoi( optarg );
 
-                if ( chan >= NUM_CHANNELS )
+                if ( !chan || ( chan >= NUM_CHANNELS ) )
                 {
                     genericsReport( V_ERROR, "Channel index out of range" EOL );
                     return false;
@@ -821,6 +839,12 @@ bool _processOptions( int argc, char *argv[] )
                     chanIndex++;
                 }
 
+                if ( !*chanIndex )
+                {
+                    genericsReport( V_ERROR, "Channel output spec missing" EOL );
+                    return false;
+                }
+
                 /* Step over delimiter */
                 chanIndex++;
 
@@ -836,7 +860,6 @@ bool _processOptions( int argc, char *argv[] )
                     return false;
                 }
 
-                //*chanIndex++ = 0;
                 options.presFormat[chan] = strdup( genericsUnescape( chanIndex ) );
                 break;
 

diff --git a/Src/orbuculum.c b/Src/orbuculum.c
@@ -429,6 +429,13 @@ bool _processOptions( int argc, char *argv[], struct RunTime *r )
             case 'a':
                 r->options->speed = atoi( optarg );
                 r->options->dataSpeed = r->options->speed;
+
+                if ( r->options->speed <= 0 )
+                {
+                    genericsReport( V_ERROR, "Speed out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
@@ -462,12 +469,26 @@ bool _processOptions( int argc, char *argv[], struct RunTime *r )
 
             case 'l':
                 r->options->listenPort = atoi( optarg );
+
+                if ( ( r->options->listenPort <= 0 ) || ( r->options->listenPort > 0xffff ) )
+                {
+                    genericsReport( V_ERROR, "Port to listen on is out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
 
             case 'm':
                 r->options->intervalReportTime = atoi( optarg );
+
+                if ( r->options->intervalReportTime )
+                {
+                    genericsReport( V_ERROR, "intervalReportTime is out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
@@ -504,6 +525,13 @@ bool _processOptions( int argc, char *argv[], struct RunTime *r )
 
             case 'P':
                 r->options->paceDelay = atoi( optarg );
+
+                if ( r->options->paceDelay <= 0 )
+                {
+                    genericsReport( V_ERROR, "paceDelay is out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------
@@ -550,7 +578,12 @@ bool _processOptions( int argc, char *argv[], struct RunTime *r )
                     return false;
                 }
 
-                genericsSetReportLevel( atoi( optarg ) );
+                if ( !genericsSetReportLevel( atoi( optarg ) ) )
+                {
+                    genericsReport( V_ERROR, "Verbosity out of range" EOL );
+                    return false;
+                }
+
                 break;
 
             // ------------------------------------