On 06/18/2012 06:55 PM, Herton Ronaldo Krzesinski wrote: >> + default: >> + WARN_ONCE(1, "unknown plane in flip command\n"); >> + ret = -ENODEV; >> + goto err; > > Shouldn't this have been "goto err_unpin;"? > > It's not a problem in the backport for 3.2, same code is in the original > patch, just noticed it seems a problem while looking at it; but by what > changelog says the default case is said to not trigger in "practice" > (catch programming error), so I think it doesn't pose a real issue in > practice.
You are right, it should be err_unpin. I sent a patch rectifying this. In practice, this should not happen, but still, better be safe than sorry. Thanks for catching that! Eugeni -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
