diff options
author | Jim Meyering <meyering@redhat.com> | 2011-05-25 12:29:18 +0200 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2011-05-26 22:13:37 +0200 |
commit | 419b6c9d42ba643265f802cd150d0b232e43186a (patch) | |
tree | 993d42ce541942f5863de308c98ff773cf29ab32 /src/dircolors.c | |
parent | d7e23652039b5e145ab0d809a7de75ebb3facc24 (diff) | |
download | coreutils-419b6c9d42ba643265f802cd150d0b232e43186a.tar.xz |
maint: accommodate gcc's -Wstrict-overflow option
* src/factor.c (factor_using_pollard_rho): Change type of "i"
to unsigned to avoid warning from gcc's -Wstrict-overflow.
* src/expr.c: Use an unsigned intermediate.
* src/dircolors.c (main): Reorder operations to avoid the risk of
pointer overflow.
* src/tr.c (squeeze_filter): Change NOT_A_CHAR from an anonymous
"enum" to an "int", to avoid this warning:
tr.c:1624:10: error: assuming signed overflow does not occur when
simplifying conditional to constant [-Werror=strict-overflow]
* src/pr.c (main): Make index "i" unsigned.
Diffstat (limited to 'src/dircolors.c')
-rw-r--r-- | src/dircolors.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/dircolors.c b/src/dircolors.c index d1962eae9..9aaa87fc2 100644 --- a/src/dircolors.c +++ b/src/dircolors.c @@ -456,7 +456,7 @@ to select a shell syntax are mutually exclusive")); if (print_database) { char const *p = G_line; - while (p < G_line + sizeof G_line) + while (p - G_line < sizeof G_line) { puts (p); p += strlen (p) + 1; |