* [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one
@ 2006-03-11 3:42 Adrian Bunk
2006-03-11 4:03 ` Randy.Dunlap
0 siblings, 1 reply; 3+ messages in thread
From: Adrian Bunk @ 2006-03-11 3:42 UTC (permalink / raw)
To: tim; +Cc: linux-parport, linux-kernel
The Coverity checker found this off-by-one error.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
--- linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c.old 2006-03-11 02:07:21.000000000 +0100
+++ linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c 2006-03-11 02:07:50.000000000 +0100
@@ -275,7 +275,7 @@
int i;
printk("%s: %s: status = 0x%x =", disk->name, msg, status);
- for (i = 0; i < 18; i++)
+ for (i = 0; i < 17; i++)
if (status & (1 << i))
printk(" %s", pd_errs[i]);
printk("\n");
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one
2006-03-11 3:42 [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one Adrian Bunk
@ 2006-03-11 4:03 ` Randy.Dunlap
2006-03-11 15:11 ` [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one error Adrian Bunk
0 siblings, 1 reply; 3+ messages in thread
From: Randy.Dunlap @ 2006-03-11 4:03 UTC (permalink / raw)
To: Adrian Bunk; +Cc: tim, linux-parport, linux-kernel
On Sat, 11 Mar 2006 04:42:53 +0100 Adrian Bunk wrote:
> The Coverity checker found this off-by-one error.
>
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
>
> --- linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c.old 2006-03-11 02:07:21.000000000 +0100
> +++ linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c 2006-03-11 02:07:50.000000000 +0100
> @@ -275,7 +275,7 @@
> int i;
>
> printk("%s: %s: status = 0x%x =", disk->name, msg, status);
> - for (i = 0; i < 18; i++)
> + for (i = 0; i < 17; i++)
> if (status & (1 << i))
> printk(" %s", pd_errs[i]);
> printk("\n");
Please use ARRAY_SIZE(pd_errs)
and #include <linux/kernel.h>
---
~Randy
Please use an email client that implements proper (compliant) threading.
(You know who you are.)
^ permalink raw reply [flat|nested] 3+ messages in thread
* [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one error
2006-03-11 4:03 ` Randy.Dunlap
@ 2006-03-11 15:11 ` Adrian Bunk
0 siblings, 0 replies; 3+ messages in thread
From: Adrian Bunk @ 2006-03-11 15:11 UTC (permalink / raw)
To: Randy.Dunlap; +Cc: tim, linux-parport, linux-kernel
On Fri, Mar 10, 2006 at 08:03:50PM -0800, Randy.Dunlap wrote:
> On Sat, 11 Mar 2006 04:42:53 +0100 Adrian Bunk wrote:
>
> > The Coverity checker found this off-by-one error.
> >
> >
> > Signed-off-by: Adrian Bunk <bunk@stusta.de>
> >
> > --- linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c.old 2006-03-11 02:07:21.000000000 +0100
> > +++ linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c 2006-03-11 02:07:50.000000000 +0100
> > @@ -275,7 +275,7 @@
> > int i;
> >
> > printk("%s: %s: status = 0x%x =", disk->name, msg, status);
> > - for (i = 0; i < 18; i++)
> > + for (i = 0; i < 17; i++)
> > if (status & (1 << i))
> > printk(" %s", pd_errs[i]);
> > printk("\n");
>
> Please use ARRAY_SIZE(pd_errs)
> and #include <linux/kernel.h>
Sounds reasonable, updated patch below.
> ~Randy
cu
Adrian
<-- snip -->
The Coverity checker found this off-by-one error.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
drivers/block/paride/pd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
--- linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c.old 2006-03-11 02:07:21.000000000 +0100
+++ linux-2.6.16-rc5-mm3-full/drivers/block/paride/pd.c 2006-03-11 15:36:00.000000000 +0100
@@ -151,6 +151,7 @@ enum {D_PRT, D_PRO, D_UNI, D_MOD, D_GEO,
#include <linux/cdrom.h> /* for the eject ioctl */
#include <linux/blkdev.h>
#include <linux/blkpg.h>
+#include <linux/kernel.h>
#include <asm/uaccess.h>
#include <linux/sched.h>
#include <linux/workqueue.h>
@@ -275,7 +276,7 @@ static void pd_print_error(struct pd_uni
int i;
printk("%s: %s: status = 0x%x =", disk->name, msg, status);
- for (i = 0; i < 18; i++)
+ for (i = 0; i < ARRAY_SIZE(pd_errs); i++)
if (status & (1 << i))
printk(" %s", pd_errs[i]);
printk("\n");
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-03-11 15:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-03-11 3:42 [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one Adrian Bunk
2006-03-11 4:03 ` Randy.Dunlap
2006-03-11 15:11 ` [2.6 patch] drivers/block/paride/pd.c: fix an off-by-one error Adrian Bunk
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.