* [PATCH] ROMFS: romfs_iget() - unsigned ino >= 0 is always true
@ 2008-11-29 16:25 roel kluin
0 siblings, 0 replies; only message in thread
From: roel kluin @ 2008-11-29 16:25 UTC (permalink / raw)
To: linux-kernel
romfs_strnlen() returns int
unsigned X >= 0 is always true
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
---
for romfs_strnlen(), see
vi fs/romfs/inode.c +200
diff --git a/fs/romfs/inode.c b/fs/romfs/inode.c
index 60d2f82..28b180b 100644
--- a/fs/romfs/inode.c
+++ b/fs/romfs/inode.c
@@ -490,7 +490,7 @@ static mode_t romfs_modemap[] =
static struct inode *
romfs_iget(struct super_block *sb, unsigned long ino)
{
- int nextfh;
+ int nextfh, ret;
struct romfs_inode ri;
struct inode *i;
@@ -527,9 +527,9 @@ romfs_iget(struct super_block *sb, unsigned long ino)
i->i_uid = i->i_gid = 0;
/* Precalculate the data offset */
- ino = romfs_strnlen(i, ino+ROMFH_SIZE, ROMFS_MAXFN);
- if (ino >= 0)
- ino = ((ROMFH_SIZE+ino+1+ROMFH_PAD)&ROMFH_MASK);
+ ret = romfs_strnlen(i, ino+ROMFH_SIZE, ROMFS_MAXFN);
+ if (ret >= 0)
+ ino = ((ROMFH_SIZE+ret+1+ROMFH_PAD)&ROMFH_MASK);
else
ino = 0;
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2008-11-29 16:26 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-29 16:25 [PATCH] ROMFS: romfs_iget() - unsigned ino >= 0 is always true roel kluin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.