]> git.wh0rd.org - dump.git/blobdiff - dump/traverse.c
'filetype' feature bug when dumping huge filesystems.
[dump.git] / dump / traverse.c
index ccc03f77244c9b56be9c294a337d6575e0b758c6..9c275c1a1d4934a658943151c9118c82f8c084ba 100644 (file)
@@ -1,8 +1,8 @@
 /*
  *     Ported to Linux's Second Extended File System as part of the
  *     dump and restore backup suit
- *     Remy Card <card@Linux.EU.Org>, 1994, 1995, 1996
- *
+ *     Remy Card <card@Linux.EU.Org>, 1994-1997
+ *     Stelian Pop <pop@cybercable.fr>, 1999 
  */
 
 /*-
  */
 
 #ifndef lint
-static char sccsid[] = "@(#)traverse.c 8.7 (Berkeley) 6/15/95";
+static const char rcsid[] =
+       "$Id: traverse.c,v 1.12 2000/01/09 23:47:33 tiniou Exp $";
 #endif /* not lint */
 
 #include <sys/param.h>
-#include <sys/time.h>
 #include <sys/stat.h>
 #ifdef __linux__
 #include <linux/ext2_fs.h>
 #include <bsdcompat.h>
-#include <err.h>
+#include <compaterr.h>
 #include <stdlib.h>
+#define swab32(x) ext2fs_swab32(x)
 #else  /* __linux__ */
+#define swab32(x) x
 #ifdef sunos
 #include <sys/vnode.h>
 
@@ -88,16 +90,44 @@ typedef     quad_t fsizeT;
 typedef        long fsizeT;
 #endif
 
-#ifndef        __linux__
-static int dirindir __P((ino_t ino, daddr_t blkno, int level, long *size));
-#endif
-static void dmpindir __P((ino_t ino, daddr_t blk, int level, fsizeT *size));
 #ifdef __linux__
 static int searchdir __P((struct ext2_dir_entry *dp, int offset,
                           int blocksize, char *buf, void *private));
 #else
+static int dirindir __P((ino_t ino, daddr_t blkno, int level, long *size));
+static void dmpindir __P((ino_t ino, daddr_t blk, int level, fsizeT *size));
 static int searchdir __P((ino_t ino, daddr_t blkno, long size, long filesize));
 #endif
+static void mapfileino __P((ino_t ino, long *tapesize, int *dirskipped));
+
+/* #define EXT3_FEATURE_INCOMPAT_RECOVER */
+
+int dump_fs_open(const char *disk, ext2_filsys *fs)
+{
+       int retval;
+       struct ext2fs_sb *s;
+
+#ifdef EXT3_FEATURE_INCOMPAT_RECOVER
+       retval = ext2fs_open(disk, EXT2_FLAG_FORCE, 0, 0, unix_io_manager, fs);
+#else
+       retval = ext2fs_open(disk, 0, 0, 0, unix_io_manager, fs);
+#endif
+       if (!retval) {
+               s = (struct ext2fs_sb *) (*fs)->super;
+               if ((s->s_feature_compat & ~EXT2_LIB_FEATURE_COMPAT_SUPP) ||
+#ifdef EXT3_FEATURE_INCOMPAT_RECOVER
+                   (s->s_feature_incompat & ~(EXT3_FEATURE_INCOMPAT_RECOVER | EXT2_LIB_FEATURE_INCOMPAT_SUPP))) {
+#else
+                   (s->s_feature_incompat & ~EXT2_LIB_FEATURE_INCOMPAT_SUPP)) {
+#endif
+                       retval = EXT2_ET_UNSUPP_FEATURE;
+               }
+               else if (s->s_feature_ro_compat & ~EXT2_LIB_FEATURE_RO_COMPAT_SUPP) {
+                       retval = EXT2_ET_RO_UNSUPP_FEATURE;
+               }
+       }
+       return retval;
+}
 
 /*
  * This is an estimation of the number of TP_BSIZE blocks in the file.
@@ -107,8 +137,7 @@ static      int searchdir __P((ino_t ino, daddr_t blkno, long size, long filesize));
  * hence the estimate may be high.
  */
 long
