2024-04-09 10:35:18

by Thorsten Blum

[permalink] [raw]
Subject: [PATCH] microblaze: Improve data type of return variable

Since restore_sigcontext() returns an int and err is its return variable,
which can be negative, it should be of type int instead of unsigned int.

Signed-off-by: Thorsten Blum <[email protected]>
---
arch/microblaze/kernel/signal.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/microblaze/kernel/signal.c b/arch/microblaze/kernel/signal.c
index c78a0ff48066..51f718a42e2c 100644
--- a/arch/microblaze/kernel/signal.c
+++ b/arch/microblaze/kernel/signal.c
@@ -57,7 +57,7 @@ struct rt_sigframe {
static int restore_sigcontext(struct pt_regs *regs,
struct sigcontext __user *sc, int *rval_p)
{
- unsigned int err = 0;
+ int err = 0;

#define COPY(x) {err |= __get_user(regs->x, &sc->regs.x); }
COPY(r0);
--
2.44.0