From ce33fefd5c0e4238855551e6f5f75fc106f141b7 Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Mon, 20 May 2019 12:18:54 +0200 Subject: cleanup: remove squashfs type from fstree nodes Signed-off-by: David Oberhollenzer --- include/fstree.h | 14 --------- include/squashfs.h | 2 +- mkfs/meta.c | 70 +++++++++++++++++++------------------------ unpack/tree_node_from_inode.c | 1 - 4 files changed, 31 insertions(+), 56 deletions(-) diff --git a/include/fstree.h b/include/fstree.h index 27cb39d..6a29093 100644 --- a/include/fstree.h +++ b/include/fstree.h @@ -133,20 +133,6 @@ struct tree_node_t { /* Inode number. Generated on the fly when writing inodes. */ uint32_t inode_num; - /* SquashFS inode type used for this tree node. - - Generated on the fly when writing inodes. It can't be easily - determined in advance since it depends also on the size of the - node, which means for directories the size of the directory - entries once written to disk. - - All code that actually processes tree nodes should use the mode - field instead (mode & S_IFMT gives us the node type). It is stored - here when generating inodes since we need it later on to generate - directory entries. - */ - int type; - /* Type specific data. Pointers are into payload area blow. */ union { dir_info_t *dir; diff --git a/include/squashfs.h b/include/squashfs.h index f9df73d..ed3fd38 100644 --- a/include/squashfs.h +++ b/include/squashfs.h @@ -13,7 +13,7 @@ #define SQFS_DEVBLK_SIZE 4096 #define SQFS_MAX_DIR_ENT 256 -#define SQFS_DIR_TYPE_MOD 8 +#define SQFS_INODE_EXT_TYPE(type) ((type) - 1 + 8) typedef struct { uint32_t magic; diff --git a/mkfs/meta.c b/mkfs/meta.c index ee37054..a0ea071 100644 --- a/mkfs/meta.c +++ b/mkfs/meta.c @@ -18,6 +18,21 @@ typedef struct { idx_ref_t idx_nodes[]; } dir_index_t; +static int get_type(mode_t mode) +{ + switch (mode & S_IFMT) { + case S_IFSOCK: return SQFS_INODE_SOCKET; + case S_IFIFO: return SQFS_INODE_FIFO; + case S_IFLNK: return SQFS_INODE_SLINK; + case S_IFBLK: return SQFS_INODE_BDEV; + case S_IFCHR: return SQFS_INODE_CDEV; + case S_IFDIR: return SQFS_INODE_DIR; + case S_IFREG: return SQFS_INODE_FILE; + default: + assert(0); + } +} + static int dir_index_grow(dir_index_t **index) { size_t size = sizeof(dir_index_t) + sizeof(idx_ref_t) * 10; @@ -119,7 +134,7 @@ static int write_dir(meta_writer_t *dm, dir_info_t *dir, dir_index_t **index) for (i = 0; i < count; ++i) { ent.offset = htole16(c->inode_ref & 0x0000FFFF); ent.inode_number = htole16(c->inode_num - d->inode_num); - ent.type = htole16(c->type % SQFS_DIR_TYPE_MOD); + ent.type = htole16(get_type(c->mode)); ent.size = htole16(strlen(c->name) - 1); dir->size += sizeof(ent) + strlen(c->name); @@ -144,6 +159,7 @@ static int write_inode(sqfs_info_t *info, meta_writer_t *im, meta_writer_t *dm, uint32_t bs, offset; sqfs_inode_t base; uint64_t i, block; + int type; if (id_table_id_to_index(&info->idtbl, node->uid, &uid_idx)) return -1; @@ -157,61 +173,35 @@ static int write_inode(sqfs_info_t *info, meta_writer_t *im, meta_writer_t *dm, info->super.inode_count += 1; - switch (node->mode & S_IFMT) { - case S_IFSOCK: - node->type = SQFS_INODE_SOCKET; - if (node->xattr != NULL) - node->type = SQFS_INODE_EXT_SOCKET; - break; - case S_IFIFO: - node->type = SQFS_INODE_FIFO; - if (node->xattr != NULL) - node->type = SQFS_INODE_EXT_FIFO; - break; - case S_IFLNK: - node->type = SQFS_INODE_SLINK; - if (node->xattr != NULL) - node->type = SQFS_INODE_EXT_SLINK; - break; - case S_IFBLK: - node->type = SQFS_INODE_BDEV; - if (node->xattr != NULL) - node->type = SQFS_INODE_EXT_BDEV; - break; - case S_IFCHR: - node->type = SQFS_INODE_CDEV; - if (node->xattr != NULL) - node->type = SQFS_INODE_EXT_CDEV; - break; - case S_IFDIR: + type = get_type(node->mode); + + if (node->xattr != NULL) + type = SQFS_INODE_EXT_TYPE(type); + + if (S_ISDIR(node->mode)) { di = node->data.dir; - node->type = SQFS_INODE_DIR; if (write_dir(dm, di, &diridx)) return -1; if ((di->start_block) > 0xFFFFFFFFUL || di->size > 0xFFFF || (node->xattr != NULL && di->size != 0)) { - node->type = SQFS_INODE_EXT_DIR; + type = SQFS_INODE_EXT_DIR; } else { + type = SQFS_INODE_DIR; free(diridx); diridx = NULL; } - break; - case S_IFREG: + } else if (S_ISREG(node->mode)) { fi = node->data.file; - node->type = SQFS_INODE_FILE; if (fi->startblock > 0xFFFFFFFFUL || fi->size > 0xFFFFFFFFUL || - hard_link_count(node) > 1 || node->xattr != NULL) { - node->type = SQFS_INODE_EXT_FILE; + hard_link_count(node) > 1) { + type = SQFS_INODE_EXT_FILE; } - break; - default: - assert(0); } - base.type = htole16(node->type); + base.type = htole16(type); base.mode = htole16(node->mode); base.uid_idx = htole16(uid_idx); base.gid_idx = htole16(gid_idx); @@ -223,7 +213,7 @@ static int write_inode(sqfs_info_t *info, meta_writer_t *im, meta_writer_t *dm, return -1; } - switch (node->type) { + switch (type) { case SQFS_INODE_FIFO: case SQFS_INODE_SOCKET: { sqfs_inode_ipc_t ipc = { diff --git a/unpack/tree_node_from_inode.c b/unpack/tree_node_from_inode.c index 2e720d3..0fd7da1 100644 --- a/unpack/tree_node_from_inode.c +++ b/unpack/tree_node_from_inode.c @@ -72,7 +72,6 @@ tree_node_t *tree_node_from_inode(sqfs_inode_generic_t *inode, out->uid = idtbl->ids[inode->base.uid_idx]; out->gid = idtbl->ids[inode->base.gid_idx]; out->mode = inode->base.mode; - out->type = inode->base.type; out->inode_num = inode->base.inode_number; out->name = (char *)out->payload; -- cgit v1.2.3