-blockest(dp)
-       register struct dinode *dp;
+blockest(struct dinode *dp)
 {
        long blkest, sizeest;
 
@@ -161,6 +190,52 @@ blockest(dp)
 #define        WANTTODUMP(dp) CHANGEDSINCE(dp, spcl.c_ddate)
 #endif
 
+/*
+ * Determine if given inode should be dumped
+ */
+void
+mapfileino(ino_t ino, long *tapesize, int *dirskipped)
+{
+       register int mode;
+       register struct dinode *dp;
+
+       /*
+        * Skip inode if we've already marked it for dumping
+        */
+       if (TSTINO(ino, usedinomap))
+               return;
+       dp = getino(ino);
+       if ((mode = (dp->di_mode & IFMT)) == 0)
+               return;
+#ifdef __linux__
+       if (dp->di_nlink == 0 || dp->di_dtime != 0)
+               return;
+#endif
+       /*
+        * Put all dirs in dumpdirmap, inodes that are to be dumped in the
+        * used map. All inode but dirs who have the nodump attribute go
+        * to the usedinomap.
+        */
+       SETINO(ino, usedinomap);
+       if (mode == IFDIR)
+               SETINO(ino, dumpdirmap);
+       if (WANTTODUMP(dp)) {
+               SETINO(ino, dumpinomap);
+               if (mode != IFREG && mode != IFDIR && mode != IFLNK)
+                       *tapesize += 1;
+               else
+                       *tapesize += blockest(dp);
+               return;
+       }
+       if (mode == IFDIR) {
+#ifdef UF_NODUMP
+               if (!nonodump && (dp->di_flags & UF_NODUMP))
+                       CLRINO(ino, usedinomap);
+#endif
+               *dirskipped = 1;
+       }
+}
+
 /*
  * Dump pass 1.
  *
@@ -169,37 +244,14 @@ blockest(dp)
  * the directories in the filesystem.
  */
 int
-mapfiles(maxino, tapesize)
-       ino_t maxino;
-       long *tapesize;
+mapfiles(ino_t maxino, long *tapesize)
 {
-       register int mode;
        register ino_t ino;
-       register struct dinode *dp;
        int anydirskipped = 0;
 
-       for (ino = ROOTINO; ino < maxino; ino++) {
-               dp = getino(ino);
-               if ((mode = (dp->di_mode & IFMT)) == 0)
-                       continue;
-#ifdef __linux__
-               if (dp->di_nlink == 0 || dp->di_dtime != 0)
-                       continue;
-#endif
-               SETINO(ino, usedinomap);
-               if (mode == IFDIR)
-                       SETINO(ino, dumpdirmap);
-               if (WANTTODUMP(dp)) {
-                       SETINO(ino, dumpinomap);
-                       if (mode != IFREG && mode != IFDIR && mode != IFLNK)
-                               *tapesize += 1;
-                       else
-                               *tapesize += blockest(dp);
-                       continue;
-               }
-               if (mode == IFDIR)
-                       anydirskipped = 1;
-       }
+       for (ino = ROOTINO; ino < maxino; ino++)
+               mapfileino(ino, tapesize, &anydirskipped);
+
        /*
         * Restore gets very upset if the root is not dumped,
         * so ensure that it always is dumped.
@@ -211,48 +263,33 @@ mapfiles(maxino, tapesize)
 #ifdef __linux__
 struct mapfile_context {
        long *tapesize;
-       int anydirskipped;
+       int *anydirskipped;
 };
 
 static int
-mapfilesindir(dirent, offset, blocksize, buf, private)
-       struct ext2_dir_entry *dirent;
-       int offset;
-       int blocksize;
-       char *buf;
-       void *private;
+mapfilesindir(struct ext2_dir_entry *dirent, int offset, int blocksize, char *buf, void *private)
 {
        register struct dinode *dp;
        register int mode;
-       ino_t ino;
        errcode_t retval;
        struct mapfile_context *mfc;
+       ino_t ino;
 
        ino = dirent->inode;
        mfc = (struct mapfile_context *)private;
-       dp = getino(ino);
-       if ((mode = (dp->di_mode & IFMT)) != 0 &&
-           dp->di_nlink != 0 && dp->di_dtime == 0) {
-               SETINO(ino, usedinomap);
-               if (mode == IFDIR)
-                       SETINO(ino, dumpdirmap);
-               if (WANTTODUMP(dp)) {
-                       SETINO(ino, dumpinomap);
-                       if (mode != IFREG && mode != IFDIR && mode != IFLNK)
-                               *mfc->tapesize += 1;
-                       else
-                               *mfc->tapesize += blockest(dp);
-               }
-               if (mode == IFDIR) {
-                       mfc->anydirskipped = 1;
-                       if ((dirent->name[0] != '.' || dirent->name_len != 1) &&
-                           (dirent->name[0] != '.' || dirent->name[1] != '.' ||
-                            dirent->name_len != 2)) {
-                       retval = ext2fs_dir_iterate(fs, ino, 0, NULL,
-                                                   mapfilesindir, private);
-                       if (retval)
-                               return retval;
-                       }
+
+       mapfileino(dirent->inode, mfc->tapesize, mfc->anydirskipped);
+
+       dp = getino(dirent->inode);
+       mode = dp->di_mode & IFMT;
+       if (mode == IFDIR && dp->di_nlink != 0 && dp->di_dtime == 0) {
+               if ((dirent->name[0] != '.' || ( dirent->name_len & 0xFF ) != 1) &&
+                   (dirent->name[0] != '.' || dirent->name[1] != '.' ||
+                    ( dirent->name_len & 0xFF ) != 2)) {
+               retval = ext2fs_dir_iterate(fs, ino, 0, NULL,
+                                           mapfilesindir, private);
+               if (retval)
+                       return retval;
                }
        }
        return 0;
@@ -266,16 +303,13 @@ mapfilesindir(dirent, offset, blocksize, buf, private)
  * the directories in the filesystem.
  */
 int
-mapfilesfromdir(maxino, tapesize, directory)
-       ino_t maxino;
-       long *tapesize;
-       char *directory;
+mapfilesfromdir(ino_t maxino, long *tapesize, char *directory)
 {
        errcode_t retval;
        struct mapfile_context mfc;
        ino_t dir_ino;
        char dir_name [MAXPATHLEN];
-       int i;
+       int i, anydirskipped = 0;
 
        /*
         * Mark every directory in the path as being dumped
@@ -291,8 +325,7 @@ mapfilesfromdir(maxino, tapesize, directory)
                                        dir_name);
                                exit(X_ABORT);
                        }
-/*                     SETINO(dir_ino, dumpinomap); */
-                       SETINO(dir_ino, dumpdirmap);
+                       mapfileino(dir_ino, tapesize, &anydirskipped);
                }
        }
        /*
@@ -303,11 +336,10 @@ mapfilesfromdir(maxino, tapesize, directory)
                com_err(disk, retval, "while translating %s", directory);
                exit(X_ABORT);
        }
-/*     SETINO(dir_ino, dumpinomap); */
-       SETINO(dir_ino, dumpdirmap);
+       mapfileino(dir_ino, tapesize, &anydirskipped);
 
        mfc.tapesize = tapesize;
-       mfc.anydirskipped = 0;
+       mfc.anydirskipped = &anydirskipped;
        retval = ext2fs_dir_iterate(fs, dir_ino, 0, NULL, mapfilesindir,
                                    (void *)&mfc);
 
@@ -315,16 +347,28 @@ mapfilesfromdir(maxino, tapesize, directory)
                com_err(disk, retval, "while mapping files in %s", directory);
                exit(X_ABORT);
        }
