linux-mips
[Top] [All Lists]

[PATCH] [MIPS] Invalid semicolon after if statement

To: ralf@linux-mips.org
Subject: [PATCH] [MIPS] Invalid semicolon after if statement
From: "Ilpo Järvinen" <ilpo.jarvinen@helsinki.fi>
Date: Thu, 16 Aug 2007 01:03:01 +0300 (EEST)
Cc: linux-mips@linux-mips.org
Original-recipient: rfc822;linux-mips@linux-mips.org
Sender: linux-mips-bounce@linux-mips.org
A similar fix to netfilter from Eric Dumazet inspired me to
look around a bit by using some grep/sed stuff as looking for
this kind of bugs seemed easy to automate. This is one of them
I found where it looks like this semicolon is not valid.

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
---

...Since I'm not familiar with these parts of the kernel, you might know 
better than I do if this is stuff is valid...

 arch/mips/kernel/irixsig.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/mips/kernel/irixsig.c b/arch/mips/kernel/irixsig.c
index 6980deb..28b2a8f 100644
--- a/arch/mips/kernel/irixsig.c
+++ b/arch/mips/kernel/irixsig.c
@@ -725,7 +725,7 @@ asmlinkage int irix_getcontext(struct pt_regs *regs)
               current->comm, current->pid, ctx);
 #endif
 
-       if (!access_ok(VERIFY_WRITE, ctx, sizeof(*ctx)));
+       if (!access_ok(VERIFY_WRITE, ctx, sizeof(*ctx)))
                return -EFAULT;
 
        error = __put_user(current->thread.irix_oldctx, &ctx->link);
-- 
1.5.0.6
<Prev in Thread] Current Thread [Next in Thread>