Re: [PATCH 16/16] fix handling of integer constant expressions

From: Linus Torvalds
Date: Tue Jun 26 2007 - 20:25:39 EST




On Wed, 27 Jun 2007, Al Viro wrote:
>
> > extern int a;
> > extern int as1[(a = 2)];
>
> sparse simply doesn't check that. We don't have anything resembling
> support of VLA.

Well, the above has two bugs that sparse could notice _independently_ of
variable-sized arrays:
- assignment outside of a function
- variable size array that isn't an automatic variable

(strictly speaking, that's not even a variable size - it's a constant 2,
just with a non-constant expression - maybe you misread the "=" as an
"==")

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