Restrict usage on -e option (prevent array overflow with many -e and prevent excludin...

[dump.git] / dump / main.c

diff --git a/dump/main.c b/dump/main.c
index c2e4eaf329c94e77e671c50915ec4e027313d792..eac1f5baad11fdea07589273ef5b15d370c9742b 100644 (file)
--- a/dump/main.c
+++ b/dump/main.c
@@ -40,7 +40,7 @@
 
 #ifndef lint
 static const char rcsid[] =
-       "$Id: main.c,v 1.23 2000/08/19 23:48:10 stelian Exp $";
+       "$Id: main.c,v 1.25 2000/09/01 14:40:27 stelian Exp $";
 #endif /* not lint */
 
 #include <sys/param.h>
@@ -79,6 +79,7 @@ static const char rcsid[] =
 
 #include "dump.h"
 #include "pathnames.h"
+#include "bylabel.h"
 
 #ifndef SBOFF
 #define SBOFF (SBLOCK * DEV_BSIZE)
@@ -126,6 +127,7 @@ main(int argc, char *argv[])
 #endif
        time_t tnow;
        char labelstr[LBLSIZE];
+       char *diskparam;
 
        spcl.c_date = 0;
 #ifdef __linux__
@@ -200,8 +202,15 @@ main(int argc, char *argv[])
                        
                                        /* 04-Feb-00 ILC */
                case 'e':               /* exclude an inode */
-                       iexclude_list[iexclude_num++] = 
-                         numarg("inode to exclude",0L,0L);
+                       if (iexclude_num == IEXCLUDE_MAXNUM) {
+                               (void)fprintf(stderr, "Too many -e options\n");
+                               exit(X_STARTUP);
+                       }
+                       iexclude_list[iexclude_num++] = numarg("inode to exclude",0L,0L);
+                       if (iexclude_list[iexclude_num-1] <= ROOTINO) {
+                               (void)fprintf(stderr, "Cannot exclude inode %d\n", iexclude_list[iexclude_num-1]);
+                               exit(X_STARTUP);
+                       }
                        msg("Added %d to exclude list\n",
                            iexclude_list[iexclude_num-1]);
                        break;
@@ -287,9 +296,9 @@ main(int argc, char *argv[])
                (void)fprintf(stderr, "Must specify disk or filesystem\n");
                exit(X_STARTUP);
        }
-       disk = *argv++;
-       if (strlen(disk) >= MAXPATHLEN) {
-               (void)fprintf(stderr, "Disk or filesystem name too long: %s\n", disk);
+       diskparam = *argv++;
+       if (strlen(diskparam) >= MAXPATHLEN) {
+               (void)fprintf(stderr, "Disk or filesystem name too long: %s\n", diskparam);
                exit(X_STARTUP);
        }
        argc--;
@@ -363,6 +372,14 @@ main(int argc, char *argv[])
                signal(SIGINT, SIG_IGN);
        set_operators();        /* /etc/group snarfed */
        getfstab();             /* /etc/fstab snarfed */
+
+       disk = get_device_name(diskparam);
+       if (!disk) {            /* null means the disk is some form
+                                  of LABEL= or UID= but it was not
+                                  found */
+               msg("Cannot find a disk having %s\n", diskparam);
+               exit(X_STARTUP);
+       }
        /*
         *      disk may end in / and this can confuse
         *      fstabsearch.
@@ -421,7 +438,6 @@ main(int argc, char *argv[])
                        exit(X_STARTUP);
                }
        }
-
        spcl.c_dev[NAMELEN-1]='\0';
        spcl.c_filesys[NAMELEN-1]='\0';
        (void)strncpy(spcl.c_label, labelstr, sizeof(spcl.c_label) - 1);