]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
ext4: remove useless test and assignment in strtohash functions
authorDamien Guibouret <damien.guibouret@partition-saving.com>
Thu, 24 Aug 2017 19:11:34 +0000 (15:11 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 24 Aug 2017 19:11:34 +0000 (15:11 -0400)
On transformation of str to hash, computed value is initialised before
first byte modulo 4. But it is already initialised before entering loop
and after processing last byte modulo 4. So the corresponding test and
initialisation could be removed.

Signed-off-by: Damien Guibouret <damien.guibouret@partition-saving.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/hash.c

index 38b8a96eb97c2f59932993e704b15a4b6b89e207..00c6dd29e621f91a0adab2c1acfef4096347f810 100644 (file)
@@ -148,8 +148,6 @@ static void str2hashbuf_signed(const char *msg, int len, __u32 *buf, int num)
        if (len > num*4)
                len = num * 4;
        for (i = 0; i < len; i++) {
-               if ((i % 4) == 0)
-                       val = pad;
                val = ((int) scp[i]) + (val << 8);
                if ((i % 4) == 3) {
                        *buf++ = val;
@@ -176,8 +174,6 @@ static void str2hashbuf_unsigned(const char *msg, int len, __u32 *buf, int num)
        if (len > num*4)
                len = num * 4;
        for (i = 0; i < len; i++) {
-               if ((i % 4) == 0)
-                       val = pad;
                val = ((int) ucp[i]) + (val << 8);
                if ((i % 4) == 3) {
                        *buf++ = val;