+       /*
+        * Ensure that the root inode actually appears in the file list
+        * for a subdir
+        */
+       mapfileino(ROOTINO, tapesize, &anydirskipped);
        /*
         * Restore gets very upset if the root is not dumped,
         * so ensure that it always is dumped.
         */
-/*     SETINO(ROOTINO, dumpinomap); */
-       SETINO(ROOTINO, dumpdirmap);
-       return (mfc.anydirskipped);
+       SETINO(ROOTINO, dumpinomap);
+       return anydirskipped;
 }
 #endif
 
+#ifdef __linux__
+struct mapdirs_context {
+       int *ret;
+       int nodump;
+       long *tapesize;
+};
+#endif
+
 /*
  * Dump pass 2.
  *
@@ -338,16 +382,19 @@ mapfilesfromdir(maxino, tapesize, directory)
  * pass using this algorithm.
  */
 int
-mapdirs(maxino, tapesize)
-       ino_t maxino;
-       long *tapesize;
+mapdirs(ino_t maxino, long *tapesize)
 {
        register struct dinode *dp;
-       register int i, isdir;
+       register int isdir;
        register char *map;
        register ino_t ino;
+#ifndef __linux__
+       register int i;
        long filesize;
-       int ret, change = 0;
+#else
+       struct mapdirs_context mdc;
+#endif
+       int ret, change = 0, nodump;
 
        isdir = 0;              /* XXX just to get gcc to shut up */
        for (map = dumpdirmap, ino = 1; ino < maxino; ino++) {
@@ -355,14 +402,25 @@ mapdirs(maxino, tapesize)
                        isdir = *map++;
                else
                        isdir >>= 1;
-               if ((isdir & 1) == 0 || TSTINO(ino, dumpinomap))
+               /*
+                * If dir has been removed from the used map, it's either
+                * because it had the nodump flag, or it herited it from
+                * its parent. A directory can't be in dumpinomap if not
+                * in usedinomap, but we have to go through it anyway 
+                * to propagate the nodump attribute.
+                */
+               nodump = (TSTINO(ino, usedinomap) == 0);
+               if ((isdir & 1) == 0 ||
+                   (TSTINO(ino, dumpinomap) && nodump == 0))
                        continue;
-#ifdef __linux__
                dp = getino(ino);
+#ifdef __linux__
                ret = 0;
-               ext2fs_dir_iterate(fs, ino, 0, NULL, searchdir, (void *) &ret);
+               mdc.ret = &ret;
+               mdc.nodump = nodump;
+               mdc.tapesize = tapesize;
+               ext2fs_dir_iterate(fs, ino, 0, NULL, searchdir, (void *) &mdc);
 #else  /* __linux__ */
-               dp = getino(ino);
                filesize = dp->di_size;
                for (ret = 0, i = 0; filesize > 0 && i < NDADDR; i++) {
                        if (dp->di_db[i] != 0)
@@ -386,7 +444,11 @@ mapdirs(maxino, tapesize)
                        change = 1;
                        continue;
                }
-               if ((ret & HASSUBDIRS) == 0) {
+               if (nodump) {
+                       if (ret & HASSUBDIRS)
+                               change = 1; /* subdirs have inherited nodump */
+                       CLRINO(ino, dumpdirmap);
+               } else if ((ret & HASSUBDIRS) == 0) {
                        if (!TSTINO(ino, dumpinomap)) {
                                CLRINO(ino, dumpdirmap);
                                change = 1;
@@ -403,11 +465,7 @@ mapdirs(maxino, tapesize)
  * require the directory to be dumped.
  */
 static int
-dirindir(ino, blkno, ind_level, filesize)
-       ino_t ino;
-       daddr_t blkno;
-       int ind_level;
-       long *filesize;
+dirindir(ino_t ino, daddr_t blkno, int ind_level, long *filesize)
 {
        int ret = 0;
        register int i;
@@ -444,32 +502,48 @@ dirindir(ino, blkno, ind_level, filesize)
  */
 #ifdef __linux__
 static int
-searchdir(dp, offset, blocksize, buf, private)
-       struct ext2_dir_entry *dp;
-       int offset;
-       int blocksize;
-       char *buf;
-       void *private;
+searchdir(struct ext2_dir_entry *dp, int offset, int blocksize, char *buf, void *private)
 {
-       int *ret = (int *) private;
+       struct mapdirs_context *mdc;
+       int *ret;
+       long *tapesize;
+       struct dinode *ip;
+
+       mdc = (struct mapdirs_context *)private;
+       ret = mdc->ret;
+       tapesize = mdc->tapesize;
 
        if (dp->inode == 0)
                return 0;
        if (dp->name[0] == '.') {
-               if (dp->name_len == 1)
+               if (( dp->name_len & 0xFF ) == 1)
                        return 0;
-               if (dp->name[1] == '.' && dp->name_len == 2)
+               if (dp->name[1] == '.' && ( dp->name_len & 0xFF ) == 2)
                        return 0;
        }
-       if (TSTINO(dp->inode, dumpinomap)) {
-               *ret |= HASDUMPEDFILE;
-               if (*ret & HASSUBDIRS)
-                       return DIRENT_ABORT;
-       }
-       if (TSTINO(dp->inode, dumpdirmap)) {
-               *ret |= HASSUBDIRS;
-               if (*ret & HASDUMPEDFILE)
-                       return DIRENT_ABORT;
+       if (mdc->nodump) {
+               ip = getino(dp->inode);
+               if (TSTINO(dp->inode, dumpinomap)) {
+                       CLRINO(dp->inode, dumpinomap);
+                       CLRINO(dp->inode, usedinomap);
+                       *tapesize -= blockest(ip);
+               }
+               /* Add dir back to the dir map, to propagate nodump */
+               if ((ip->di_mode & IFMT) == IFDIR) {
+                       SETINO(dp->inode, dumpdirmap);
+                       *ret |= HASSUBDIRS;
+               }
+       } else {
+               if (TSTINO(dp->inode, dumpinomap)) {
+                       *ret |= HASDUMPEDFILE;
+                       if (*ret & HASSUBDIRS)
+                               return DIRENT_ABORT;
+               }
+               if (TSTINO(dp->inode, dumpdirmap)) {
+                       *ret |= HASSUBDIRS;
+                       if (*ret & HASDUMPEDFILE)
+                               return DIRENT_ABORT;
+               }
        }
        return 0;
 }
@@ -477,11 +551,7 @@ searchdir(dp, offset, blocksize, buf, private)
 #else  /* __linux__ */
 
 static int
-searchdir(ino, blkno, size, filesize)
-       ino_t ino;
-       daddr_t blkno;
-       register long size;
-       long filesize;
+searchdir(ino_t ino, daddr_t blkno, long size, long filesize)
 {
        register struct direct *dp;
        register long loc, ret = 0;
@@ -565,16 +635,16 @@ dumponeblock(ext2_filsys fs, blk_t *blocknr, int blockcnt, void *private)
  * Dump the contents of an inode to tape.
  */
 void
-dumpino(dp, ino)
-       register struct dinode *dp;
-       ino_t ino;
+dumpino(struct dinode *dp, ino_t ino)
 {
-       int ind_level, cnt;
+       int cnt;
        fsizeT size;
        char buf[TP_BSIZE];
        struct old_bsd_inode obi;
 #ifdef __linux__
        struct block_context bc;
+#else
+       int ind_level;
 #endif
 
        if (newtape) {
@@ -597,7 +667,7 @@ dumpino(dp, ino)
        obi.di_gen = dp->di_gen;
        memmove(&obi.di_db, &dp->di_db, (NDADDR + NIADDR) * sizeof(daddr_t));
        if (dp->di_file_acl || dp->di_dir_acl)
-               warn("ACLs in inode #%d won't be dumped", ino);
+               warn("ACLs in inode #%ld won't be dumped", ino);
        memmove(&spcl.c_dinode, &obi, sizeof(obi));
 #else  /* __linux__ */
        spcl.c_dinode = *dp;
@@ -676,7 +746,7 @@ dumpino(dp, ino)
        else
                cnt = howmany(dp->di_size, sblock->fs_fsize);
        blksout(&dp->di_db[0], cnt, ino);
-       if ((size = ((int)dp->di_size) - NDADDR * sblock->fs_bsize) <= 0)
+       if ((size = dp->di_size - NDADDR * sblock->fs_bsize) <= 0)
                return;
 #ifdef __linux__
        bc.max = NINDIR(sblock) * EXT2_FRAGS_PER_BLOCK(fs->super);
@@ -689,6 +759,7 @@ dumpino(dp, ino)
        if (bc.cnt > 0) {
                blksout (bc.buf, bc.cnt, bc.ino);
        }
+       free(bc.buf);
 #else
        for (ind_level = 0; ind_level < NIADDR; ind_level++) {
                dmpindir(ino, dp->di_ib[ind_level], ind_level, &size);
@@ -714,12 +785,7 @@ struct convert_dir_context {
  * size of the entry, and creates it in a temporary buffer
  */
 static int
-convert_dir(dirent, offset, blocksize, buf, private)
-       struct ext2_dir_entry *dirent;
-       int offset;
-       int blocksize;
-       char *buf;
-       void *private;
+convert_dir(struct ext2_dir_entry *dirent, int offset, int blocksize, char *buf, void *private)
 {
        struct convert_dir_context *p;
        struct direct *dp;
@@ -727,7 +793,7 @@ convert_dir(dirent, offset, blocksize, buf, private)
 
        p = (struct convert_dir_context *)private;
 
-       reclen = EXT2_DIR_REC_LEN(dirent->name_len + 1);
+       reclen = EXT2_DIR_REC_LEN((dirent->name_len & 0xFF) + 1);
        if (((p->offset + reclen - 1) / p->bs) != (p->offset / p->bs)) {
                dp = (struct direct *)(p->buf + p->prev_offset);
                dp->d_reclen += p->bs - (p->offset % p->bs);
@@ -738,8 +804,8 @@ convert_dir(dirent, offset, blocksize, buf, private)
        dp->d_ino = dirent->inode;
        dp->d_reclen = reclen;
        dp->d_type = 0;
-       dp->d_namlen = dirent->name_len;
-       strncpy(dp->d_name, dirent->name, dirent->name_len);
+       dp->d_namlen = dirent->name_len & 0xFF;
+       strncpy(dp->d_name, dirent->name, dp->d_namlen);
        dp->d_name[dp->d_namlen] = '\0';
        p->prev_offset = p->offset;
        p->offset += reclen;
@@ -753,9 +819,7 @@ convert_dir(dirent, offset, blocksize, buf, private)
  * Dumps a directory to tape after converting it to the BSD format
  */
 void
-dumpdirino(dp, ino)
-       register struct dinode *dp;
-       ino_t ino;
+dumpdirino(struct dinode *dp, ino_t ino)
 {
        fsizeT size;
        char buf[TP_BSIZE];
@@ -778,8 +842,8 @@ dumpdirino(dp, ino)
           ext2fs directory to avoid problems ;-) */
        cdc.buf = (char *)malloc(dp->di_size * 2 * sizeof(char));
        if (cdc.buf == NULL)
-               err(1, "Cannot allocate buffer to convert directory #%ld\n",
-                   ino);
+               err(1, "Cannot allocate buffer to convert directory #%lu\n",
+                   (unsigned long)ino);
        cdc.offset = 0;
        cdc.prev_offset = 0;
        cdc.bs = MIN(DIRBLKSIZ, TP_BSIZE);
@@ -813,7 +877,7 @@ dumpdirino(dp, ino)
        obi.di_gen = dp->di_gen;
        memmove(&obi.di_db, &dp->di_db, (NDADDR + NIADDR) * sizeof(daddr_t));
        if (dp->di_file_acl || dp->di_dir_acl)
-               warn("ACLs in inode #%d won't be dumped", ino);
+               warn("ACLs in inode #%ld won't be dumped", ino);
        memmove(&spcl.c_dinode, &obi, sizeof(obi));
 #else  /* __linux__ */
        spcl.c_dinode = *dp;
@@ -858,17 +922,34 @@ dumpdirino(dp, ino)
  * Read indirect blocks, and pass the data blocks to be dumped.
  */
 static void
-dmpindir(ino, blk, ind_level, size)
-       ino_t ino;
-       daddr_t blk;
-       int ind_level;
-       fsizeT *size;
+dmpindir(ino_t ino, daddr_t blk, int ind_level, fsizeT *size)
 {
        int i, cnt;
+#ifdef __linux__
+       int max;
+       blk_t *swapme;
+#endif
        daddr_t idblk[MAXNINDIR];
 
-       if (blk != 0)
+       if (blk != 0) {
                bread(fsbtodb(sblock, blk), (char *)idblk, (int) sblock->fs_bsize);
+#ifdef __linux__
+       /* 
+        * My RedHat 4.0 system doesn't have these flags; I haven't
+        * upgraded e2fsprogs yet
+        */
+#if defined(EXT2_FLAG_SWAP_BYTES)
+       if ((fs->flags & EXT2_FLAG_SWAP_BYTES) ||
+           (fs->flags & EXT2_FLAG_SWAP_BYTES_READ)) {
+#endif
+               max = sblock->fs_bsize >> 2;
+               swapme = (blk_t *) idblk;
+               for (i = 0; i < max; i++, swapme++)
+                       *swapme = swab32(*swapme);
+#if defined(EXT2_FLAG_SWAP_BYTES)
+       }
+#endif
+#endif
        else
                memset(idblk, 0, (int)sblock->fs_bsize);
        if (ind_level <= 0) {
@@ -897,10 +978,7 @@ dmpindir(ino, blk, ind_level, size)
  * Collect up the data into tape record sized buffers and output them.
  */
 void
-blksout(blkp, frags, ino)
-       daddr_t *blkp;
-       int frags;
-       ino_t ino;
+blksout(daddr_t *blkp, int frags, ino_t ino)
 {
        register daddr_t *bp;
        int i, j, count, blks, tbperdb;
@@ -920,12 +998,14 @@ blksout(blkp, frags, ino)
                spcl.c_count = count - i;
                writeheader(ino);
                bp = &blkp[i / tbperdb];
-               for (j = i; j < count; j += tbperdb, bp++)
-                       if (*bp != 0)
+               for (j = i; j < count; j += tbperdb, bp++) {
+                       if (*bp != 0) {
                                if (j + tbperdb <= count)
                                        dumpblock(*bp, (int)sblock->fs_bsize);
                                else
                                        dumpblock(*bp, (count - j) * TP_BSIZE);
+                       }
+               }
                spcl.c_type = TS_ADDR;
        }
 }
@@ -934,10 +1014,7 @@ blksout(blkp, frags, ino)
  * Dump a map to the tape.
  */
 void
-dumpmap(map, type, ino)
-       char *map;
-       int type;
-       ino_t ino;
+dumpmap(char *map, int type, ino_t ino)
 {
        register int i;
        char *cp;
@@ -953,13 +1030,12 @@ dumpmap(map, type, ino)
  * Write a header record to the dump tape.
  */
 void
-writeheader(ino)
-       ino_t ino;
+writeheader(ino_t ino)
 {
 #ifdef __linux__
        register __s32 sum, cnt, *lp;
 #else
-       register long sum, cnt, *lp;
+       register int32_t sum, cnt, *lp;
 #endif
 
        spcl.c_inumber = ino;
@@ -968,13 +1044,13 @@ writeheader(ino)
 #ifdef __linux__
        lp = (__s32 *)&spcl;
 #else
-       lp = (long *)&spcl;
+       lp = (int32_t *)&spcl;
 #endif
        sum = 0;
 #ifdef __linux__
        cnt = sizeof(union u_spcl) / (4 * sizeof(__s32));
 #else
-       cnt = sizeof(union u_spcl) / (4 * sizeof(long));
+       cnt = sizeof(union u_spcl) / (4 * sizeof(int32_t));
 #endif
        while (--cnt >= 0) {
                sum += *lp++;
@@ -988,8 +1064,7 @@ writeheader(ino)
 
 #ifdef __linux__
 struct dinode *
-getino(inum)
-       ino_t inum;
+getino(ino_t inum)
 {
        static struct dinode dinode;
 
@@ -999,8 +1074,7 @@ getino(inum)
 }
 #else  /* __linux__ */
 struct dinode *
-getino(inum)
-       ino_t inum;
+getino(ino_t inum)
 {
        static daddr_t minino, maxino;
        static struct dinode inoblock[MAXINOPB];
@@ -1022,24 +1096,22 @@ getino(inum)
  * Error recovery is attempted at most BREADEMAX times before seeking
  * consent from the operator to continue.
  */
-int    breaderrors = 0;                
+int    breaderrors = 0;
 #define        BREADEMAX 32
 
 void
-bread(blkno, buf, size)
-       daddr_t blkno;
-       char *buf;
-       int size;       
+bread(daddr_t blkno, char *buf, int size)
 {
        int cnt, i;
        extern int errno;
 
 loop:
 #ifdef __linux__
-       if (ext2_llseek(diskfd, ((ext2_loff_t)blkno << dev_bshift), 0) !=
-                       ((ext2_loff_t)blkno << dev_bshift))
+       if (ext2fs_llseek(diskfd, (((ext2_loff_t)blkno) << dev_bshift), 0) !=
+                       (((ext2_loff_t)blkno) << dev_bshift))
 #else
-       if ((int)lseek(diskfd, ((off_t)blkno << dev_bshift), 0) == -1)
+       if (lseek(diskfd, ((off_t)blkno << dev_bshift), 0) !=
+                                               ((off_t)blkno << dev_bshift))
 #endif
                msg("bread: lseek fails\n");
        if ((cnt = read(diskfd, buf, size)) == size)
@@ -1081,10 +1153,11 @@ loop:
        memset(buf, 0, size);
        for (i = 0; i < size; i += dev_bsize, buf += dev_bsize, blkno++) {
 #ifdef __linux__
-               if (ext2_llseek(diskfd, ((ext2_loff_t)blkno << dev_bshift), 0) !=
-                               ((ext2_loff_t)blkno << dev_bshift))
+               if (ext2fs_llseek(diskfd, (((ext2_loff_t)blkno) << dev_bshift), 0) !=
+                               (((ext2_loff_t)blkno) << dev_bshift))
 #else
-               if ((int)lseek(diskfd, ((off_t)blkno << dev_bshift), 0) == -1)
+               if (lseek(diskfd, ((off_t)blkno << dev_bshift), 0) !=
+                                               ((off_t)blkno << dev_bshift))
 #endif
                        msg("bread: lseek2 fails!\n");
                if ((cnt = read(diskfd, buf, (int)dev_bsize)) == dev_bsize)