On 23/01/2015 15:32, Dan Carpenter wrote: > Smatch complains that there are some paths where we use uninitialized > data in em_sysenter(). > > arch/x86/kvm/emulate.c:2410 em_sysenter() > error: potentially using uninitialized 'msr_data'. The right thing to do is to ensure that the value is set if get_msr returns 0. If it returns 1, msr_data is not used. This one is obviously a false positive, so I'm not applying the patch for now. Paolo > A couple examples of paths which don't set "pdata" are found in > get_msr_hyperv() and kvm_x2apic_msr_read(). I looked at this code and > it seems like setting it to zero is a common default behaviour. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > From static analysis only, not from real life. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
