From: Eric Sesterhenn This fixes coverity bug #517. Since IESIZE is greater than IESIZE_NI1 we might run past the end of ielist_ni1. This fixes it by using the proper IESIZE_NI1 define. Signed-off-by: Eric Sesterhenn Cc: Karsten Keil Signed-off-by: Andrew Morton --- drivers/isdn/hisax/q931.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -puN drivers/isdn/hisax/q931.c~fix-typo-in-drivers-isdn-hisax-q931c drivers/isdn/hisax/q931.c --- devel/drivers/isdn/hisax/q931.c~fix-typo-in-drivers-isdn-hisax-q931c 2006-06-06 01:14:05.000000000 -0700 +++ devel-akpm/drivers/isdn/hisax/q931.c 2006-06-06 01:14:05.000000000 -0700 @@ -1402,12 +1402,12 @@ dlogframe(struct IsdnCardState *cs, stru } /* No, locate it in the table */ if (cset == 0) { - for (i = 0; i < IESIZE; i++) + for (i = 0; i < IESIZE_NI1; i++) if (*buf == ielist_ni1[i].nr) break; /* When not found, give appropriate msg */ - if (i != IESIZE) { + if (i != IESIZE_NI1) { dp += sprintf(dp, " %s\n", ielist_ni1[i].descr); dp += ielist_ni1[i].f(dp, buf); } else _