Hi, On Tue, Dec 30, 2014 at 11:57 AM, Syam Sidhardhan <s.syam@xxxxxxxxxxx> wrote: > return statement never get executed after goto. > --- > tools/bluemoon.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/tools/bluemoon.c b/tools/bluemoon.c > index 1fd516a..663d1f9 100644 > --- a/tools/bluemoon.c > +++ b/tools/bluemoon.c > @@ -776,7 +776,6 @@ static void analyze_firmware(const char *path) > if (len != st.st_size) { > fprintf(stderr, "Failed to read complete firmware file\n"); > goto done; > - return; > } > > if ((size_t) len < sizeof(*css)) { > -- ping -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html