There's a function-wide variable rc, so no need to re-declare it in individual case handling blocks. Signed-off-by: Jan Beulich --- 2010-01-06.orig/xen/arch/x86/mm.c 2010-01-06 11:23:46.000000000 +0100 +++ 2010-01-06/xen/arch/x86/mm.c 2010-01-12 12:03:44.000000000 +0100 @@ -4237,7 +4237,6 @@ long arch_memory_op(int op, XEN_GUEST_HA struct xen_add_to_physmap xatp; unsigned long prev_mfn, mfn = 0, gpfn; struct domain *d; - int rc; if ( copy_from_guest(&xatp, arg, 1) ) return -EFAULT; @@ -4349,7 +4348,6 @@ long arch_memory_op(int op, XEN_GUEST_HA { struct xen_foreign_memory_map fmap; struct domain *d; - int rc; if ( copy_from_guest(&fmap, arg, 1) ) return -EFAULT; @@ -4401,7 +4399,6 @@ long arch_memory_op(int op, XEN_GUEST_HA struct memory_map_context ctxt; XEN_GUEST_HANDLE(e820entry_t) buffer; unsigned int i; - int rc; if ( !IS_PRIV(current->domain) ) return -EINVAL;