>>> On 11.01.17 at 18:57, <andrew.coop...@citrix.com> wrote:
> --- a/xen/arch/x86/sysctl.c
> +++ b/xen/arch/x86/sysctl.c
> @@ -229,7 +229,10 @@ long arch_do_sysctl(
>
> /* Bad featureset index? */
> if ( !p )
> + {
> ret = -EINVAL;
> + break;
> + }
>
> cpuid_policy_to_featureset(p, featureset);
Considering how the following code is written, adding an "else"
would seem more natural. With the patch above the !ret check
right out of context would then be dead code. With either of
the possible adjustments
Reviewed-by: Jan Beulich <jbeul...@suse.com>
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
