On Thu, Jan 15, 2015 at 1:23 AM, Stephen Boyd <sboyd@xxxxxxxxxxxxxx> wrote: > On 01/14/2015 03:54 PM, Timur Tabi wrote: >> On Wed, Jan 14, 2015 at 5:44 PM, Stephen Boyd <sboyd@xxxxxxxxxxxxxx> wrote: >>> I think so. Can you send a patch? Plus it would be nice to move i = 0 in >>> the for loop at the same time. >> Sadly, Qualcomm policy makes that very cumbersome for me. I won't be >> able to send a patch any time soon. >> > > Hm... we'll have to figure out that part. Anyway here's the patch. > > ---8<---- > From: Stephen Boyd <sboyd@xxxxxxxxxxxxxx> > Subject: [PATCH] pinctrl: qcom: Don't iterate past end of function array > > Timur reports that this code crashes if nfunctions is 0. Fix the > loop iteration to only consider valid elements of the functions > array. > > Reported-by: Timur Tabi <timur@xxxxxxxxxxxxxx> > Cc: Pramod Gurav <pramod.gurav@xxxxxxxxxxxxxxx> > Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxxxxxx> > Cc: Ivan T. Ivanov <iivanov@xxxxxxxxxx> > Cc: Andy Gross <agross@xxxxxxxxxxxxxx> > Fixes: 327455817a92 "pinctrl: qcom: Add support for reset for apq8064" > Signed-off-by: Stephen Boyd <sboyd@xxxxxxxxxxxxxx> This patch applied for fixes. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
