Re: [PATCH] 2.5.56 sound/oss/sb_mixer.c bounds check

From: ryan (ryan@ryanflynn.com)
Date: Tue Jan 14 2003 - 23:21:06 EST


> Yup.
>
> It would be better to do:
>
> if (dev < 0 || dev >= ARRAY_SIZE(smw_mix_regs))

yup, much better. i did a little housecleaning on the whole file, as
well as 2 more bounds checks in appropriate places.

i'm sorry to ask, but i'm new -- i've got a ~500 line patch, and my
email client is wrapping at 80 chars (unfortunately some lines run over
80 chars), is sending an attachment in ascii format ok? i've seen some
patches sent as attachments, not sure.

waiting for a yes/no response, please cc me

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Jan 15 2003 - 22:00:53 EST