On Wednesday, April 18, 2012 12:37 PM, Julia Lawall wrote: > From: Julia Lawall <Julia.Lawall@xxxxxxx> > > There were two problems in this code: failure of the setup function should > free locally allocated resources like other nearby failures, and the test > if (&info->cmap) can never be false. To generally clean things up, this > patch reorders the error handling code at the failed label and adds labels > so that the conditionals are not necessary. > > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> > Thanks! Acked-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html