On Sat, 2013-03-02 at 23:10 +0000, Ben Hutchings wrote: > On Sat, 2013-03-02 at 23:35 +0100, Paul Bolle wrote: > > 1) So if xen_vbd_translate() fails, it can return before setting > > preq.dev. That makes the call of pr_debug() use an uninitialized value, > > doesn't it? > > Oh yes, so it's a completely valid warning in this case! Thanks. Perhaps Konrad, Jan, Or Ian can tell whether the patch still needs to go in stable as is, because the problem it fixes is more severe than the problem it apparently creates. Maybe a mainline fix is needed before this can go in, or perhaps even a stable specific fix (if context changes are to blame). Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html