du, copy_file: fix file matching on cramfs. Closes 5456

function                                             old     new   delta
is_in_ino_dev_hashtable                               88     108     +20
add_to_ino_dev_hashtable                             150     142      -8
reset_ino_dev_hashtable                               84      75      -9
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/2 up/down: 20/-17)              Total: 3 bytes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

1 files changed, 15 insertions, 4 deletions

diff --git a/libbb/inode_hash.c b/libbb/inode_hash.c
index 2f02d0746..f11c2afb2 100644
--- a/libbb/inode_hash.c
+++ b/libbb/inode_hash.c
@@ -11,14 +11,23 @@
 #include "libbb.h"
 
 typedef struct ino_dev_hash_bucket_struct {
-        struct ino_dev_hash_bucket_struct *next;
         ino_t ino;
         dev_t dev;
+        /*
+         * Above fields can be 64-bit, while pointer may be 32-bit.
+         * Putting "next" field here may reduce size of this struct:
+         */
+        struct ino_dev_hash_bucket_struct *next;
+        /*
+         * Reportedly, on cramfs a file and a dir can have same ino.
+         * Need to also remember "file/dir" bit:
+         */
+        char isdir; /* bool */
         char name[1];
 } ino_dev_hashtable_bucket_t;
 
-#define HASH_SIZE      311   /* Should be prime */
-#define hash_inode(i)  ((i) % HASH_SIZE)
+#define HASH_SIZE      311u   /* Should be prime */
+#define hash_inode(i)  ((unsigned)(i) % HASH_SIZE)
 
 /* array of [HASH_SIZE] elements */
 static ino_dev_hashtable_bucket_t **ino_dev_hashtable;
@@ -38,6 +47,7 @@ char* FAST_FUNC is_in_ino_dev_hashtable(const struct stat *statbuf)
         while (bucket != NULL) {
                 if ((bucket->ino == statbuf->st_ino)
                  && (bucket->dev == statbuf->st_dev)
+                 && (bucket->isdir == !!S_ISDIR(statbuf->st_mode))
                 ) {
                         return bucket->name;
                 }
@@ -52,17 +62,18 @@ void FAST_FUNC add_to_ino_dev_hashtable(const struct stat *statbuf, const char *
         int i;
         ino_dev_hashtable_bucket_t *bucket;
 
-        i = hash_inode(statbuf->st_ino);
         if (!name)
                 name = "";
         bucket = xmalloc(sizeof(ino_dev_hashtable_bucket_t) + strlen(name));
         bucket->ino = statbuf->st_ino;
         bucket->dev = statbuf->st_dev;
+        bucket->isdir = !!S_ISDIR(statbuf->st_mode);
         strcpy(bucket->name, name);
 
         if (!ino_dev_hashtable)
                 ino_dev_hashtable = xzalloc(HASH_SIZE * sizeof(*ino_dev_hashtable));
 
+        i = hash_inode(statbuf->st_ino);
         bucket->next = ino_dev_hashtable[i];
         ino_dev_hashtable[i] = bucket;
 }